Skip to content

Commit 57da558

Browse files
GGonryuncaptain-yossarian
GGonryun
authored and
captain-yossarian
committed
standardize filenames in e2e (jestjs#7471)
* standardizing file names to fit facebook standards * updating filenames to better fit facebook standards * standardizing jest-runner filenames to fit facebook standards * standardizing file names in /e2e/before-all-filtered * resolving naming discrepencies inside of files * standardizing filenames in e2e/before-each-queue and corresponding tests * standardizing several tests in ./e2e/__test__/ * standardizing filenames for babel-plugin-jest-hoist and related test files * standardizing file-names for folders 'browser-support' and 'clear-cache' and their corresponding test files * resolving issue with karma referencing non-standard browserTest * resolving issue with karma referencing non-standard browserTest filename * standardizing file names to fit facebook standards * updating filenames to better fit facebook standards * Add issue template labels (jestjs#7470) * Add issue template labels * Lint * standardizing jest-runner filenames to fit facebook standards * allow `bail` setting to control when to bail out of a failing test run (jestjs#7335) * standardizing file names in /e2e/before-all-filtered * resolving naming discrepencies inside of files * standardizing filenames in e2e/before-each-queue and corresponding tests * standardizing several tests in ./e2e/__test__/ * standardizing filenames for babel-plugin-jest-hoist and related test files * standardizing file-names for folders 'browser-support' and 'clear-cache' and their corresponding test files * resolving issue with karma referencing non-standard browserTest * resolving issue with karma referencing non-standard browserTest filename * standardizing file names in e2e/__test__ * standardizing file names in compare-dom-nodes * standardizing file names for coverage-report and coverage-remap * updating case sensitive filenames * renaming files to fit facebook standards * adding case sensitive file changes * resolving dependency issues after renaming case sensitive files. * standardizing file names to fit facebook standards * adjusting file names and snapshots to fit facebook standards * adjusting file names to fit facebook standards * resolves bug with nodeAssertionError not being found by test suite * changing references to match file name * changing file name to check test-jest-circus * readding a previously deleted snapshot * Update e2e/bad-source-map/__tests__/badSourceMap.js comment to reflect new file name Co-Authored-By: GGonryun <[email protected]> * improving file naming conventions letters g & j * changing filename to fit facebook standards * renaming filenames to fit facebook standards for 'L' * renaming filenames to fit facebook standards for 'M' * renaming filenames to fit facebook standards for 'N' * resolving leftover merge conflicts * reverting error in naming by changing filename back to uppercase * renaming filenames 'O' to fit facebook standards. * renaming filenames 'P' to fit facebook standards. * renaming filenames 'R' to fit facebook standards. * renaming filenames 'S' to fit facebook standards. * changing regex to match temporary folder name * renaming filenames 'T' to fit facebook standards. * renaming filenames 'U' to fit facebook standards. * renaming filenames 'V' to fit facebook standards. * renaming filenames 'W' to fit facebook standards. * updating snapshots to reflect standardized file names * Rename e2e/test-name-pattern-skipped/package.json to e2e/test-name-pattern-temp/package.json * Rename e2e/test-name-pattern/package.json to e2e/test-name-pattern-skipped/package.json * Rename e2e/test-name-pattern-skipped/package.json to e2e/test-name-pattern.package.json * Rename e2e/test-name-pattern-temp/package.json to e2e/test-name-pattern/package.json * Rename e2e/test-name-pattern.package.json to e2e/test-name-pattern-skipped/package.json * removing left over files and adjusting file names previously missed * deleting more leftover files
1 parent 0bca646 commit 57da558

File tree

307 files changed

+401
-400
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

307 files changed

+401
-400
lines changed

‎CHANGELOG.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,7 @@
147147
- `[jest-haste-map]` Refactor `dependencyExtractor` and tests ([#7385](https://github.com/facebook/jest/pull/7385))
148148
- `[docs]` Clearify conditional setting of `NODE_ENV` ([#7369](https://github.com/facebook/jest/pull/7369))
149149
- `[docs]` Clarify conditional setting of `NODE_ENV` ([#7369](https://github.com/facebook/jest/pull/7369))
150-
- `[*]` Standardize file names ([#7316](https://github.com/facebook/jest/pull/7316), [#7266](https://github.com/facebook/jest/pull/7266), [#7238](https://github.com/facebook/jest/pull/7238), [#7314](https://github.com/facebook/jest/pull/7314), [#7467](https://github.com/facebook/jest/pull/7467), [#7464](https://github.com/facebook/jest/pull/7464))
150+
- `[*]` Standardize file names ([#7316](https://github.com/facebook/jest/pull/7316), [#7266](https://github.com/facebook/jest/pull/7266), [#7238](https://github.com/facebook/jest/pull/7238), [#7314](https://github.com/facebook/jest/pull/7314), [#7467](https://github.com/facebook/jest/pull/7467), [#7464](https://github.com/facebook/jest/pull/7464)), [#7471](https://github.com/facebook/jest/pull/7471))
151151
- `[docs]` Add `testPathIgnorePatterns` in CLI documentation ([#7440](https://github.com/facebook/jest/pull/7440))
152152
- `[docs]` Removed misleading text about `describe()` grouping together tests into a test suite ([#7434](https://github.com/facebook/jest/pull/7434))
153153

‎e2e/__tests__/__snapshots__/before-all-filtered.js.snap

-19
This file was deleted.

‎e2e/__tests__/__snapshots__/before-each-queue.js.snap

-19
This file was deleted.

‎e2e/__tests__/__snapshots__/beforeAllFiltered.js.snap

+19
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
// Jest Snapshot v1, https://goo.gl/fbAQLP
2+
3+
exports[`Correct BeforeAll run ensures the BeforeAll of ignored suite is not run 1`] = `
4+
" console.log __tests__/beforeAllFiltered.test.js:5
5+
beforeAll 1
6+
7+
console.log __tests__/beforeAllFiltered.test.js:8
8+
beforeEach 1
9+
10+
console.log __tests__/beforeAllFiltered.test.js:17
11+
It Foo
12+
13+
console.log __tests__/beforeAllFiltered.test.js:11
14+
afterEach 1
15+
16+
console.log __tests__/beforeAllFiltered.test.js:14
17+
afterAll 1
18+
"
19+
`;

‎e2e/__tests__/__snapshots__/beforeEachQueue.js.snap

+19
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,19 @@
1+
// Jest Snapshot v1, https://goo.gl/fbAQLP
2+
3+
exports[`Correct beforeEach order ensures the correct order for beforeEach 1`] = `
4+
" console.log __tests__/beforeEachQueue.test.js:5
5+
BeforeEach
6+
7+
console.log __tests__/beforeEachQueue.test.js:9
8+
It Foo
9+
10+
console.log __tests__/beforeEachQueue.test.js:12
11+
BeforeEach Inline Foo
12+
13+
console.log __tests__/beforeEachQueue.test.js:5
14+
BeforeEach
15+
16+
console.log __tests__/beforeEachQueue.test.js:17
17+
It Bar
18+
"
19+
`;

‎e2e/__tests__/__snapshots__/cli-handles-exact-filenames.test.js.snap renamed to ‎e2e/__tests__/__snapshots__/cliHandlesExactFilenames.test.js.snap

File renamed without changes.

‎e2e/__tests__/__snapshots__/console_log_output_when_run_in_band.test.js.snap renamed to ‎e2e/__tests__/__snapshots__/consoleLogOutputWhenRunInBand.test.js.snap

File renamed without changes.

‎e2e/__tests__/__snapshots__/coverage_remapping.test.js.snap renamed to ‎e2e/__tests__/__snapshots__/coverageRemapping.test.js.snap

File renamed without changes.

‎e2e/__tests__/__snapshots__/coverage_report.test.js.snap renamed to ‎e2e/__tests__/__snapshots__/coverageReport.test.js.snap

+24-24
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,9 @@ File | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s |
66
---------------|----------|----------|----------|----------|-------------------|
77
All files | 100 | 100 | 100 | 100 | |
88
a | 100 | 100 | 100 | 100 | |
9-
Identical.js | 100 | 100 | 100 | 100 | |
9+
identical.js | 100 | 100 | 100 | 100 | |
1010
b | 100 | 100 | 100 | 100 | |
11-
Identical.js | 100 | 100 | 100 | 100 | |
11+
identical.js | 100 | 100 | 100 | 100 | |
1212
---------------|----------|----------|----------|----------|-------------------|"
1313
`;
1414

@@ -17,7 +17,7 @@ exports[`collects coverage only from multiple specified files 1`] = `
1717
File | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s |
1818
--------------|----------|----------|----------|----------|-------------------|
1919
All files | 100 | 100 | 100 | 100 | |
20-
OtherFile.js | 100 | 100 | 100 | 100 | |
20+
otherFile.js | 100 | 100 | 100 | 100 | |
2121
setup.js | 100 | 100 | 100 | 100 | |
2222
--------------|----------|----------|----------|----------|-------------------|"
2323
`;
@@ -36,7 +36,7 @@ exports[`collects coverage only from specified files avoiding dependencies 1`] =
3636
File | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s |
3737
----------|----------|----------|----------|----------|-------------------|
3838
All files | 85.71 | 100 | 50 | 100 | |
39-
Sum.js | 85.71 | 100 | 50 | 100 | |
39+
sum.js | 85.71 | 100 | 50 | 100 | |
4040
----------|----------|----------|----------|----------|-------------------|"
4141
`;
4242

@@ -48,14 +48,14 @@ File | % Stmts | % Branch | % Funcs | % Lines
4848
-------------------------------------|----------|----------|----------|----------|-------------------|
4949
All files | 56.52 | 0 | 50 | 55.56 | |
5050
coverage-report | 41.18 | 0 | 25 | 42.86 | |
51-
OtherFile.js | 100 | 100 | 100 | 100 | |
52-
Sum.js | 85.71 | 100 | 50 | 100 | |
53-
SumDependency.js | 0 | 0 | 0 | 0 | 8,10,12 |
5451
notRequiredInTestSuite.js | 0 | 0 | 0 | 0 | 8,15,16,17,19 |
52+
otherFile.js | 100 | 100 | 100 | 100 | |
53+
sum.js | 85.71 | 100 | 50 | 100 | |
54+
sumDependency.js | 0 | 0 | 0 | 0 | 8,10,12 |
5555
coverage-report/cached-duplicates/a | 100 | 100 | 100 | 100 | |
56-
Identical.js | 100 | 100 | 100 | 100 | |
56+
identical.js | 100 | 100 | 100 | 100 | |
5757
coverage-report/cached-duplicates/b | 100 | 100 | 100 | 100 | |
58-
Identical.js | 100 | 100 | 100 | 100 | |
58+
identical.js | 100 | 100 | 100 | 100 | |
5959
-------------------------------------|----------|----------|----------|----------|-------------------|"
6060
`;
6161

@@ -74,14 +74,14 @@ File | % Stmts | % Branch | % Funcs | % Lines
7474
-------------------------------------|----------|----------|----------|----------|-------------------|
7575
All files | 56.52 | 0 | 50 | 55.56 | |
7676
coverage-report | 41.18 | 0 | 25 | 42.86 | |
77-
OtherFile.js | 100 | 100 | 100 | 100 | |
78-
Sum.js | 85.71 | 100 | 50 | 100 | |
79-
SumDependency.js | 0 | 0 | 0 | 0 | 8,10,12 |
8077
notRequiredInTestSuite.js | 0 | 0 | 0 | 0 | 8,15,16,17,19 |
78+
otherFile.js | 100 | 100 | 100 | 100 | |
79+
sum.js | 85.71 | 100 | 50 | 100 | |
80+
sumDependency.js | 0 | 0 | 0 | 0 | 8,10,12 |
8181
coverage-report/cached-duplicates/a | 100 | 100 | 100 | 100 | |
82-
Identical.js | 100 | 100 | 100 | 100 | |
82+
identical.js | 100 | 100 | 100 | 100 | |
8383
coverage-report/cached-duplicates/b | 100 | 100 | 100 | 100 | |
84-
Identical.js | 100 | 100 | 100 | 100 | |
84+
identical.js | 100 | 100 | 100 | 100 | |
8585
-------------------------------------|----------|----------|----------|----------|-------------------|"
8686
`;
8787
@@ -98,14 +98,14 @@ File | % Stmts | % Branch | % Funcs | % Lines
9898
-------------------------------------|----------|----------|----------|----------|-------------------|
9999
All files | 56.52 | 0 | 50 | 55.56 | |
100100
coverage-report | 41.18 | 0 | 25 | 42.86 | |
101-
OtherFile.js | 100 | 100 | 100 | 100 | |
102-
Sum.js | 85.71 | 100 | 50 | 100 | |
103-
SumDependency.js | 0 | 0 | 0 | 0 | 8,10,12 |
104101
notRequiredInTestSuite.js | 0 | 0 | 0 | 0 | 8,15,16,17,19 |
102+
otherFile.js | 100 | 100 | 100 | 100 | |
103+
sum.js | 85.71 | 100 | 50 | 100 | |
104+
sumDependency.js | 0 | 0 | 0 | 0 | 8,10,12 |
105105
coverage-report/cached-duplicates/a | 100 | 100 | 100 | 100 | |
106-
Identical.js | 100 | 100 | 100 | 100 | |
106+
identical.js | 100 | 100 | 100 | 100 | |
107107
coverage-report/cached-duplicates/b | 100 | 100 | 100 | 100 | |
108-
Identical.js | 100 | 100 | 100 | 100 | |
108+
identical.js | 100 | 100 | 100 | 100 | |
109109
-------------------------------------|----------|----------|----------|----------|-------------------|"
110110
`;
111111
@@ -115,14 +115,14 @@ File | % Stmts | % Branch | % Funcs | % Lines
115115
-------------------------------------|----------|----------|----------|----------|-------------------|
116116
All files | 56.52 | 0 | 50 | 55.56 | |
117117
coverage-report | 41.18 | 0 | 25 | 42.86 | |
118-
OtherFile.js | 100 | 100 | 100 | 100 | |
119-
Sum.js | 85.71 | 100 | 50 | 100 | |
120-
SumDependency.js | 0 | 0 | 0 | 0 | 8,10,12 |
121118
notRequiredInTestSuite.js | 0 | 0 | 0 | 0 | 8,15,16,17,19 |
119+
otherFile.js | 100 | 100 | 100 | 100 | |
120+
sum.js | 85.71 | 100 | 50 | 100 | |
121+
sumDependency.js | 0 | 0 | 0 | 0 | 8,10,12 |
122122
coverage-report/cached-duplicates/a | 100 | 100 | 100 | 100 | |
123-
Identical.js | 100 | 100 | 100 | 100 | |
123+
identical.js | 100 | 100 | 100 | 100 | |
124124
coverage-report/cached-duplicates/b | 100 | 100 | 100 | 100 | |
125-
Identical.js | 100 | 100 | 100 | 100 | |
125+
identical.js | 100 | 100 | 100 | 100 | |
126126
-------------------------------------|----------|----------|----------|----------|-------------------|"
127127
`;
128128

‎e2e/__tests__/__snapshots__/coverage_threshold.test.js.snap renamed to ‎e2e/__tests__/__snapshots__/coverageThreshold.test.js.snap

File renamed without changes.

‎e2e/__tests__/__snapshots__/coverage_transform_instrumented.test.js.snap renamed to ‎e2e/__tests__/__snapshots__/coverageTransformInstrumented.test.js.snap

File renamed without changes.

‎e2e/__tests__/__snapshots__/custom_matcher_stack_trace.test.js.snap renamed to ‎e2e/__tests__/__snapshots__/customMatcherStackTrace.test.js.snap

+7-7
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
// Jest Snapshot v1, https://goo.gl/fbAQLP
22

33
exports[`works with custom matchers 1`] = `
4-
"FAIL __tests__/custom_matcher.test.js
4+
"FAIL __tests__/customMatcher.test.js
55
Custom matcher
66
✓ passes
77
✓ fails
@@ -19,12 +19,12 @@ exports[`works with custom matchers 1`] = `
1919
47 |
2020
48 | // This expecation fails due to an error we throw (intentionally)
2121
22-
at Error (__tests__/custom_matcher.test.js:45:13)
23-
at baz (__tests__/custom_matcher.test.js:43:23)
24-
at bar (__tests__/custom_matcher.test.js:42:23)
25-
at foo (__tests__/custom_matcher.test.js:52:7)
26-
at Object.callback (__tests__/custom_matcher.test.js:11:18)
27-
at Object.toCustomMatch (__tests__/custom_matcher.test.js:53:8)
22+
at Error (__tests__/customMatcher.test.js:45:13)
23+
at baz (__tests__/customMatcher.test.js:43:23)
24+
at bar (__tests__/customMatcher.test.js:42:23)
25+
at foo (__tests__/customMatcher.test.js:52:7)
26+
at Object.callback (__tests__/customMatcher.test.js:11:18)
27+
at Object.toCustomMatch (__tests__/customMatcher.test.js:53:8)
2828
2929
"
3030
`;

‎e2e/__tests__/__snapshots__/custom_reporters.test.js.snap renamed to ‎e2e/__tests__/__snapshots__/customReporters.test.js.snap

File renamed without changes.

‎e2e/__tests__/__snapshots__/detect_open_handles.js.snap renamed to ‎e2e/__tests__/__snapshots__/detectOpenHandles.js.snap

File renamed without changes.

‎e2e/__tests__/__snapshots__/each.test.js.snap

+6-6
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ exports[`formats args with pretty format when given %p 1`] = `
3535
`;
3636

3737
exports[`runs only the describe.only.each tests 1`] = `
38-
"PASS __tests__/describe-only.test.js
38+
"PASS __tests__/describeOnly.test.js
3939
passes all rows expected true == true
4040
✓ passes
4141
✓ passes
@@ -46,7 +46,7 @@ exports[`runs only the describe.only.each tests 1`] = `
4646
`;
4747

4848
exports[`shows error message when not enough arguments are supplied to tests 1`] = `
49-
"FAIL __tests__/each-exception.test.js
49+
"FAIL __tests__/eachException.test.js
5050
✕ throws exception when one argument too few are supplied $left == $right
5151
✕ throws exception when not enough arguments are supplied $left == $right
5252
@@ -72,7 +72,7 @@ exports[`shows error message when not enough arguments are supplied to tests 1`]
7272
10 | \${true} | \${true}
7373
11 | \${true}
7474
75-
at Object.<anonymous> (__tests__/each-exception.test.js:8:1)
75+
at Object.<anonymous> (__tests__/eachException.test.js:8:1)
7676
7777
● throws exception when not enough arguments are supplied $left == $right
7878
@@ -95,13 +95,13 @@ exports[`shows error message when not enough arguments are supplied to tests 1`]
9595
21 | \${true} | \${true}
9696
22 | \`(
9797
98-
at Object.<anonymous> (__tests__/each-exception.test.js:19:1)
98+
at Object.<anonymous> (__tests__/eachException.test.js:19:1)
9999
100100
"
101101
`;
102102
103103
exports[`shows only the tests with .only as being ran 1`] = `
104-
"PASS __tests__/each-only.test.js
104+
"PASS __tests__/eachOnly.test.js
105105
✓ passes one row expected true == true
106106
✓ passes one row expected true == true
107107
✓ passes one row expected true == true
@@ -112,7 +112,7 @@ exports[`shows only the tests with .only as being ran 1`] = `
112112
`;
113113
114114
exports[`shows only the tests without .skip as being ran 1`] = `
115-
"PASS __tests__/each-skip.test.js
115+
"PASS __tests__/eachSkip.test.js
116116
✓ passes one row expected true == true
117117
✓ passes one row expected true == true
118118
✓ passes one row expected true == true

0 commit comments

Comments
 (0)