Skip to content

Commit

Permalink
chore: Delete obsolete FlowFixMe comments (#8885)
Browse files Browse the repository at this point in the history
  • Loading branch information
pedrottimark authored and SimenB committed Aug 29, 2019
1 parent ef0b327 commit ff81c28
Show file tree
Hide file tree
Showing 23 changed files with 6 additions and 47 deletions.
12 changes: 6 additions & 6 deletions e2e/__tests__/__snapshots__/failures.test.ts.snap
Expand Up @@ -265,14 +265,14 @@ FAIL __tests__/testMacro.test.js
Expected: 2
Received: 1
10 |
11 | module.exports = (one: any, two: any) => {
> 12 | expect(one).toEqual(two);
8 |
9 | module.exports = (one: any, two: any) => {
> 10 | expect(one).toEqual(two);
| ^
13 | };
14 |
11 | };
12 |
at toEqual (macros.js:12:15)
at toEqual (macros.js:10:15)
at Object.shouldEqual (__tests__/testMacro.test.js:13:3)
`;
Expand Down
1 change: 0 additions & 1 deletion e2e/__tests__/detectOpenHandles.ts
Expand Up @@ -9,7 +9,6 @@ import {wrap} from 'jest-snapshot-serializer-raw';
import runJest, {until} from '../runJest';

try {
// $FlowFixMe: Node core
require('async_hooks');
} catch (e) {
if (e.code === 'MODULE_NOT_FOUND') {
Expand Down
2 changes: 0 additions & 2 deletions e2e/__tests__/iterator-to-null-test.ts
Expand Up @@ -8,9 +8,7 @@

'use strict';

// $FlowFixMe
Array.prototype[Symbol.iterator] = null;
// $FlowFixMe
String.prototype[Symbol.iterator] = null;

test('modifying global object does not affect test runner', () => {});
3 changes: 0 additions & 3 deletions e2e/__tests__/snapshot.test.ts
Expand Up @@ -107,7 +107,6 @@ describe('Snapshot', () => {
expect(json.numPendingTests).toBe(0);
expect(result.status).toBe(0);

// $FlowFixMe dynamic require
const content = require(snapshotFile);
expect(
content['snapshot is not influenced by previous counter 1'],
Expand Down Expand Up @@ -238,7 +237,6 @@ describe('Snapshot', () => {
it('works on subsequent runs without `-u`', () => {
const firstRun = runWithJson('snapshot', ['-w=1', '--ci=false']);

// $FlowFixMe dynamic require
const content = require(snapshotOfCopy);
expect(content).not.toBe(undefined);
const secondRun = runWithJson('snapshot', []);
Expand All @@ -257,7 +255,6 @@ describe('Snapshot', () => {
const firstRun = runWithJson('snapshot', ['-w=1', '--ci=false']);
fs.unlinkSync(copyOfTestPath);

// $FlowFixMe dynamic require
const content = require(snapshotOfCopy);
expect(content).not.toBe(undefined);
const secondRun = runWithJson('snapshot', ['-w=1', '--ci=false', '-u']);
Expand Down
1 change: 0 additions & 1 deletion e2e/__tests__/snapshotMockFs.test.ts
Expand Up @@ -28,7 +28,6 @@ test('store snapshot even if fs is mocked', () => {
expect(stderr).toMatch('1 snapshot written from 1 test suite.');
expect(wrap(extractSummary(stderr).summary)).toMatchSnapshot();

// $FlowFixMe dynamic require
const content = require(snapshotFile);
expect(content['snapshot 1']).toBe(`
Object {
Expand Down
1 change: 0 additions & 1 deletion e2e/__tests__/snapshotResolver.test.ts
Expand Up @@ -33,7 +33,6 @@ describe('Custom snapshot resolver', () => {

expect(result.stderr).toMatch('1 snapshot written from 1 test suite');

// $FlowFixMe dynamic require
const content = require(snapshotFile);
expect(content).toHaveProperty(
'snapshots are written to custom location 1',
Expand Down
1 change: 0 additions & 1 deletion e2e/__tests__/snapshotSerializers.test.ts
Expand Up @@ -33,7 +33,6 @@ describe('Snapshot serializers', () => {

it('renders snapshot', () => {
runAndAssert();
// $FlowFixMe dynamic require
const snapshot = require(snapshotPath);
expect(snapshot).toMatchSnapshot();
});
Expand Down
2 changes: 0 additions & 2 deletions e2e/auto-clear-mocks/with-auto-clear/index.js
Expand Up @@ -3,8 +3,6 @@
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @flow
*/

module.exports = () => {};
2 changes: 0 additions & 2 deletions e2e/bad-source-map/__tests__/badSourceMap.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 0 additions & 2 deletions e2e/clear-cache/__tests__/clearCache.test.js
Expand Up @@ -3,8 +3,6 @@
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @flow
*/
'use strict';

Expand Down
2 changes: 0 additions & 2 deletions e2e/failures/macros.js
Expand Up @@ -3,8 +3,6 @@
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @flow
*/
'use strict';

Expand Down
2 changes: 0 additions & 2 deletions e2e/mock-names/with-empty-mock-name-not-called/index.js
Expand Up @@ -3,8 +3,6 @@
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @flow
*/

module.exports = () => {};
2 changes: 0 additions & 2 deletions e2e/mock-names/with-empty-mock-name/index.js
Expand Up @@ -3,8 +3,6 @@
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @flow
*/

module.exports = () => {};
2 changes: 0 additions & 2 deletions e2e/mock-names/with-mock-name-call-times-fail/index.js
Expand Up @@ -3,8 +3,6 @@
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @flow
*/

module.exports = () => {};
2 changes: 0 additions & 2 deletions e2e/mock-names/with-mock-name-call-times-pass/index.js
Expand Up @@ -3,8 +3,6 @@
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @flow
*/

module.exports = () => {};
2 changes: 0 additions & 2 deletions e2e/mock-names/with-mock-name-not-called-fail/index.js
Expand Up @@ -3,8 +3,6 @@
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @flow
*/

module.exports = () => {};
2 changes: 0 additions & 2 deletions e2e/mock-names/with-mock-name-not-called-pass/index.js
Expand Up @@ -3,8 +3,6 @@
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @flow
*/

module.exports = () => {};
2 changes: 0 additions & 2 deletions e2e/mock-names/with-mock-name-not-called/index.js
Expand Up @@ -3,8 +3,6 @@
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @flow
*/

module.exports = () => {};
2 changes: 0 additions & 2 deletions e2e/mock-names/with-mock-name/index.js
Expand Up @@ -3,8 +3,6 @@
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @flow
*/

module.exports = () => {};
2 changes: 0 additions & 2 deletions e2e/mock-names/without-mock-name-not-called/index.js
Expand Up @@ -3,8 +3,6 @@
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @flow
*/

module.exports = () => {};
2 changes: 0 additions & 2 deletions e2e/mock-names/without-mock-name/index.js
Expand Up @@ -3,8 +3,6 @@
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @flow
*/

module.exports = () => {};
2 changes: 0 additions & 2 deletions e2e/override-globals/index.js
Expand Up @@ -3,8 +3,6 @@
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @flow
*/

global.Promise = function() {
Expand Down
2 changes: 0 additions & 2 deletions e2e/run-programmatically/index.js
Expand Up @@ -4,8 +4,6 @@
*
* This source code is licensed under the MIT license found in the
* LICENSE file in the root directory of this source tree.
*
* @flow
*/

require('@babel/register');
Expand Down

0 comments on commit ff81c28

Please sign in to comment.