feat(no-unnecessary-act): make isStrict option true by default

BREAKING CHANGE: `isStrict` option is now `true` by default
testing-library · Oct 13, 2021 · fe6f6a5 · fe6f6a5
1 parent db42e69
commit fe6f6a5
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 20 deletions.
diff --git a/lib/configs/react.ts b/lib/configs/react.ts
@@ -14,7 +14,7 @@ export = {
     'testing-library/no-node-access': 'error',
     'testing-library/no-promise-in-fire-event': 'error',
     'testing-library/no-render-in-setup': 'error',
-    'testing-library/no-unnecessary-act': ['error', { isStrict: true }],
+    'testing-library/no-unnecessary-act': 'error',
     'testing-library/no-wait-for-empty-callback': 'error',
     'testing-library/no-wait-for-multiple-assertions': 'error',
     'testing-library/prefer-find-by': 'error',

diff --git a/lib/rules/no-unnecessary-act.ts b/lib/rules/no-unnecessary-act.ts
@@ -27,7 +27,7 @@ export default createTestingLibraryRule<Options, MessageIds>({
       recommendedConfig: {
         dom: false,
         angular: false,
-        react: ['error', { isStrict: true }],
+        react: 'error',
         vue: false,
       },
     },
@@ -40,18 +40,20 @@ export default createTestingLibraryRule<Options, MessageIds>({
       {
         type: 'object',
         properties: {
-          isStrict: { type: 'boolean' },
+          isStrict: {
+            type: 'boolean',
+          },
         },
       },
     ],
   },
   defaultOptions: [
     {
-      isStrict: false,
+      isStrict: true,
     },
   ],
 
-  create(context, [options], helpers) {
+  create(context, [{ isStrict = true }], helpers) {
     function getStatementIdentifier(statement: TSESTree.Statement) {
       const callExpression = getStatementCallExpression(statement);
 
@@ -113,7 +115,6 @@ export default createTestingLibraryRule<Options, MessageIds>({
     function checkNoUnnecessaryActFromBlockStatement(
       blockStatementNode: TSESTree.BlockStatement
     ) {
-      const { isStrict } = options;
       const functionNode = blockStatementNode.parent as
         | TSESTree.ArrowFunctionExpression
         | TSESTree.FunctionExpression

diff --git a/tests/__snapshots__/index.test.ts.snap b/tests/__snapshots__/index.test.ts.snap
@@ -58,12 +58,7 @@ Object {
       "testing-library/no-node-access": "error",
       "testing-library/no-promise-in-fire-event": "error",
       "testing-library/no-render-in-setup": "error",
-      "testing-library/no-unnecessary-act": Array [
-        "error",
-        Object {
-          "isStrict": true,
-        },
-      ],
+      "testing-library/no-unnecessary-act": "error",
       "testing-library/no-wait-for-empty-callback": "error",
       "testing-library/no-wait-for-multiple-assertions": "error",
       "testing-library/prefer-find-by": "error",

diff --git a/tests/lib/rules/no-unnecessary-act.test.ts b/tests/lib/rules/no-unnecessary-act.test.ts
@@ -13,15 +13,18 @@ type ValidTestCase = TSESLint.ValidTestCase<Options>;
 type InvalidTestCase = TSESLint.InvalidTestCase<MessageIds, Options>;
 type TestCase = InvalidTestCase | ValidTestCase;
 
-const enableStrict = <T extends TestCase>(array: T[]): T[] =>
+const addOptions = <T extends TestCase>(
+  array: T[],
+  options?: Options[number]
+): T[] =>
   array.map((testCase) => ({
     ...testCase,
-    options: [
-      {
-        isStrict: true,
-      },
-    ],
+    options: [options],
   }));
+const disableStrict = <T extends TestCase>(array: T[]): T[] =>
+  addOptions(array, { isStrict: false });
+const enableStrict = <T extends TestCase>(array: T[]): T[] =>
+  addOptions(array, { isStrict: true });
 
 /**
  * - AGR stands for Aggressive Reporting
@@ -886,12 +889,12 @@ const invalidTestCases: InvalidTestCase[] = [
 ruleTester.run(RULE_NAME, rule, {
   valid: [
     ...validTestCases,
-    ...validNonStrictTestCases,
+    ...disableStrict(validNonStrictTestCases),
     ...enableStrict(validTestCases),
   ],
   invalid: [
     ...invalidTestCases,
-    ...invalidStrictTestCases,
+    ...disableStrict(invalidStrictTestCases),
     ...enableStrict(invalidTestCases),
   ],
 });