Merge branch 'feature/in-document-fixes' of github.com:testing-library/eslint-plugin-jest-dom into feature/in-document-fixes

benmonro · benmonro · commit 668393510498 · 2020-11-29T10:46:03.000-08:00
diff --git a/.all-contributorsrc b/.all-contributorsrc
@@ -118,7 +118,9 @@
       "avatar_url": "https://avatars2.githubusercontent.com/u/14806298?v=4",
       "profile": "https://codepen.io/ariperkkio/",
       "contributions": [
-        "bug"
+        "bug",
+        "code",
+        "test"
       ]
     },
     {
diff --git a/.github/workflows/smoke-test.yml b/.github/workflows/smoke-test.yml
@@ -0,0 +1,22 @@
+name: Smoke test
+
+on:
+  schedule:
+    - cron: '0 0 * * SUN'
+  workflow_dispatch:
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+    - uses: actions/setup-node@v1
+      with:
+        node-version: 12.11
+    - run: yarn install
+    - run: yarn build
+    - run: yarn link
+    - run: yarn link eslint-plugin-jest-dom
+    - run: yarn test:smoke
+      env:
+        CI: true
diff --git a/.gitignore b/.gitignore
@@ -7,3 +7,7 @@ dist
 # when working with contributors
 package-lock.json
 yarn.lock
+
+# Smoke test
+.cache-eslint-remote-tester
+eslint-remote-tester-results
diff --git a/.prettierignore b/.prettierignore
@@ -1,3 +1,5 @@
 node_modules
 coverage
 dist
+.cache-eslint-remote-tester
+eslint-remote-tester-results
diff --git a/README.md b/README.md
@@ -154,14 +154,13 @@ Thanks goes to these people ([emoji key][emojis]):
     <td align="center"><a href="https://skovy.dev"><img src="https://avatars1.githubusercontent.com/u/5247455?v=4" width="100px;" alt=""/><br /><sub><b>Spencer Miskoviak</b></sub></a><br /><a href="https://github.com/testing-library/eslint-plugin-jest-dom/commits?author=skovy" title="Code">💻</a> <a href="https://github.com/testing-library/eslint-plugin-jest-dom/commits?author=skovy" title="Tests">⚠️</a></td>
     <td align="center"><a href="https://github.com/atsikov"><img src="https://avatars3.githubusercontent.com/u/1422928?v=4" width="100px;" alt=""/><br /><sub><b>Aleksei Tsikov</b></sub></a><br /><a href="https://github.com/testing-library/eslint-plugin-jest-dom/issues?q=author%3Aatsikov" title="Bug reports">🐛</a></td>
     <td align="center"><a href="https://mario.dev"><img src="https://avatars1.githubusercontent.com/u/2677072?v=4" width="100px;" alt=""/><br /><sub><b>Mario Beltrán Alarcón</b></sub></a><br /><a href="https://github.com/testing-library/eslint-plugin-jest-dom/commits?author=Belco90" title="Documentation">📖</a></td>
-    <td align="center"><a href="https://codepen.io/ariperkkio/"><img src="https://avatars2.githubusercontent.com/u/14806298?v=4" width="100px;" alt=""/><br /><sub><b>Ari Perkkiö</b></sub></a><br /><a href="https://github.com/testing-library/eslint-plugin-jest-dom/issues?q=author%3AAriPerkkio" title="Bug reports">🐛</a></td>
+    <td align="center"><a href="https://codepen.io/ariperkkio/"><img src="https://avatars2.githubusercontent.com/u/14806298?v=4" width="100px;" alt=""/><br /><sub><b>Ari Perkkiö</b></sub></a><br /><a href="https://github.com/testing-library/eslint-plugin-jest-dom/issues?q=author%3AAriPerkkio" title="Bug reports">🐛</a> <a href="https://github.com/testing-library/eslint-plugin-jest-dom/commits?author=AriPerkkio" title="Code">💻</a> <a href="https://github.com/testing-library/eslint-plugin-jest-dom/commits?author=AriPerkkio" title="Tests">⚠️</a></td>
     <td align="center"><a href="http://www.antn.se"><img src="https://avatars0.githubusercontent.com/u/785676?v=4" width="100px;" alt=""/><br /><sub><b>Anton Niklasson</b></sub></a><br /><a href="https://github.com/testing-library/eslint-plugin-jest-dom/commits?author=AntonNiklasson" title="Code">💻</a> <a href="https://github.com/testing-library/eslint-plugin-jest-dom/commits?author=AntonNiklasson" title="Tests">⚠️</a> <a href="https://github.com/testing-library/eslint-plugin-jest-dom/commits?author=AntonNiklasson" title="Documentation">📖</a></td>
   </tr>
 </table>
 
 <!-- markdownlint-enable -->
 <!-- prettier-ignore-end -->
-
 <!-- ALL-CONTRIBUTORS-LIST:END -->
 
 This project follows the [all-contributors][all-contributors] specification.
diff --git a/package.json b/package.json
@@ -34,6 +34,7 @@
     "lint": "kcd-scripts lint",
     "setup": "npm install && npm run validate -s",
     "test": "kcd-scripts test",
+    "test:smoke": "eslint-remote-tester --config ./smoke-test/eslint-remote-tester.config.js",
     "test:update": "npm test -- --updateSnapshot --coverage",
     "validate": "kcd-scripts validate"
   },
