docs(no-container): update

| add description about testing library frameworks
| add link to container docs
| remove recommended badge

Author: Mateus Felix

README.md

@@ -140,7 +140,7 @@ To enable this configuration use the `extends` property in your
 | [await-fire-event](docs/rules/await-fire-event.md)                     | Enforce async fire event methods to be awaited                             | ![vue-badge][]                                                            |                    |
 | [consistent-data-testid](docs/rules/consistent-data-testid.md)         | Ensure `data-testid` values match a provided regex.                        |                                                                           |                    |
 | [no-await-sync-query](docs/rules/no-await-sync-query.md)               | Disallow unnecessary `await` for sync queries                              | ![recommended-badge][] ![angular-badge][] ![react-badge][] ![vue-badge][] |                    |
-| [no-container](docs/rules/no-container.md)                             | Disallow the use of `container` methods                                    | ![recommended-badge][] ![angular-badge][] ![react-badge][] ![vue-badge][] |                    |
+| [no-container](docs/rules/no-container.md)                             | Disallow the use of `container` methods                                    | ![angular-badge][] ![react-badge][] ![vue-badge][]                        |                    |
 | [no-debug](docs/rules/no-debug.md)                                     | Disallow the use of `debug`                                                | ![angular-badge][] ![react-badge][] ![vue-badge][]                        |                    |
 | [no-dom-import](docs/rules/no-dom-import.md)                           | Disallow importing from DOM Testing Library                                | ![angular-badge][] ![react-badge][] ![vue-badge][]                        | ![fixable-badge][] |
 | [no-manual-cleanup](docs/rules/no-manual-cleanup.md)                   | Disallow the use of `cleanup`                                              |                                                                           |                    |

docs/rules/no-container.md

@@ -2,6 +2,8 @@
 
 By using `container` methods like `.querySelector` you may lose a lot of the confidence that the user can really interact with your UI. Also, the test becomes harder to read, and it will break more frequently.
 
+This applies to Testing Library frameworks built on top of **DOM Testing Library**
+
 ## Rule Details
 
 This rule aims to disallow the use of `container` methods in your tests.
@@ -33,4 +35,5 @@ If you use [custom render functions](https://testing-library.com/docs/example-re
 
 ## Further Reading
 
+- [about the `container` element](https://testing-library.com/docs/react-testing-library/api#container-1)
 - [querying with `screen`](https://testing-library.com/docs/dom-testing-library/api-queries#screen)

lib/index.ts

@@ -36,7 +36,6 @@ const recommendedRules = {
   'testing-library/await-async-query': 'error',
   'testing-library/await-async-utils': 'error',
   'testing-library/no-await-sync-query': 'error',
-  'testing-library/no-container': 'error',
   'testing-library/no-wait-for-empty-callback': 'error',
   'testing-library/prefer-find-by': 'error',
   'testing-library/prefer-screen-queries': 'error',
@@ -53,6 +52,7 @@ export = {
       plugins: ['testing-library'],
       rules: {
         ...recommendedRules,
+        'testing-library/no-container': 'error',
         'testing-library/no-debug': 'warn',
         'testing-library/no-dom-import': ['error', 'angular'],
       },
@@ -61,6 +61,7 @@ export = {
       plugins: ['testing-library'],
       rules: {
         ...recommendedRules,
+        'testing-library/no-container': 'error',
         'testing-library/no-debug': 'warn',
         'testing-library/no-dom-import': ['error', 'react'],
       },
@@ -70,6 +71,7 @@ export = {
       rules: {
         ...recommendedRules,
         'testing-library/await-fire-event': 'error',
+        'testing-library/no-container': 'error',
         'testing-library/no-debug': 'warn',
         'testing-library/no-dom-import': ['error', 'vue'],
       },

tests/__snapshots__/index.test.ts.snap

@@ -53,7 +53,6 @@ Object {
     "testing-library/await-async-query": "error",
     "testing-library/await-async-utils": "error",
     "testing-library/no-await-sync-query": "error",
-    "testing-library/no-container": "error",
     "testing-library/no-wait-for-empty-callback": "error",
     "testing-library/prefer-find-by": "error",
     "testing-library/prefer-screen-queries": "error",