feat(no-unnecessary-act): add to Angular & Vue configs by default

BREAKING CHANGE: `no-unnecessary-act` is now enabled by default in the Angular & Vue configs

Author: MichaelDeBoey

README.md

@@ -199,7 +199,7 @@ To enable this configuration use the `extends` property in your
 | [`testing-library/no-node-access`](./docs/rules/no-node-access.md)                                   | Disallow direct Node access                                                                  |     | ![angular-badge][] ![react-badge][] ![vue-badge][]                |
 | [`testing-library/no-promise-in-fire-event`](./docs/rules/no-promise-in-fire-event.md)               | Disallow the use of promises passed to a `fireEvent` method                                  |     | ![dom-badge][] ![angular-badge][] ![react-badge][] ![vue-badge][] |
 | [`testing-library/no-render-in-setup`](./docs/rules/no-render-in-setup.md)                           | Disallow the use of `render` in testing frameworks setup functions                           |     |                                                                   |
-| [`testing-library/no-unnecessary-act`](./docs/rules/no-unnecessary-act.md)                           | Disallow wrapping Testing Library utils or empty callbacks in `act`                          |     | ![react-badge][]                                                  |
+| [`testing-library/no-unnecessary-act`](./docs/rules/no-unnecessary-act.md)                           | Disallow wrapping Testing Library utils or empty callbacks in `act`                          |     | ![dom-badge][] ![angular-badge][] ![react-badge][] ![vue-badge][] |
 | [`testing-library/no-wait-for-empty-callback`](./docs/rules/no-wait-for-empty-callback.md)           | Disallow empty callbacks for `waitFor` and `waitForElementToBeRemoved`                       |     | ![dom-badge][] ![angular-badge][] ![react-badge][] ![vue-badge][] |
 | [`testing-library/no-wait-for-multiple-assertions`](./docs/rules/no-wait-for-multiple-assertions.md) | Disallow the use of multiple `expect` calls inside `waitFor`                                 |     |                                                                   |
 | [`testing-library/no-wait-for-side-effects`](./docs/rules/no-wait-for-side-effects.md)               | Disallow the use of side effects in `waitFor`                                                |     |                                                                   |

lib/configs/angular.ts

@@ -13,6 +13,7 @@ export = {
     'testing-library/no-dom-import': ['error', 'angular'],
     'testing-library/no-node-access': 'error',
     'testing-library/no-promise-in-fire-event': 'error',
+    'testing-library/no-unnecessary-act': 'error',
     'testing-library/no-wait-for-empty-callback': 'error',
     'testing-library/prefer-find-by': 'error',
     'testing-library/prefer-screen-queries': 'error',

lib/configs/dom.ts

@@ -9,6 +9,7 @@ export = {
     'testing-library/await-async-utils': 'error',
     'testing-library/no-await-sync-query': 'error',
     'testing-library/no-promise-in-fire-event': 'error',
+    'testing-library/no-unnecessary-act': 'error',
     'testing-library/no-wait-for-empty-callback': 'error',
     'testing-library/prefer-find-by': 'error',
     'testing-library/prefer-screen-queries': 'error',

lib/configs/vue.ts

@@ -14,6 +14,7 @@ export = {
     'testing-library/no-dom-import': ['error', 'vue'],
     'testing-library/no-node-access': 'error',
     'testing-library/no-promise-in-fire-event': 'error',
+    'testing-library/no-unnecessary-act': 'error',
     'testing-library/no-wait-for-empty-callback': 'error',
     'testing-library/prefer-find-by': 'error',
     'testing-library/prefer-screen-queries': 'error',

lib/rules/no-unnecessary-act.ts

@@ -25,10 +25,10 @@ export default createTestingLibraryRule<Options, MessageIds>({
         'Disallow wrapping Testing Library utils or empty callbacks in `act`',
       category: 'Possible Errors',
       recommendedConfig: {
-        dom: false,
-        angular: false,
+        dom: 'error',
+        angular: 'error',
         react: 'error',
-        vue: false,
+        vue: 'error',
       },
     },
     messages: {

tests/__snapshots__/index.test.ts.snap

@@ -18,6 +18,7 @@ Object {
       ],
       "testing-library/no-node-access": "error",
       "testing-library/no-promise-in-fire-event": "error",
+      "testing-library/no-unnecessary-act": "error",
       "testing-library/no-wait-for-empty-callback": "error",
       "testing-library/prefer-find-by": "error",
       "testing-library/prefer-screen-queries": "error",
@@ -33,6 +34,7 @@ Object {
       "testing-library/await-async-utils": "error",
       "testing-library/no-await-sync-query": "error",
       "testing-library/no-promise-in-fire-event": "error",
+      "testing-library/no-unnecessary-act": "error",
       "testing-library/no-wait-for-empty-callback": "error",
       "testing-library/prefer-find-by": "error",
       "testing-library/prefer-screen-queries": "error",
@@ -78,6 +80,7 @@ Object {
       ],
       "testing-library/no-node-access": "error",
       "testing-library/no-promise-in-fire-event": "error",
+      "testing-library/no-unnecessary-act": "error",
       "testing-library/no-wait-for-empty-callback": "error",
       "testing-library/prefer-find-by": "error",
       "testing-library/prefer-screen-queries": "error",