Fix lint

Author: lolmaus

addon/index.js

@@ -9,7 +9,7 @@ export default function setupDeprecationWorkflow(config) {
   };
 
   registerDeprecationHandler((message, options, next) =>
-    handleDeprecationWorkflow(config, message, options, next)
+    handleDeprecationWorkflow(config, message, options, next),
   );
 
   registerDeprecationHandler(deprecationCollector);
@@ -87,7 +87,7 @@ export function handleDeprecationWorkflow(config, message, options, next) {
           console.warn('DEPRECATION: ' + message);
           if (count === LOG_LIMIT) {
             console.warn(
-              'To avoid console overflow, this deprecation will not be logged any more in this run.'
+              'To avoid console overflow, this deprecation will not be logged any more in this run.',
             );
           }
         }

tests/acceptance/flush-deprecations-test.js

@@ -61,7 +61,7 @@ setupDeprecationWorkflow({
     { handler: "silence", matchId: "test" },
     { handler: "silence", matchId: "log-strict" }
   ]
-});`
+});`,
     );
   });
 });

tests/unit/deprecation-collector-test.js

@@ -37,7 +37,7 @@ module('deprecationCollector', function (hooks) {
         until: 'forever',
         for: 'testing',
       },
-      () => {}
+      () => {},
     );
     deprecationCollector(
       'Second deprecation',
@@ -47,7 +47,7 @@ module('deprecationCollector', function (hooks) {
         until: 'forever',
         for: 'testing',
       },
-      () => {}
+      () => {},
     );
 
     assert.deepEqual(self.deprecationWorkflow.deprecationLog.messages, {
@@ -71,7 +71,7 @@ module('deprecationCollector', function (hooks) {
         until: 'forever',
         for: 'testing',
       },
-      next
+      next,
     );
   });
 
@@ -84,7 +84,7 @@ module('deprecationCollector', function (hooks) {
         until: 'forever',
         for: 'testing',
       },
-      () => {}
+      () => {},
     );
     deprecationCollector(
       'Second deprecation',
@@ -94,7 +94,7 @@ module('deprecationCollector', function (hooks) {
         until: 'forever',
         for: 'testing',
       },
-      () => {}
+      () => {},
     );
 
     // do it again
@@ -106,7 +106,7 @@ module('deprecationCollector', function (hooks) {
         until: 'forever',
         for: 'testing',
       },
-      () => {}
+      () => {},
     );
     deprecationCollector(
       'Second deprecation',
@@ -116,7 +116,7 @@ module('deprecationCollector', function (hooks) {
         until: 'forever',
         for: 'testing',
       },
-      () => {}
+      () => {},
     );
 
     assert.deepEqual(self.deprecationWorkflow.deprecationLog.messages, {

tests/unit/flush-deprecations-test.js

@@ -43,7 +43,7 @@ setupDeprecationWorkflow({
     { handler: "silence", matchId: "first" },
     { handler: "silence", matchId: "second" }
   ]
-});`
+});`,
     );
   });
 });

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

