From 00b8ec2b2d05e89a67bcc7c6e3068bbba23d56ba Mon Sep 17 00:00:00 2001 From: Hypnosphi Date: Tue, 22 Oct 2019 01:10:12 +0200 Subject: [PATCH] Automate codemod tests creation and update --- lib/codemod/package.json | 3 +- ... add-component-parameters.output.snapshot} | 28 +++--- ...{basic.output.js => basic.output.snapshot} | 8 +- ...s.output.js => decorators.output.snapshot} | 8 +- ...put.js => exclude-stories.output.snapshot} | 8 +- ...s.output.js => parameters.output.snapshot} | 8 +- ...tput.js => story-function.output.snapshot} | 8 +- ...ut.js => story-parameters.output.snapshot} | 8 +- .../mdx-to-csf/basic.output.js | 31 ------- .../mdx-to-csf/basic.output.snapshot | 35 +++++++ .../mdx-to-csf/decorators.output.js | 14 --- .../mdx-to-csf/decorators.output.snapshot | 18 ++++ ...put.js => exclude-stories.output.snapshot} | 14 ++- ...s.output.js => parameters.output.snapshot} | 10 +- .../mdx-to-csf/plaintext.output.js | 3 - .../mdx-to-csf/plaintext.output.snapshot | 7 ++ ...tput.js => story-function.output.snapshot} | 8 +- .../mdx-to-csf/story-parameters.output.js | 26 ------ .../story-parameters.output.snapshot | 30 ++++++ ...s.output.js => story-refs.output.snapshot} | 8 +- .../move-buildin-addons/default.output.js | 4 - .../default.output.snapshot | 8 ++ .../with-no-change.output.js | 3 - .../with-no-change.output.snapshot | 7 ++ .../storiesof-to-csf/basic.output.js | 41 --------- .../storiesof-to-csf/basic.output.snapshot | 45 +++++++++ ...on.output.js => collision.output.snapshot} | 8 +- .../storiesof-to-csf/const.output.js | 9 -- .../storiesof-to-csf/const.output.snapshot | 13 +++ .../storiesof-to-csf/decorators.output.js | 14 --- .../decorators.output.snapshot | 18 ++++ .../storiesof-to-csf/default.output.js | 13 --- .../storiesof-to-csf/default.output.snapshot | 17 ++++ .../storiesof-to-csf/digit.output.js | 5 - .../storiesof-to-csf/digit.output.snapshot | 9 ++ .../storiesof-to-csf/exports.output.js | 12 --- .../storiesof-to-csf/exports.output.snapshot | 16 ++++ .../storiesof-to-csf/module.output.js | 12 --- .../storiesof-to-csf/module.output.snapshot | 16 ++++ .../storiesof-to-csf/multi.output.js | 35 ------- .../storiesof-to-csf/multi.output.snapshot | 39 ++++++++ .../storiesof-to-csf/parameters.output.js | 19 ---- .../parameters.output.snapshot | 23 +++++ ...ut.js => story-decorators.output.snapshot} | 14 ++- .../story-parameters.output.js | 28 ------ .../story-parameters.output.snapshot | 32 +++++++ ...t.js => update-addon-info.output.snapshot} | 92 ++++++++++--------- ... update-organisation-name.output.snapshot} | 8 +- .../add-component-parameters.test.js | 8 -- .../transforms/__tests__/csf-to-mdx.test.js | 14 --- .../transforms/__tests__/mdx-to-csf.test.js | 16 ---- .../__tests__/move-buildin-addons.js | 4 - .../__tests__/storiesof-to-csf.test.js | 14 --- .../transforms/__tests__/transforms.tests.js | 32 +++++++ .../__tests__/update-addon-info.test.js | 3 - .../update-organisation-name.test.js | 8 -- yarn.lock | 39 ++++++++ 57 files changed, 555 insertions(+), 426 deletions(-) rename lib/codemod/src/transforms/__testfixtures__/add-component-parameters/{add-component-parameters.output.js => add-component-parameters.output.snapshot} (60%) rename lib/codemod/src/transforms/__testfixtures__/csf-to-mdx/{basic.output.js => basic.output.snapshot} (69%) rename lib/codemod/src/transforms/__testfixtures__/csf-to-mdx/{decorators.output.js => decorators.output.snapshot} (55%) rename lib/codemod/src/transforms/__testfixtures__/csf-to-mdx/{exclude-stories.output.js => exclude-stories.output.snapshot} (70%) rename lib/codemod/src/transforms/__testfixtures__/csf-to-mdx/{parameters.output.js => parameters.output.snapshot} (60%) rename lib/codemod/src/transforms/__testfixtures__/csf-to-mdx/{story-function.output.js => story-function.output.snapshot} (61%) rename lib/codemod/src/transforms/__testfixtures__/csf-to-mdx/{story-parameters.output.js => story-parameters.output.snapshot} (60%) delete mode 100644 lib/codemod/src/transforms/__testfixtures__/mdx-to-csf/basic.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/mdx-to-csf/basic.output.snapshot delete mode 100644 lib/codemod/src/transforms/__testfixtures__/mdx-to-csf/decorators.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/mdx-to-csf/decorators.output.snapshot rename lib/codemod/src/transforms/__testfixtures__/mdx-to-csf/{exclude-stories.output.js => exclude-stories.output.snapshot} (51%) rename lib/codemod/src/transforms/__testfixtures__/mdx-to-csf/{parameters.output.js => parameters.output.snapshot} (51%) delete mode 100644 lib/codemod/src/transforms/__testfixtures__/mdx-to-csf/plaintext.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/mdx-to-csf/plaintext.output.snapshot rename lib/codemod/src/transforms/__testfixtures__/mdx-to-csf/{story-function.output.js => story-function.output.snapshot} (62%) delete mode 100644 lib/codemod/src/transforms/__testfixtures__/mdx-to-csf/story-parameters.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/mdx-to-csf/story-parameters.output.snapshot rename lib/codemod/src/transforms/__testfixtures__/mdx-to-csf/{story-refs.output.js => story-refs.output.snapshot} (76%) delete mode 100644 lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/default.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/default.output.snapshot delete mode 100644 lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/with-no-change.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/with-no-change.output.snapshot delete mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/basic.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/basic.output.snapshot rename lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/{collision.output.js => collision.output.snapshot} (74%) delete mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/const.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/const.output.snapshot delete mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/decorators.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/decorators.output.snapshot delete mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/default.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/default.output.snapshot delete mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/digit.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/digit.output.snapshot delete mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/exports.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/exports.output.snapshot delete mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/module.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/module.output.snapshot delete mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/multi.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/multi.output.snapshot delete mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/parameters.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/parameters.output.snapshot rename lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/{story-decorators.output.js => story-decorators.output.snapshot} (50%) delete mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/story-parameters.output.js create mode 100644 lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/story-parameters.output.snapshot rename lib/codemod/src/transforms/__testfixtures__/update-addon-info/{update-addon-info.output.js => update-addon-info.output.snapshot} (67%) rename lib/codemod/src/transforms/__testfixtures__/update-organisation-name/{update-organisation-name.output.js => update-organisation-name.output.snapshot} (71%) delete mode 100644 lib/codemod/src/transforms/__tests__/add-component-parameters.test.js delete mode 100644 lib/codemod/src/transforms/__tests__/csf-to-mdx.test.js delete mode 100644 lib/codemod/src/transforms/__tests__/mdx-to-csf.test.js delete mode 100644 lib/codemod/src/transforms/__tests__/move-buildin-addons.js delete mode 100644 lib/codemod/src/transforms/__tests__/storiesof-to-csf.test.js create mode 100644 lib/codemod/src/transforms/__tests__/transforms.tests.js delete mode 100644 lib/codemod/src/transforms/__tests__/update-addon-info.test.js delete mode 100644 lib/codemod/src/transforms/__tests__/update-organisation-name.test.js diff --git a/lib/codemod/package.json b/lib/codemod/package.json index 49bd9ca30ab..53867a699e6 100644 --- a/lib/codemod/package.json +++ b/lib/codemod/package.json @@ -27,13 +27,14 @@ "prepare": "node ../../scripts/prepare.js" }, "dependencies": { + "@hypnosphi/jscodeshift": "^0.6.4", "@mdx-js/mdx": "^1.0.0", "@storybook/node-logger": "5.3.0-alpha.23", "@storybook/router": "5.3.0-alpha.23", "core-js": "^3.0.1", "cross-spawn": "^7.0.0", "globby": "^10.0.1", - "jscodeshift": "^0.6.3", + "jest-specific-snapshot": "^2.0.0", "lodash": "^4.17.15", "prettier": "^1.16.4", "recast": "^0.16.1", diff --git a/lib/codemod/src/transforms/__testfixtures__/add-component-parameters/add-component-parameters.output.js b/lib/codemod/src/transforms/__testfixtures__/add-component-parameters/add-component-parameters.output.snapshot similarity index 60% rename from lib/codemod/src/transforms/__testfixtures__/add-component-parameters/add-component-parameters.output.js rename to lib/codemod/src/transforms/__testfixtures__/add-component-parameters/add-component-parameters.output.snapshot index c196d5e03a1..5391358b111 100644 --- a/lib/codemod/src/transforms/__testfixtures__/add-component-parameters/add-component-parameters.output.js +++ b/lib/codemod/src/transforms/__testfixtures__/add-component-parameters/add-component-parameters.output.snapshot @@ -1,4 +1,7 @@ -/* eslint-disable */ +// Jest Snapshot v1, https://goo.gl/fbAQLP + +exports[`add-component-parameters transforms correctly using "add-component-parameters.input.js" data 1`] = ` +"/* eslint-disable */ import React from 'react'; import Button from './Button'; @@ -7,15 +10,15 @@ import { action } from '@storybook/addon-actions'; storiesOf('Button', module).addParameters({ component: Button -}).add('basic', () => soloStory.story = { name: 'solo story', -}; +};" +`; diff --git a/lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/default.output.js b/lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/default.output.js deleted file mode 100644 index a48ab2f6cf8..00000000000 --- a/lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/default.output.js +++ /dev/null @@ -1,4 +0,0 @@ -/* eslint-disable */ -import { storyOf } from '@storybook/react'; -import { linkTo } from '@storybook/addon-links'; -import { action } from '@storybook/addon-actions'; diff --git a/lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/default.output.snapshot b/lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/default.output.snapshot new file mode 100644 index 00000000000..3e4d1faec48 --- /dev/null +++ b/lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/default.output.snapshot @@ -0,0 +1,8 @@ +// Jest Snapshot v1, https://goo.gl/fbAQLP + +exports[`move-buildin-addons transforms correctly using "default.input.js" data 1`] = ` +"/* eslint-disable */ +import { storyOf } from '@storybook/react'; +import { linkTo } from '@storybook/addon-links'; +import { action } from '@storybook/addon-actions';" +`; diff --git a/lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/with-no-change.output.js b/lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/with-no-change.output.js deleted file mode 100644 index edc6a6d2e28..00000000000 --- a/lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/with-no-change.output.js +++ /dev/null @@ -1,3 +0,0 @@ -/* eslint-disable */ -import { action } from '@kadira/storybook-addons'; -import { storyOf } from '@storybook/react'; diff --git a/lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/with-no-change.output.snapshot b/lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/with-no-change.output.snapshot new file mode 100644 index 00000000000..eef34d05c50 --- /dev/null +++ b/lib/codemod/src/transforms/__testfixtures__/move-buildin-addons/with-no-change.output.snapshot @@ -0,0 +1,7 @@ +// Jest Snapshot v1, https://goo.gl/fbAQLP + +exports[`move-buildin-addons transforms correctly using "with-no-change.input.js" data 1`] = ` +"/* eslint-disable */ +import { action } from '@kadira/storybook-addons'; +import { storyOf } from '@storybook/react';" +`; diff --git a/lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/basic.output.js b/lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/basic.output.js deleted file mode 100644 index a678481c047..00000000000 --- a/lib/codemod/src/transforms/__testfixtures__/storiesof-to-csf/basic.output.js +++ /dev/null @@ -1,41 +0,0 @@ -/* eslint-disable */ -import React from 'react'; -import Button from './Button'; - -import { configure } from '@storybook/react'; -import { action } from '@storybook/addon-actions'; - -export default { - title: 'Button', -}; - -export const story1 = () =>