@@ -43,9 +44,12 @@
     "requireindex": "^1.2.0"
   },
   "devDependencies": {
+    "@typescript-eslint/parser": "^4.8.2",
     "eslint": "7.14",
+    "eslint-remote-tester": "^0.3.3",
     "jest-extended": "^0.11.5",
-    "kcd-scripts": "6.5.1"
+    "kcd-scripts": "6.5.1",
+    "typescript": "^4.1.2"
   },
   "peerDependencies": {
     "eslint": ">=6.8"
@@ -62,7 +66,9 @@
   "eslintIgnore": [
     "node_modules",
     "coverage",
-    "dist"
+    "dist",
+    ".cache-eslint-remote-tester",
+    "eslint-remote-tester-results"
   ],
   "engines": {
     "node": "^10.12.0 || >=12.0.0",
diff --git a/smoke-test/eslint-remote-tester.config.js b/smoke-test/eslint-remote-tester.config.js
@@ -0,0 +1,73 @@
+module.exports = {
+    /** Repositories to scan */
+    repositories: require('./repositories.json'),
+
+    /** Extensions of files under scanning */
+    extensions: ['js', 'jsx', 'ts', 'tsx'],
+
+    /** Optional pattern used to exclude paths */
+    pathIgnorePattern: `(${[
+        'node_modules',
+        '\\/\\.', // Any file or directory starting with dot, e.g. ".git"
+        '/dist/',
+        '/build/',
+
+        // Common patterns for minified JS
+        'babel\\.js',
+        'vendor\\.js',
+        'vendors\\.js',
+        'chunk\\.js',
+        'bundle\\.js',
+        'react-dom\\.development\\.js',
+        '\\.min\\.js', // Any *.min.js
+
+        // Project specific ignores
+        'codesandbox-client/packages/app/static/js',
+        'codesandbox-client/standalone-packages',
+        'dockunit/platform/assets',
+        'hyper/bin',
+        'react-solitaire/lib/index\\.js',
+        'Khan/perseus/lib',
+        'glortho/react-keydown/example/public',
+        'reach/reach-ui/packages/combobox/examples/cities\\.ts',
+        'reach/reach-ui/website/src/components/cities\\.js',
+        'reach/reach-ui/website/static/router/static',
+        'Automattic/wp-calypso/client/components/phone-input/data\\.js',
+        'test262-main\\.ts',
+        'sample_vis\\.test\\.mocks\\.ts',
+    ].join('|')})`,
+
+    /** Empty array since we are only interested in linter crashes */
+    rulesUnderTesting: [],
+
+    /** Maximum amount of tasks ran concurrently */
+    concurrentTasks: 2,
+
+    /** Optional boolean flag used to enable caching of cloned repositories. For CIs it's ideal to disable caching. Defauls to true. */
+    cache: false,
+
+    /** ESLint configuration */
+    eslintrc: {
+        root: true,
+        env: {
+            es6: true,
+        },
+        parser: '@typescript-eslint/parser',
+        parserOptions: {
+            ecmaVersion: 2020,
+            sourceType: 'module',
+            ecmaFeatures: {
+                jsx: true,
+            },
+        },
+        plugins: [
+            'jest-dom',
+        ],
+        extends: [
+            'plugin:jest-dom/recommended',
+        ],
+        rules: {
+            'prefer-in-document': 'error'
+        },
+    },
+};
diff --git a/smoke-test/repositories.json b/smoke-test/repositories.json

Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,9 @@`
`118`	`118`	`"avatar_url": "https://avatars2.githubusercontent.com/u/14806298?v=4",`
`119`	`119`	`"profile": "https://codepen.io/ariperkkio/",`
`120`	`120`	`"contributions": [`
`121`		`- "bug"`
	`121`	`+ "bug",`
	`122`	`+ "code",`
	`123`	`+ "test"`
`122`	`124`	`]`
`123`	`125`	`},`
`124`	`126`	`{`