Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove unnecessary test commands from packages. They don’t work anymore. #2606

Merged
merged 1 commit into from
Jan 15, 2017
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -43,9 +43,10 @@
"lint": "eslint . --cache",
"postinstall": "node ./scripts/postinstall.js && node ./scripts/build.js && (cd packages/eslint-plugin-jest && yarn link) && yarn link eslint-plugin-jest",
"publish": "yarn run build-clean && yarn run build && lerna publish",
"test": "yarn run typecheck && yarn run lint && yarn run build && yarn run jest && yarn run test-examples",
"test-ci": "yarn run typecheck && yarn run lint && yarn run build && yarn run jest-coverage -- -i && yarn run test-examples && node scripts/mapCoverage.js && codecov",
"test-examples": "node scripts/test_examples.js",
"test-pretty-format-perf": "node packages/pretty-format/perf/test.js",
"test": "yarn run typecheck && yarn run lint && yarn run build && yarn run jest && yarn run test-examples",
"typecheck": "flow check",
"watch": "yarn run build; node ./scripts/watch.js"
},
Expand Down
5 changes: 1 addition & 4 deletions packages/babel-plugin-jest-hoist/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,5 @@
"url": "https://github.com/facebook/jest.git"
},
"license": "BSD-3-Clause",
"main": "build/index.js",
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
}
"main": "build/index.js"
}
5 changes: 1 addition & 4 deletions packages/jest-changed-files/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,5 @@
"url": "https://github.com/facebook/jest.git"
},
"license": "BSD-3-Clause",
"main": "build/index.js",
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
}
"main": "build/index.js"
}
3 changes: 0 additions & 3 deletions packages/jest-cli/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -44,9 +44,6 @@
"type": "git",
"url": "https://github.com/facebook/jest"
},
"scripts": {
"test": "./bin/jest.js"
},
"bugs": {
"url": "https://github.com/facebook/jest/issues"
},
Expand Down
3 changes: 0 additions & 3 deletions packages/jest-config/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,5 @@
"jest-resolve": "^18.1.0",
"jest-util": "^18.1.0",
"json-stable-stringify": "^1.0.0"
},
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
}
}
3 changes: 0 additions & 3 deletions packages/jest-diff/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,5 @@
"diff": "^3.0.0",
"jest-matcher-utils": "^18.1.0",
"pretty-format": "^18.1.0"
},
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
}
}
3 changes: 0 additions & 3 deletions packages/jest-editor-support/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,6 @@
},
"license": "BSD-3-Clause",
"main": "build/index.js",
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
},
"optionalDependencies": {
"typescript": "^2.0.10"
},
Expand Down
3 changes: 0 additions & 3 deletions packages/jest-environment-node/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,5 @@
"dependencies": {
"jest-mock": "^18.0.0",
"jest-util": "^18.1.0"
},
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
}
}
5 changes: 1 addition & 4 deletions packages/jest-file-exists/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,5 @@
"url": "https://github.com/facebook/jest.git"
},
"license": "BSD-3-Clause",
"main": "build/index.js",
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
}
"main": "build/index.js"
}
3 changes: 0 additions & 3 deletions packages/jest-haste-map/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,5 @@
"micromatch": "^2.3.11",
"sane": "~1.4.1",
"worker-farm": "^1.3.1"
},
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
}
}
3 changes: 0 additions & 3 deletions packages/jest-jasmine2/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,5 @@
"jest-matchers": "^18.1.0",
"jest-snapshot": "^18.1.0",
"jest-util": "^18.1.0"
},
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
}
}
3 changes: 0 additions & 3 deletions packages/jest-matcher-utils/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,6 @@
},
"license": "BSD-3-Clause",
"main": "build/index.js",
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
},
"dependencies": {
"chalk": "^1.1.3",
"pretty-format": "^18.1.0"
Expand Down
3 changes: 0 additions & 3 deletions packages/jest-matchers/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,5 @@
"jest-matcher-utils": "^18.1.0",
"jest-util": "^18.1.0",
"pretty-format": "^18.1.0"
},
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
}
}
5 changes: 1 addition & 4 deletions packages/jest-mock/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,5 @@
"url": "https://github.com/facebook/jest.git"
},
"license": "BSD-3-Clause",
"main": "build/index.js",
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
}
"main": "build/index.js"
}
3 changes: 0 additions & 3 deletions packages/jest-repl/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,5 @@
},
"bin": {
"jest-repl": "./bin/jest-repl.js"
},
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
}
}
3 changes: 0 additions & 3 deletions packages/jest-resolve-dependencies/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,5 @@
"dependencies": {
"jest-file-exists": "^17.0.0",
"jest-resolve": "^18.1.0"
},
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
}
}
3 changes: 0 additions & 3 deletions packages/jest-resolve/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,5 @@
"jest-file-exists": "^17.0.0",
"jest-haste-map": "^18.1.0",
"resolve": "^1.2.0"
},
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
}
}
7 changes: 2 additions & 5 deletions packages/jest-runtime/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,15 +25,12 @@
"strip-bom": "3.0.0",
"yargs": "^6.3.0"
},
"bin": {
"jest-runtime": "./bin/jest-runtime.js"
},
"devDependencies": {
"jest-config": "^18.1.0",
"jest-environment-node": "^18.1.0",
"jest-environment-jsdom": "^18.1.0"
},
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
"bin": {
"jest-runtime": "./bin/jest-runtime.js"
}
}
3 changes: 0 additions & 3 deletions packages/jest-snapshot/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,5 @@
"jest-util": "^18.1.0",
"natural-compare": "^1.4.0",
"pretty-format": "^18.1.0"
},
"scripts": {
"test": "../jest-cli/bin/jest.js"
}
}
3 changes: 0 additions & 3 deletions packages/jest-util/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,5 @@
"jest-file-exists": "^17.0.0",
"jest-mock": "^18.0.0",
"mkdirp": "^0.5.1"
},
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js"
}
}
4 changes: 0 additions & 4 deletions packages/pretty-format/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,6 @@
"description": "Stringify any JavaScript value.",
"main": "build/index.js",
"author": "James Kyle <me@thejameskyle.com>",
"scripts": {
"test": "../../packages/jest-cli/bin/jest.js",
"perf": "node perf/test.js"
},
"dependencies": {
"ansi-styles": "^2.2.1"
}
Expand Down
12 changes: 6 additions & 6 deletions packages/pretty-format/perf/test.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ const chalk = require('chalk');
const leftPad = require('left-pad');
const worldGeoJson = require('./world.geo.json');
const React = require('react');
const ReactTestRenderer = require('react/lib/ReactTestRenderer');
const ReactTestRenderer = require('react-test-renderer');
const ReactTestComponent = require('../build/plugins/ReactTestComponent');

const NANOSECONDS = 1000000000;
Expand Down Expand Up @@ -54,7 +54,7 @@ function testCase(name, fn) {
function test(name, value, ignoreResult, prettyFormatOpts) {
const formatted = testCase(
'prettyFormat() ',
() => prettyFormat(value, prettyFormatOpts),
() => prettyFormat(value, prettyFormatOpts)
);

const inspected = testCase('util.inspect() ', () => {
Expand All @@ -73,7 +73,7 @@ function test(name, value, ignoreResult, prettyFormatOpts) {
});

const winner = results[0];

results.forEach((item, index) => {
item.isWinner = index === 0;
item.isLoser = index === results.length - 1;
Expand Down Expand Up @@ -202,9 +202,9 @@ const element = React.createElement(
React.createElement('div'),
React.createElement('div', {prop: {a: 1, b: 2}},
React.createElement('div', null,
React.createElement('div'),
),
),
React.createElement('div')
)
)
);

test('react', ReactTestRenderer.create(element).toJSON(), false, {
Expand Down