eslint-plugin-jest 23.8.0 → 23.8.1

package/CHANGELOG.md

@@ -1,3 +1,11 @@
+## [23.8.1](https://github.com/jest-community/eslint-plugin-jest/compare/v23.8.0...v23.8.1) (2020-02-29)
+
+### Bug Fixes
+
+- remove tests from published package
+  ([#541](https://github.com/jest-community/eslint-plugin-jest/issues/541))
+  ([099a150](https://github.com/jest-community/eslint-plugin-jest/commit/099a150b87fa693ccf1c512ee501aed1457ba656))
+
 # [23.8.0](https://github.com/jest-community/eslint-plugin-jest/compare/v23.7.0...v23.8.0) (2020-02-23)
 
 ### Bug Fixes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-jest",
-  "version": "23.8.0",
+  "version": "23.8.1",
   "description": "Eslint rules for Jest",
   "repository": "jest-community/eslint-plugin-jest",
   "license": "MIT",
@@ -33,7 +33,7 @@
     "pretest": "yarn build",
     "test": "jest",
     "build": "babel --extensions .js,.ts src --out-dir lib --copy-files",
-    "postbuild": "rimraf lib/**/__tests__/**",
+    "postbuild": "rimraf lib/__tests__ lib/**/__tests__",
     "typecheck": "tsc -p ."
   },
   "dependencies": {

package/lib/__tests__/__snapshots__/rules.test.ts.snap

@@ -1,94 +0,0 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
-
-exports[`rules should export configs that refer to actual rules 1`] = `
-Object {
-  "all": Object {
-    "env": Object {
-      "jest/globals": true,
-    },
-    "plugins": Array [
-      "jest",
-    ],
-    "rules": Object {
-      "jest/consistent-test-it": "error",
-      "jest/expect-expect": "error",
-      "jest/lowercase-name": "error",
-      "jest/no-alias-methods": "error",
-      "jest/no-commented-out-tests": "error",
-      "jest/no-disabled-tests": "error",
-      "jest/no-duplicate-hooks": "error",
-      "jest/no-expect-resolves": "error",
-      "jest/no-export": "error",
-      "jest/no-focused-tests": "error",
-      "jest/no-hooks": "error",
-      "jest/no-identical-title": "error",
-      "jest/no-if": "error",
-      "jest/no-jasmine-globals": "error",
-      "jest/no-jest-import": "error",
-      "jest/no-large-snapshots": "error",
-      "jest/no-mocks-import": "error",
-      "jest/no-standalone-expect": "error",
-      "jest/no-test-callback": "error",
-      "jest/no-test-prefixes": "error",
-      "jest/no-test-return-statement": "error",
-      "jest/no-truthy-falsy": "error",
-      "jest/no-try-expect": "error",
-      "jest/prefer-called-with": "error",
-      "jest/prefer-expect-assertions": "error",
-      "jest/prefer-hooks-on-top": "error",
-      "jest/prefer-inline-snapshots": "error",
-      "jest/prefer-spy-on": "error",
-      "jest/prefer-strict-equal": "error",
-      "jest/prefer-to-be-null": "error",
-      "jest/prefer-to-be-undefined": "error",
-      "jest/prefer-to-contain": "error",
-      "jest/prefer-to-have-length": "error",
-      "jest/prefer-todo": "error",
-      "jest/require-to-throw-message": "error",
-      "jest/require-top-level-describe": "error",
-      "jest/valid-describe": "error",
-      "jest/valid-expect": "error",
-      "jest/valid-expect-in-promise": "error",
-      "jest/valid-title": "error",
-    },
-  },
-  "recommended": Object {
-    "env": Object {
-      "jest/globals": true,
-    },
-    "plugins": Array [
-      "jest",
-    ],
-    "rules": Object {
-      "jest/expect-expect": "warn",
-      "jest/no-commented-out-tests": "warn",
-      "jest/no-disabled-tests": "warn",
-      "jest/no-export": "error",
-      "jest/no-focused-tests": "error",
-      "jest/no-identical-title": "error",
-      "jest/no-jasmine-globals": "warn",
-      "jest/no-jest-import": "error",
-      "jest/no-mocks-import": "error",
-      "jest/no-standalone-expect": "error",
-      "jest/no-test-callback": "error",
-      "jest/no-test-prefixes": "error",
-      "jest/no-try-expect": "error",
-      "jest/valid-describe": "error",
-      "jest/valid-expect": "error",
-      "jest/valid-expect-in-promise": "error",
-    },
-  },
-  "style": Object {
-    "plugins": Array [
-      "jest",
-    ],
-    "rules": Object {
-      "jest/no-alias-methods": "warn",
-      "jest/prefer-to-be-null": "error",
-      "jest/prefer-to-be-undefined": "error",
-      "jest/prefer-to-contain": "error",
-      "jest/prefer-to-have-length": "error",
-    },
-  },
-}
-`;

package/lib/__tests__/rules.test.js

@@ -1,47 +0,0 @@
-"use strict";
-
-var _fs = require("fs");
-
-var _path = require("path");
-
-var _ = _interopRequireDefault(require("../"));
-
-function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
-
-const ruleNames = Object.keys(_.default.rules);
-const numberOfRules = 40;
-describe('rules', () => {
-  it('should have a corresponding doc for each rule', () => {
-    ruleNames.forEach(rule => {
-      const docPath = (0, _path.resolve)(__dirname, '../../docs/rules', `${rule}.md`);
-
-      if (!(0, _fs.existsSync)(docPath)) {
-        throw new Error(`Could not find documentation file for rule "${rule}" in path "${docPath}"`);
-      }
-    });
-  });
-  it('should have the correct amount of rules', () => {
-    const {
-      length
-    } = ruleNames;
-
-    if (length !== numberOfRules) {
-      throw new Error(`There should be exactly ${numberOfRules} rules, but there are ${length}. If you've added a new rule, please update this number.`);
-    }
-  });
-  it('should export configs that refer to actual rules', () => {
-    const recommendedConfigs = _.default.configs;
-    expect(recommendedConfigs).toMatchSnapshot();
-    expect(Object.keys(recommendedConfigs)).toEqual(['all', 'recommended', 'style']);
-    expect(Object.keys(recommendedConfigs.all.rules)).toHaveLength(ruleNames.length);
-    const allConfigRules = Object.values(recommendedConfigs).map(config => Object.keys(config.rules)).reduce((previousValue, currentValue) => [...previousValue, ...currentValue]);
-    allConfigRules.forEach(rule => {
-      const ruleNamePrefix = 'jest/';
-      const ruleName = rule.slice(ruleNamePrefix.length);
-      expect(rule.startsWith(ruleNamePrefix)).toBe(true);
-      expect(ruleNames).toContain(ruleName); // eslint-disable-next-line @typescript-eslint/no-require-imports
-
-      expect(() => require(`../rules/${ruleName}`)).not.toThrow();
-    });
-  });
-});