fix: loosen regex in no-commented-tests (fixes #61) (#415)

Author: platinumazure

lib/rules/no-commented-tests.js

@@ -26,7 +26,7 @@ module.exports = {
     create: function (context) {
         const sourceCode = context.getSourceCode(),
             newlineRegExp = /\r\n|\r|\n/g,
-            warningRegExp = /\b(QUnit\.test|QUnit\.asyncTest|QUnit\.skip|test|asyncTest)\s*\(/g;
+            warningRegExp = /\b(QUnit\.test|QUnit\.asyncTest|QUnit\.skip|test|asyncTest)\s*\(\s*["'`]/g;
 
         function getNewlineIndexes(text) {
             const indexes = [];

tests/lib/rules/no-commented-tests.js

@@ -23,7 +23,11 @@ ruleTester.run("no-commented-tests", rule, {
         "QUnit.skip('Name', function () { ok(true); });",
 
         // shebang comments
-        "#!/some-test()"
+        "#!/some-test()",
+
+        // Allow test, parentheses, no quote (https://github.com/platinumazure/eslint-plugin-qunit/issues/61)
+        "// TODO: refactor with a Component test (instead of an Acceptance test)",
+        "// Run actual test (without context)."
     ],
 
     invalid: [