@@ -28,8 +28,6 @@ module('handleDeprecationWorkflow', function (hooks) {
   });
 
   test('specifying `throwOnUnhandled` as true raises', function (assert) {
-    assert.expect(2);
-
     const config = {
       throwOnUnhandled: true,
       workflow: [{ handler: 'silence', matchMessage: 'Sshhhhh!!' }],
@@ -46,11 +44,11 @@ module('handleDeprecationWorkflow', function (hooks) {
             id: 'foobar',
             for: 'testing',
           },
-          () => {}
+          () => {},
         );
       },
       /Foobarrrzzzz/,
-      'setting raiseOnUnhandled throws for unknown workflows'
+      'setting raiseOnUnhandled throws for unknown workflows',
     );
 
     handleDeprecationWorkflow(
@@ -62,14 +60,12 @@ module('handleDeprecationWorkflow', function (hooks) {
         until: 'forever',
         for: 'testing',
       },
-      () => {}
+      () => {},
     );
     assert.ok(true, 'did not throw when silenced');
   });
 
   test('specifying `throwOnUnhandled` as false does nothing', function (assert) {
-    assert.expect(1);
-
     const config = {
       throwOnUnhandled: false,
     };
@@ -83,7 +79,7 @@ module('handleDeprecationWorkflow', function (hooks) {
         until: 'forever',
         for: 'testing',
       },
-      () => {}
+      () => {},
     );
 
     assert.ok(true, 'does not die when throwOnUnhandled is false');
@@ -103,6 +99,7 @@ module('handleDeprecationWorkflow', function (hooks) {
     assert.ok(true, 'Deprecation did not raise');
   });
 
+  // eslint-disable-next-line qunit/require-expect
   test('deprecation logs with string matcher', function (assert) {
     assert.expect(1);
 
@@ -111,7 +108,7 @@ module('handleDeprecationWorkflow', function (hooks) {
       assert.strictEqual(
         passedMessage.indexOf('DEPRECATION: ' + message),
         0,
-        'deprecation logs'
+        'deprecation logs',
       );
     };
 
@@ -128,7 +125,7 @@ module('handleDeprecationWorkflow', function (hooks) {
         id: 'interesting',
         for: 'testing',
       },
-      () => {}
+      () => {},
     );
   });
 
@@ -147,7 +144,7 @@ module('handleDeprecationWorkflow', function (hooks) {
           until: 'forever',
           for: 'testing',
         },
-        () => {}
+        () => {},
       );
     }, 'deprecation throws');
   });
@@ -166,22 +163,23 @@ module('handleDeprecationWorkflow', function (hooks) {
         until: 'forever',
         for: 'testing',
       },
-      () => {}
+      () => {},
     );
 
     assert.ok(true, 'Deprecation did not raise');
   });
 
+  // eslint-disable-next-line qunit/require-expect
   test('deprecation logs with regex matcher', function (assert) {
     assert.expect(1);
 
     let message = 'Interesting';
 
     console.warn = function (passedMessage) {
-      assert.equal(
+      assert.strictEqual(
         passedMessage,
         'DEPRECATION: ' + message,
-        'deprecation logs'
+        'deprecation logs',
       );
     };
 
@@ -198,7 +196,7 @@ module('handleDeprecationWorkflow', function (hooks) {
         until: 'forever',
         for: 'testing',
       },
-      () => {}
+      () => {},
     );
   });
 
@@ -217,7 +215,7 @@ module('handleDeprecationWorkflow', function (hooks) {
           until: 'forever',
           for: 'testing',
         },
-        () => {}
+        () => {},
       );
     }, 'deprecation throws');
   });
@@ -240,7 +238,7 @@ module('handleDeprecationWorkflow', function (hooks) {
           until: 'forever',
           for: 'testing',
         },
-        () => {}
+        () => {},
       );
     }, 'deprecation throws');
   });
@@ -259,22 +257,23 @@ module('handleDeprecationWorkflow', function (hooks) {
         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 matcher', function (assert) {
     assert.expect(1);
 
     let message = 'Slightly interesting';
 
     console.warn = function (passedMessage) {
-      assert.equal(
+      assert.strictEqual(
         passedMessage,
         'DEPRECATION: ' + message,
-        'deprecation logs'
+        'deprecation logs',
       );
     };
 
@@ -291,7 +290,7 @@ module('handleDeprecationWorkflow', function (hooks) {
         until: '3.0.0',
         for: 'testing',
       },
-      () => {}
+      () => {},
     );
   });
 
@@ -309,7 +308,7 @@ module('handleDeprecationWorkflow', function (hooks) {
           until: '3.0.0',
           for: 'testing',
         },
-        () => {}
+        () => {},
       );
     }, 'deprecation throws');
   });