From 0dabe3583e71ff67e11ca0b586a0178f7dacf996 Mon Sep 17 00:00:00 2001 From: Maciej Malkowski Date: Thu, 1 Nov 2018 11:51:36 +0700 Subject: [PATCH 01/11] Rename jest-worker woker --- packages/jest-worker/README.md | 6 +++--- packages/jest-worker/src/__tests__/index.test.js | 4 ++-- packages/jest-worker/src/__tests__/worker.test.js | 2 +- packages/jest-worker/src/index.js | 2 +- packages/jest-worker/src/types.js | 2 +- 5 files changed, 8 insertions(+), 8 deletions(-) diff --git a/packages/jest-worker/README.md b/packages/jest-worker/README.md index 5087287be92a..3c98dfdb2b5e 100644 --- a/packages/jest-worker/README.md +++ b/packages/jest-worker/README.md @@ -24,7 +24,7 @@ This example covers the minimal usage: import Worker from 'jest-worker'; async function main() { - const worker = new Worker(require.resolve('./worker')); + const worker = new Worker(require.resolve('./Worker')); const result = await worker.hello('Alice'); // "Hello, Alice" } @@ -114,7 +114,7 @@ This example covers the standard usage: import Worker from 'jest-worker'; async function main() { - const myWorker = new Worker(require.resolve('./worker'), { + const myWorker = new Worker(require.resolve('./Worker'), { exposedMethods: ['foo', 'bar', 'getWorkerId'], numWorkers: 4, }); @@ -155,7 +155,7 @@ This example covers the usage with a `computeWorkerKey` method: import Worker from 'jest-worker'; async function main() { - const myWorker = new Worker(require.resolve('./worker'), { + const myWorker = new Worker(require.resolve('./Worker'), { computeWorkerKey: (method, filename) => filename, }); diff --git a/packages/jest-worker/src/__tests__/index.test.js b/packages/jest-worker/src/__tests__/index.test.js index 349a91797b5f..ba49151b6ec2 100644 --- a/packages/jest-worker/src/__tests__/index.test.js +++ b/packages/jest-worker/src/__tests__/index.test.js @@ -30,7 +30,7 @@ beforeEach(() => { // The worker mock returns a worker with custom methods, plus it stores them // in a global list, so that they can be accessed later. This list is reset in // every test. - jest.mock('../worker', () => { + jest.mock('../Worker', () => { const fakeClass = jest.fn(() => { const fakeWorker = { getStderr: () => ({once() {}, pipe() {}}), @@ -63,7 +63,7 @@ beforeEach(() => { virtual: true, }); - Worker = require('../worker').default; + Worker = require('../Worker').default; Farm = require('../index').default; }); diff --git a/packages/jest-worker/src/__tests__/worker.test.js b/packages/jest-worker/src/__tests__/worker.test.js index 4fd91d9e3de6..cad8347df5b9 100644 --- a/packages/jest-worker/src/__tests__/worker.test.js +++ b/packages/jest-worker/src/__tests__/worker.test.js @@ -38,7 +38,7 @@ beforeEach(() => { return forkInterface; }); - Worker = require('../worker').default; + Worker = require('../Worker').default; }); afterEach(() => { diff --git a/packages/jest-worker/src/index.js b/packages/jest-worker/src/index.js index 0cb636c2fc6c..be634f2fecbd 100644 --- a/packages/jest-worker/src/index.js +++ b/packages/jest-worker/src/index.js @@ -17,7 +17,7 @@ import type {FarmOptions} from './types'; import type {Readable} from 'stream'; import {CHILD_MESSAGE_CALL, CHILD_MESSAGE_END} from './types'; -import Worker from './worker'; +import Worker from './Worker'; /* istanbul ignore next */ const emptyMethod = () => {}; diff --git a/packages/jest-worker/src/types.js b/packages/jest-worker/src/types.js index a8afc71f9f58..5d7108f9fb30 100644 --- a/packages/jest-worker/src/types.js +++ b/packages/jest-worker/src/types.js @@ -29,7 +29,7 @@ export type PARENT_MESSAGE_ERROR = // Option objects. -import type Worker from './worker'; +import type Worker from './Worker'; export type ForkOptions = { cwd?: string, From d48930ecb2677463dcfd54dcdddc7ff143d191f6 Mon Sep 17 00:00:00 2001 From: Maciej Malkowski Date: Thu, 1 Nov 2018 14:25:08 +0700 Subject: [PATCH 02/11] rename in pretty-format/src/plugins/lib/escape_html.js --- .../src/plugins/lib/{escape_html.js => escapeHTML.js} | 0 packages/pretty-format/src/plugins/lib/markup.js | 2 +- 2 files changed, 1 insertion(+), 1 deletion(-) rename packages/pretty-format/src/plugins/lib/{escape_html.js => escapeHTML.js} (100%) diff --git a/packages/pretty-format/src/plugins/lib/escape_html.js b/packages/pretty-format/src/plugins/lib/escapeHTML.js similarity index 100% rename from packages/pretty-format/src/plugins/lib/escape_html.js rename to packages/pretty-format/src/plugins/lib/escapeHTML.js diff --git a/packages/pretty-format/src/plugins/lib/markup.js b/packages/pretty-format/src/plugins/lib/markup.js index eb45835588a0..70ebceebc806 100644 --- a/packages/pretty-format/src/plugins/lib/markup.js +++ b/packages/pretty-format/src/plugins/lib/markup.js @@ -9,7 +9,7 @@ import type {Config, Printer, Refs} from 'types/PrettyFormat'; -import escapeHTML from './escape_html'; +import escapeHTML from './escapeHTML'; // Return empty string if keys is empty. export const printProps = ( From 34efa09e8b531437a6da4962e69d1dd243d5a455 Mon Sep 17 00:00:00 2001 From: Maciej Malkowski Date: Thu, 1 Nov 2018 15:16:36 +0700 Subject: [PATCH 03/11] Update CHANGELOG --- CHANGELOG.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 8a0b85b5fb40..901195f8e98e 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -95,6 +95,8 @@ - `[docs]` Add correct default value for `testUrl` config option ([#7277](https://github.com/facebook/jest/pull/7277)) - `[jest-util]` [**BREAKING**] Remove long-deprecated globals for fake timers ([#7285](https://github.com/facebook/jest/pull/7285)) - `[docs]` Remove duplicate code in `MockFunctions` ([#7297](https://github.com/facebook/jest/pull/7297)) +- `[jest-haste-map]` Standardize filenames ([#7316](https://github.com/facebook/jest/pull/7316)) +- `[pretty-format]` Standardize filenames ([#7316](https://github.com/facebook/jest/pull/7316)) ### Performance From 1da498a6f141ea0da69947a7ec882a6b3e8d5d67 Mon Sep 17 00:00:00 2001 From: Maciej Malkowski Date: Thu, 1 Nov 2018 15:34:11 +0700 Subject: [PATCH 04/11] Rename with git mv --- packages/jest-worker/src/{worker.js => Worker.js} | 0 1 file changed, 0 insertions(+), 0 deletions(-) rename packages/jest-worker/src/{worker.js => Worker.js} (100%) diff --git a/packages/jest-worker/src/worker.js b/packages/jest-worker/src/Worker.js similarity index 100% rename from packages/jest-worker/src/worker.js rename to packages/jest-worker/src/Worker.js From 634a7c3537f3cb8beee4f4257149e26d772d8601 Mon Sep 17 00:00:00 2001 From: Maciej Malkowski Date: Thu, 8 Nov 2018 09:52:11 +0700 Subject: [PATCH 05/11] Code review fixes --- CHANGELOG.md | 2 +- .../src/__tests__/{worker.test.js => Worker.test.js} | 0 2 files changed, 1 insertion(+), 1 deletion(-) rename packages/jest-worker/src/__tests__/{worker.test.js => Worker.test.js} (100%) diff --git a/CHANGELOG.md b/CHANGELOG.md index 901195f8e98e..1d377b076914 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -95,7 +95,7 @@ - `[docs]` Add correct default value for `testUrl` config option ([#7277](https://github.com/facebook/jest/pull/7277)) - `[jest-util]` [**BREAKING**] Remove long-deprecated globals for fake timers ([#7285](https://github.com/facebook/jest/pull/7285)) - `[docs]` Remove duplicate code in `MockFunctions` ([#7297](https://github.com/facebook/jest/pull/7297)) -- `[jest-haste-map]` Standardize filenames ([#7316](https://github.com/facebook/jest/pull/7316)) +- `[jest-worker]` Standardize filenames ([#7316](https://github.com/facebook/jest/pull/7316)) - `[pretty-format]` Standardize filenames ([#7316](https://github.com/facebook/jest/pull/7316)) ### Performance diff --git a/packages/jest-worker/src/__tests__/worker.test.js b/packages/jest-worker/src/__tests__/Worker.test.js similarity index 100% rename from packages/jest-worker/src/__tests__/worker.test.js rename to packages/jest-worker/src/__tests__/Worker.test.js From 7dbb66c20f59116e6881cb924d86694443004c7b Mon Sep 17 00:00:00 2001 From: Maciej Malkowski Date: Tue, 13 Nov 2018 10:53:28 +0700 Subject: [PATCH 06/11] Rename plugins and clean up exports --- jest.config.js | 4 ++-- packages/pretty-format/perf/test.js | 2 +- ...c_matcher.test.js => AsymmetricMatcher.test.js} | 0 .../{convert_ansi.test.js => ConvertAnsi.test.js} | 0 ...om_collection.test.js => DOMCollection.test.js} | 2 +- .../{dom_element.test.js => DOMElement.test.js} | 2 +- .../{immutable.test.js => Immutable.test.js} | 2 +- .../{expect_util.js => getPrettyPrint.js} | 0 ...{pretty_format.test.js => prettyFormat.test.js} | 0 packages/pretty-format/src/index.js | 14 +++++++------- ...{asymmetric_matcher.js => AsymmetricMatcher.js} | 0 .../plugins/{convert_ansi.js => ConvertAnsi.js} | 5 ++--- .../{dom_collection.js => DOMCollection.js} | 4 ++-- .../src/plugins/{dom_element.js => DOMElement.js} | 4 ++-- .../src/plugins/{immutable.js => Immutable.js} | 4 ++-- .../plugins/{react_element.js => ReactElement.js} | 4 ++-- ...act_test_component.js => ReactTestComponent.js} | 4 ++-- 17 files changed, 25 insertions(+), 26 deletions(-) rename packages/pretty-format/src/__tests__/{asymmetric_matcher.test.js => AsymmetricMatcher.test.js} (100%) rename packages/pretty-format/src/__tests__/{convert_ansi.test.js => ConvertAnsi.test.js} (100%) rename packages/pretty-format/src/__tests__/{dom_collection.test.js => DOMCollection.test.js} (98%) rename packages/pretty-format/src/__tests__/{dom_element.test.js => DOMElement.test.js} (99%) rename packages/pretty-format/src/__tests__/{immutable.test.js => Immutable.test.js} (99%) rename packages/pretty-format/src/__tests__/{expect_util.js => getPrettyPrint.js} (100%) rename packages/pretty-format/src/__tests__/{pretty_format.test.js => prettyFormat.test.js} (100%) rename packages/pretty-format/src/plugins/{asymmetric_matcher.js => AsymmetricMatcher.js} (100%) rename packages/pretty-format/src/plugins/{convert_ansi.js => ConvertAnsi.js} (94%) rename packages/pretty-format/src/plugins/{dom_collection.js => DOMCollection.js} (96%) rename packages/pretty-format/src/plugins/{dom_element.js => DOMElement.js} (97%) rename packages/pretty-format/src/plugins/{immutable.js => Immutable.js} (98%) rename packages/pretty-format/src/plugins/{react_element.js => ReactElement.js} (96%) rename packages/pretty-format/src/plugins/{react_test_component.js => ReactTestComponent.js} (94%) diff --git a/jest.config.js b/jest.config.js index 06cb6ca2e84c..d60e3f1a090c 100644 --- a/jest.config.js +++ b/jest.config.js @@ -25,7 +25,7 @@ module.exports = { projects: ['', '/examples/*/'], setupFilesAfterEnv: ['/testSetupFile.js'], snapshotSerializers: [ - '/packages/pretty-format/build/plugins/convert_ansi.js', + '/packages/pretty-format/build/plugins/ConvertAnsi.js', ], testEnvironment: './packages/jest-environment-node', testPathIgnorePatterns: [ @@ -35,7 +35,7 @@ module.exports = { '\\.snap$', '/packages/.*/build', '/packages/.*/build-es5', - '/packages/.*/src/__tests__/expect_util.js', + '/packages/.*/src/__tests__/getPrettyPrint.js', '/packages/jest-cli/src/__tests__/test_root', '/packages/jest-cli/src/__tests__/__fixtures__/', '/packages/jest-cli/src/lib/__tests__/fixtures/', diff --git a/packages/pretty-format/perf/test.js b/packages/pretty-format/perf/test.js index 9bc97b44099f..e8ac99df480c 100644 --- a/packages/pretty-format/perf/test.js +++ b/packages/pretty-format/perf/test.js @@ -12,7 +12,7 @@ const React = require('react'); const ReactTestRenderer = require('react-test-renderer'); const leftPad = require('left-pad'); const prettyFormat = require('../build'); -const ReactTestComponent = require('../build/plugins/react_test_component'); +const ReactTestComponent = require('../build/plugins/ReactTestComponent'); const worldGeoJson = require('./world.geo.json'); const NANOSECONDS = 1000000000; diff --git a/packages/pretty-format/src/__tests__/asymmetric_matcher.test.js b/packages/pretty-format/src/__tests__/AsymmetricMatcher.test.js similarity index 100% rename from packages/pretty-format/src/__tests__/asymmetric_matcher.test.js rename to packages/pretty-format/src/__tests__/AsymmetricMatcher.test.js diff --git a/packages/pretty-format/src/__tests__/convert_ansi.test.js b/packages/pretty-format/src/__tests__/ConvertAnsi.test.js similarity index 100% rename from packages/pretty-format/src/__tests__/convert_ansi.test.js rename to packages/pretty-format/src/__tests__/ConvertAnsi.test.js diff --git a/packages/pretty-format/src/__tests__/dom_collection.test.js b/packages/pretty-format/src/__tests__/DOMCollection.test.js similarity index 98% rename from packages/pretty-format/src/__tests__/dom_collection.test.js rename to packages/pretty-format/src/__tests__/DOMCollection.test.js index cf0d498f8b18..19306361f0cd 100644 --- a/packages/pretty-format/src/__tests__/dom_collection.test.js +++ b/packages/pretty-format/src/__tests__/DOMCollection.test.js @@ -13,7 +13,7 @@ const prettyFormat = require('../'); const {DOMCollection, DOMElement} = prettyFormat.plugins; -const toPrettyPrintTo = require('./expect_util').getPrettyPrint([ +const toPrettyPrintTo = require('./getPrettyPrint').getPrettyPrint([ DOMCollection, DOMElement, ]); diff --git a/packages/pretty-format/src/__tests__/dom_element.test.js b/packages/pretty-format/src/__tests__/DOMElement.test.js similarity index 99% rename from packages/pretty-format/src/__tests__/dom_element.test.js rename to packages/pretty-format/src/__tests__/DOMElement.test.js index 9b06b105766c..6a92a92a4053 100644 --- a/packages/pretty-format/src/__tests__/dom_element.test.js +++ b/packages/pretty-format/src/__tests__/DOMElement.test.js @@ -13,7 +13,7 @@ const prettyFormat = require('../'); const {DOMElement} = prettyFormat.plugins; -const toPrettyPrintTo = require('./expect_util').getPrettyPrint([DOMElement]); +const toPrettyPrintTo = require('./getPrettyPrint').getPrettyPrint([DOMElement]); const expect: any = global.expect; expect.extend({toPrettyPrintTo}); diff --git a/packages/pretty-format/src/__tests__/immutable.test.js b/packages/pretty-format/src/__tests__/Immutable.test.js similarity index 99% rename from packages/pretty-format/src/__tests__/immutable.test.js rename to packages/pretty-format/src/__tests__/Immutable.test.js index b402605cf85c..de85a75bd202 100644 --- a/packages/pretty-format/src/__tests__/immutable.test.js +++ b/packages/pretty-format/src/__tests__/Immutable.test.js @@ -11,7 +11,7 @@ import React from 'react'; import Immutable from 'immutable'; -import {getPrettyPrint} from './expect_util'; +import {getPrettyPrint} from './getPrettyPrint'; const {plugins: prettyPlugins} = require('..'); const {Immutable: ImmutablePlugin, ReactElement} = prettyPlugins; diff --git a/packages/pretty-format/src/__tests__/expect_util.js b/packages/pretty-format/src/__tests__/getPrettyPrint.js similarity index 100% rename from packages/pretty-format/src/__tests__/expect_util.js rename to packages/pretty-format/src/__tests__/getPrettyPrint.js diff --git a/packages/pretty-format/src/__tests__/pretty_format.test.js b/packages/pretty-format/src/__tests__/prettyFormat.test.js similarity index 100% rename from packages/pretty-format/src/__tests__/pretty_format.test.js rename to packages/pretty-format/src/__tests__/prettyFormat.test.js diff --git a/packages/pretty-format/src/index.js b/packages/pretty-format/src/index.js index 3c193f1a17d3..618a6fc62971 100644 --- a/packages/pretty-format/src/index.js +++ b/packages/pretty-format/src/index.js @@ -28,13 +28,13 @@ import { printObjectProperties, } from './collections'; -import AsymmetricMatcher from './plugins/asymmetric_matcher'; -import ConvertAnsi from './plugins/convert_ansi'; -import DOMCollection from './plugins/dom_collection'; -import DOMElement from './plugins/dom_element'; -import Immutable from './plugins/immutable'; -import ReactElement from './plugins/react_element'; -import ReactTestComponent from './plugins/react_test_component'; +import AsymmetricMatcher from './plugins/AsymmetricMatcher'; +import ConvertAnsi from './plugins/ConvertAnsi'; +import DOMCollection from './plugins/DOMCollection'; +import DOMElement from './plugins/DOMElement'; +import Immutable from './plugins/Immutable'; +import ReactElement from './plugins/ReactElement'; +import ReactTestComponent from './plugins/ReactTestComponent'; const toString = Object.prototype.toString; const toISOString = Date.prototype.toISOString; diff --git a/packages/pretty-format/src/plugins/asymmetric_matcher.js b/packages/pretty-format/src/plugins/AsymmetricMatcher.js similarity index 100% rename from packages/pretty-format/src/plugins/asymmetric_matcher.js rename to packages/pretty-format/src/plugins/AsymmetricMatcher.js diff --git a/packages/pretty-format/src/plugins/convert_ansi.js b/packages/pretty-format/src/plugins/ConvertAnsi.js similarity index 94% rename from packages/pretty-format/src/plugins/convert_ansi.js rename to packages/pretty-format/src/plugins/ConvertAnsi.js index 1a3d952beeff..b418054f2ae9 100644 --- a/packages/pretty-format/src/plugins/convert_ansi.js +++ b/packages/pretty-format/src/plugins/ConvertAnsi.js @@ -59,10 +59,9 @@ const toHumanReadableAnsi = text => } }); -export const test = (val: any) => - typeof val === 'string' && val.match(ansiRegex()); +const test = (val: any) => typeof val === 'string' && val.match(ansiRegex()); -export const serialize = ( +const serialize = ( val: string, config: Config, indentation: string, diff --git a/packages/pretty-format/src/plugins/dom_collection.js b/packages/pretty-format/src/plugins/DOMCollection.js similarity index 96% rename from packages/pretty-format/src/plugins/dom_collection.js rename to packages/pretty-format/src/plugins/DOMCollection.js index 716e83829617..987dbb37086f 100644 --- a/packages/pretty-format/src/plugins/dom_collection.js +++ b/packages/pretty-format/src/plugins/DOMCollection.js @@ -19,7 +19,7 @@ const ARRAY_REGEXP = /^(HTML\w*Collection|NodeList)$/; const testName = (name: any) => OBJECT_NAMES.indexOf(name) !== -1 || ARRAY_REGEXP.test(name); -export const test = (val: any) => +const test = (val: any) => val && val.constructor && val.constructor.name && @@ -31,7 +31,7 @@ const propsReducer = (props, attribute) => { return props; }; -export const serialize = ( +const serialize = ( collection: any, config: Config, indentation: string, diff --git a/packages/pretty-format/src/plugins/dom_element.js b/packages/pretty-format/src/plugins/DOMElement.js similarity index 97% rename from packages/pretty-format/src/plugins/dom_element.js rename to packages/pretty-format/src/plugins/DOMElement.js index b09cbeb1d409..fd100bce6423 100644 --- a/packages/pretty-format/src/plugins/dom_element.js +++ b/packages/pretty-format/src/plugins/DOMElement.js @@ -55,7 +55,7 @@ const testNode = (nodeType: any, name: any) => (nodeType === COMMENT_NODE && name === 'Comment') || (nodeType === FRAGMENT_NODE && name === 'DocumentFragment'); -export const test = (val: any) => +const test = (val: any) => val && val.constructor && val.constructor.name && @@ -68,7 +68,7 @@ const propsReducer = (props, attribute) => { return props; }; -export const serialize = ( +const serialize = ( node: Element | Text | Comment | DocumentFragment, config: Config, indentation: string, diff --git a/packages/pretty-format/src/plugins/immutable.js b/packages/pretty-format/src/plugins/Immutable.js similarity index 98% rename from packages/pretty-format/src/plugins/immutable.js rename to packages/pretty-format/src/plugins/Immutable.js index 06f0adcdb0c2..9bf60e7fca3d 100644 --- a/packages/pretty-format/src/plugins/immutable.js +++ b/packages/pretty-format/src/plugins/Immutable.js @@ -171,7 +171,7 @@ const printImmutableValues = ( ) + ']'; -export const serialize = ( +const serialize = ( val: any, config: Config, indentation: string, @@ -235,7 +235,7 @@ export const serialize = ( // Explicitly comparing sentinel properties to true avoids false positive // when mock identity-obj-proxy returns the key as the value for any key. -export const test = (val: any) => +const test = (val: any) => val && (val[IS_ITERABLE_SENTINEL] === true || val[IS_RECORD_SENTINEL] === true); diff --git a/packages/pretty-format/src/plugins/react_element.js b/packages/pretty-format/src/plugins/ReactElement.js similarity index 96% rename from packages/pretty-format/src/plugins/react_element.js rename to packages/pretty-format/src/plugins/ReactElement.js index f83b63361c7e..621f45834dd1 100644 --- a/packages/pretty-format/src/plugins/react_element.js +++ b/packages/pretty-format/src/plugins/ReactElement.js @@ -74,7 +74,7 @@ const getPropKeys = element => { .sort(); }; -export const serialize = ( +const serialize = ( element: React$Element, config: Config, indentation: string, @@ -107,6 +107,6 @@ export const serialize = ( indentation, ); -export const test = (val: any) => val && val.$$typeof === elementSymbol; +const test = (val: any) => val && val.$$typeof === elementSymbol; export default ({serialize, test}: NewPlugin); diff --git a/packages/pretty-format/src/plugins/react_test_component.js b/packages/pretty-format/src/plugins/ReactTestComponent.js similarity index 94% rename from packages/pretty-format/src/plugins/react_test_component.js rename to packages/pretty-format/src/plugins/ReactTestComponent.js index ca892cb9ae86..fad604b9dc71 100644 --- a/packages/pretty-format/src/plugins/react_test_component.js +++ b/packages/pretty-format/src/plugins/ReactTestComponent.js @@ -34,7 +34,7 @@ const getPropKeys = object => { : []; }; -export const serialize = ( +const serialize = ( object: ReactTestObject, config: Config, indentation: string, @@ -74,6 +74,6 @@ export const serialize = ( indentation, ); -export const test = (val: any) => val && val.$$typeof === testSymbol; +const test = (val: any) => val && val.$$typeof === testSymbol; export default ({serialize, test}: NewPlugin); From 104ff3a3b92c97a77cd907982dc06c12507a872b Mon Sep 17 00:00:00 2001 From: Maciej Malkowski Date: Tue, 13 Nov 2018 10:57:29 +0700 Subject: [PATCH 07/11] Fix formatting issue --- packages/pretty-format/src/__tests__/DOMElement.test.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/packages/pretty-format/src/__tests__/DOMElement.test.js b/packages/pretty-format/src/__tests__/DOMElement.test.js index 6a92a92a4053..c32339d23f9c 100644 --- a/packages/pretty-format/src/__tests__/DOMElement.test.js +++ b/packages/pretty-format/src/__tests__/DOMElement.test.js @@ -13,7 +13,9 @@ const prettyFormat = require('../'); const {DOMElement} = prettyFormat.plugins; -const toPrettyPrintTo = require('./getPrettyPrint').getPrettyPrint([DOMElement]); +const toPrettyPrintTo = require('./getPrettyPrint').getPrettyPrint([ + DOMElement, +]); const expect: any = global.expect; expect.extend({toPrettyPrintTo}); From e07b770d591ab7c7a6a95e5137d5e407135ed0b5 Mon Sep 17 00:00:00 2001 From: Maciej Malkowski Date: Tue, 13 Nov 2018 11:15:53 +0700 Subject: [PATCH 08/11] Remove exports from AsymmetricMatcher --- packages/pretty-format/src/plugins/AsymmetricMatcher.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/pretty-format/src/plugins/AsymmetricMatcher.js b/packages/pretty-format/src/plugins/AsymmetricMatcher.js index 7cdb099fc30e..914b50a84105 100644 --- a/packages/pretty-format/src/plugins/AsymmetricMatcher.js +++ b/packages/pretty-format/src/plugins/AsymmetricMatcher.js @@ -14,7 +14,7 @@ import {printListItems, printObjectProperties} from '../collections'; const asymmetricMatcher = Symbol.for('jest.asymmetricMatcher'); const SPACE = ' '; -export const serialize = ( +const serialize = ( val: any, config: Config, indentation: string, @@ -88,6 +88,6 @@ export const serialize = ( return val.toAsymmetricMatcher(); }; -export const test = (val: any) => val && val.$$typeof === asymmetricMatcher; +const test = (val: any) => val && val.$$typeof === asymmetricMatcher; export default ({serialize, test}: NewPlugin); From 0e7a6d2e1504130757af246c1cce0a86c46b4c7f Mon Sep 17 00:00:00 2001 From: Maciej Malkowski Date: Tue, 13 Nov 2018 14:49:43 +0700 Subject: [PATCH 09/11] Revert remove exports for test/serialize --- packages/pretty-format/src/plugins/AsymmetricMatcher.js | 4 ++-- packages/pretty-format/src/plugins/ConvertAnsi.js | 5 +++-- packages/pretty-format/src/plugins/DOMCollection.js | 4 ++-- packages/pretty-format/src/plugins/DOMElement.js | 4 ++-- packages/pretty-format/src/plugins/Immutable.js | 4 ++-- packages/pretty-format/src/plugins/ReactElement.js | 4 ++-- packages/pretty-format/src/plugins/ReactTestComponent.js | 4 ++-- 7 files changed, 15 insertions(+), 14 deletions(-) diff --git a/packages/pretty-format/src/plugins/AsymmetricMatcher.js b/packages/pretty-format/src/plugins/AsymmetricMatcher.js index 914b50a84105..7cdb099fc30e 100644 --- a/packages/pretty-format/src/plugins/AsymmetricMatcher.js +++ b/packages/pretty-format/src/plugins/AsymmetricMatcher.js @@ -14,7 +14,7 @@ import {printListItems, printObjectProperties} from '../collections'; const asymmetricMatcher = Symbol.for('jest.asymmetricMatcher'); const SPACE = ' '; -const serialize = ( +export const serialize = ( val: any, config: Config, indentation: string, @@ -88,6 +88,6 @@ const serialize = ( return val.toAsymmetricMatcher(); }; -const test = (val: any) => val && val.$$typeof === asymmetricMatcher; +export const test = (val: any) => val && val.$$typeof === asymmetricMatcher; export default ({serialize, test}: NewPlugin); diff --git a/packages/pretty-format/src/plugins/ConvertAnsi.js b/packages/pretty-format/src/plugins/ConvertAnsi.js index b418054f2ae9..1a3d952beeff 100644 --- a/packages/pretty-format/src/plugins/ConvertAnsi.js +++ b/packages/pretty-format/src/plugins/ConvertAnsi.js @@ -59,9 +59,10 @@ const toHumanReadableAnsi = text => } }); -const test = (val: any) => typeof val === 'string' && val.match(ansiRegex()); +export const test = (val: any) => + typeof val === 'string' && val.match(ansiRegex()); -const serialize = ( +export const serialize = ( val: string, config: Config, indentation: string, diff --git a/packages/pretty-format/src/plugins/DOMCollection.js b/packages/pretty-format/src/plugins/DOMCollection.js index 987dbb37086f..716e83829617 100644 --- a/packages/pretty-format/src/plugins/DOMCollection.js +++ b/packages/pretty-format/src/plugins/DOMCollection.js @@ -19,7 +19,7 @@ const ARRAY_REGEXP = /^(HTML\w*Collection|NodeList)$/; const testName = (name: any) => OBJECT_NAMES.indexOf(name) !== -1 || ARRAY_REGEXP.test(name); -const test = (val: any) => +export const test = (val: any) => val && val.constructor && val.constructor.name && @@ -31,7 +31,7 @@ const propsReducer = (props, attribute) => { return props; }; -const serialize = ( +export const serialize = ( collection: any, config: Config, indentation: string, diff --git a/packages/pretty-format/src/plugins/DOMElement.js b/packages/pretty-format/src/plugins/DOMElement.js index fd100bce6423..b09cbeb1d409 100644 --- a/packages/pretty-format/src/plugins/DOMElement.js +++ b/packages/pretty-format/src/plugins/DOMElement.js @@ -55,7 +55,7 @@ const testNode = (nodeType: any, name: any) => (nodeType === COMMENT_NODE && name === 'Comment') || (nodeType === FRAGMENT_NODE && name === 'DocumentFragment'); -const test = (val: any) => +export const test = (val: any) => val && val.constructor && val.constructor.name && @@ -68,7 +68,7 @@ const propsReducer = (props, attribute) => { return props; }; -const serialize = ( +export const serialize = ( node: Element | Text | Comment | DocumentFragment, config: Config, indentation: string, diff --git a/packages/pretty-format/src/plugins/Immutable.js b/packages/pretty-format/src/plugins/Immutable.js index 9bf60e7fca3d..06f0adcdb0c2 100644 --- a/packages/pretty-format/src/plugins/Immutable.js +++ b/packages/pretty-format/src/plugins/Immutable.js @@ -171,7 +171,7 @@ const printImmutableValues = ( ) + ']'; -const serialize = ( +export const serialize = ( val: any, config: Config, indentation: string, @@ -235,7 +235,7 @@ const serialize = ( // Explicitly comparing sentinel properties to true avoids false positive // when mock identity-obj-proxy returns the key as the value for any key. -const test = (val: any) => +export const test = (val: any) => val && (val[IS_ITERABLE_SENTINEL] === true || val[IS_RECORD_SENTINEL] === true); diff --git a/packages/pretty-format/src/plugins/ReactElement.js b/packages/pretty-format/src/plugins/ReactElement.js index 621f45834dd1..f83b63361c7e 100644 --- a/packages/pretty-format/src/plugins/ReactElement.js +++ b/packages/pretty-format/src/plugins/ReactElement.js @@ -74,7 +74,7 @@ const getPropKeys = element => { .sort(); }; -const serialize = ( +export const serialize = ( element: React$Element, config: Config, indentation: string, @@ -107,6 +107,6 @@ const serialize = ( indentation, ); -const test = (val: any) => val && val.$$typeof === elementSymbol; +export const test = (val: any) => val && val.$$typeof === elementSymbol; export default ({serialize, test}: NewPlugin); diff --git a/packages/pretty-format/src/plugins/ReactTestComponent.js b/packages/pretty-format/src/plugins/ReactTestComponent.js index fad604b9dc71..ca892cb9ae86 100644 --- a/packages/pretty-format/src/plugins/ReactTestComponent.js +++ b/packages/pretty-format/src/plugins/ReactTestComponent.js @@ -34,7 +34,7 @@ const getPropKeys = object => { : []; }; -const serialize = ( +export const serialize = ( object: ReactTestObject, config: Config, indentation: string, @@ -74,6 +74,6 @@ const serialize = ( indentation, ); -const test = (val: any) => val && val.$$typeof === testSymbol; +export const test = (val: any) => val && val.$$typeof === testSymbol; export default ({serialize, test}: NewPlugin); From 152d4761e1e18c82d608bc0c429de3d668646c2f Mon Sep 17 00:00:00 2001 From: Maciej Malkowski Date: Tue, 13 Nov 2018 16:59:02 +0700 Subject: [PATCH 10/11] Change getPrettyPrint to default export --- packages/pretty-format/src/__tests__/Immutable.test.js | 2 +- packages/pretty-format/src/__tests__/getPrettyPrint.js | 4 +++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/packages/pretty-format/src/__tests__/Immutable.test.js b/packages/pretty-format/src/__tests__/Immutable.test.js index de85a75bd202..4e0e40cf91e7 100644 --- a/packages/pretty-format/src/__tests__/Immutable.test.js +++ b/packages/pretty-format/src/__tests__/Immutable.test.js @@ -11,7 +11,7 @@ import React from 'react'; import Immutable from 'immutable'; -import {getPrettyPrint} from './getPrettyPrint'; +import getPrettyPrint from './getPrettyPrint'; const {plugins: prettyPlugins} = require('..'); const {Immutable: ImmutablePlugin, ReactElement} = prettyPlugins; diff --git a/packages/pretty-format/src/__tests__/getPrettyPrint.js b/packages/pretty-format/src/__tests__/getPrettyPrint.js index 1fb9a1dc0d8b..5ee5ee04929b 100644 --- a/packages/pretty-format/src/__tests__/getPrettyPrint.js +++ b/packages/pretty-format/src/__tests__/getPrettyPrint.js @@ -14,7 +14,7 @@ import type {OptionsReceived, Plugins} from 'types/PrettyFormat'; const diff = require('jest-diff'); const prettyFormat = require('../'); -export const getPrettyPrint = (plugins: Plugins) => +const getPrettyPrint = (plugins: Plugins) => function(received: any, expected: any, options?: OptionsReceived) { const prettyFormatted = prettyFormat( received, @@ -52,3 +52,5 @@ export const getPrettyPrint = (plugins: Plugins) => return {actual: prettyFormatted, message, pass}; }; + +export default getPrettyPrint; From 64b87ff980373f4f2381b5453bddad8ca9cf1f87 Mon Sep 17 00:00:00 2001 From: Maciej Malkowski Date: Tue, 13 Nov 2018 17:53:03 +0700 Subject: [PATCH 11/11] Fix require default --- packages/pretty-format/src/__tests__/DOMCollection.test.js | 2 +- packages/pretty-format/src/__tests__/DOMElement.test.js | 4 +--- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/packages/pretty-format/src/__tests__/DOMCollection.test.js b/packages/pretty-format/src/__tests__/DOMCollection.test.js index 19306361f0cd..204068c2561c 100644 --- a/packages/pretty-format/src/__tests__/DOMCollection.test.js +++ b/packages/pretty-format/src/__tests__/DOMCollection.test.js @@ -13,7 +13,7 @@ const prettyFormat = require('../'); const {DOMCollection, DOMElement} = prettyFormat.plugins; -const toPrettyPrintTo = require('./getPrettyPrint').getPrettyPrint([ +const toPrettyPrintTo = require('./getPrettyPrint').default([ DOMCollection, DOMElement, ]); diff --git a/packages/pretty-format/src/__tests__/DOMElement.test.js b/packages/pretty-format/src/__tests__/DOMElement.test.js index c32339d23f9c..e211b33665d9 100644 --- a/packages/pretty-format/src/__tests__/DOMElement.test.js +++ b/packages/pretty-format/src/__tests__/DOMElement.test.js @@ -13,9 +13,7 @@ const prettyFormat = require('../'); const {DOMElement} = prettyFormat.plugins; -const toPrettyPrintTo = require('./getPrettyPrint').getPrettyPrint([ - DOMElement, -]); +const toPrettyPrintTo = require('./getPrettyPrint').default([DOMElement]); const expect: any = global.expect; expect.extend({toPrettyPrintTo});