Skip to content

Commit 5f4e5cd

Browse files
GGonryunthymikee
GGonryun
authored and
thymikee
committed
standardize filenames in e2e (#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 (#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 (#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 <amodestduck@gmail.com> * 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 e1c226c commit 5f4e5cd

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

Lines changed: 1 addition & 1 deletion
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

Lines changed: 0 additions & 19 deletions
This file was deleted.

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

Lines changed: 0 additions & 19 deletions
This file was deleted.

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

Lines changed: 19 additions & 0 deletions
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

Lines changed: 19 additions & 0 deletions
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

Lines changed: 24 additions & 24 deletions
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.

0 commit comments

Comments
 (0)