feat: support RegExp for matchId

Author: simonihmig

addon/index.js

@@ -37,11 +37,12 @@ export function detectWorkflow(config, message, options) {
     matcher = workflow.matchMessage;
     idMatcher = workflow.matchId;
 
-    if (typeof idMatcher === 'string' && options && idMatcher === options.id) {
-      return workflow;
-    } else if (typeof matcher === 'string' && matcher === message) {
-      return workflow;
-    } else if (matcher instanceof RegExp && matcher.exec(message)) {
+    if (
+      (typeof idMatcher === 'string' && options && idMatcher === options.id) ||
+      (idMatcher instanceof RegExp && idMatcher.exec(options.id)) ||
+      (typeof matcher === 'string' && matcher === message) ||
+      (matcher instanceof RegExp && matcher.exec(message))
+    ) {
       return workflow;
     }
   }

tests/unit/handle-deprecation-workflow-test.js

@@ -318,4 +318,76 @@ module('handleDeprecationWorkflow', function (hooks) {
       );
     }, 'deprecation throws');
   });
+
+  test('deprecation silenced with id regex', function (assert) {
+    const config = {
+      throwOnUnhandled: true,
+      workflow: [{ matchId: /^ember\..*/, handler: 'silence' }],
+    };
+
+    handleDeprecationWorkflow(
+      config,
+      'Slightly interesting',
+      {
+        id: 'ember.deprecation-workflow',
+        since: 'the beginning',
+        until: '3.0.0',
+        for: 'testing',
+      },
+      () => {},
+    );
+
+    assert.ok(true, 'Deprecation did not raise');
+  });
+
+  // eslint-disable-next-line qunit/require-expect
+  test('deprecation logs with id regex', function (assert) {
+    assert.expect(1);
+
+    let message = 'Slightly interesting';
+
+    console.warn = function (passedMessage) {
+      assert.strictEqual(
+        passedMessage,
+        'DEPRECATION: ' + message,
+        'deprecation logs',
+      );
+    };
+
+    const config = {
+      throwOnUnhandled: true,
+      workflow: [{ matchId: /^ember\..*/, handler: 'log' }],
+    };
+
+    handleDeprecationWorkflow(
+      config,
+      'Slightly interesting',
+      {
+        id: 'ember.deprecation-workflow',
+        since: 'the beginning',
+        until: '3.0.0',
+        for: 'testing',
+      },
+      () => {},
+    );
+  });
+
+  test('deprecation thrown with id regex', function (assert) {
+    const config = {
+      workflow: [{ matchId: /^ember\..*/, handler: 'throw' }],
+    };
+    assert.throws(function () {
+      handleDeprecationWorkflow(
+        config,
+        'Slightly interesting',
+        {
+          id: 'ember.deprecation-workflow',
+          since: 'the beginning',
+          until: '3.0.0',
+          for: 'testing',
+        },
+        () => {},
+      );
+    }, 'deprecation throws');
+  });
 });