@atlaskit/button 17.17.2 → 17.18.0

package/codemods/__tests__/15.1.1-data-testid/rename-data-testid.tsx

@@ -4,16 +4,16 @@ import transformer from '../../15.1.1-data-testid';
 import { check } from '../_framework';
 
 check({
-  transformer,
-  it: 'should rename data-testid to testId (default import)',
-  original: `
+	transformer,
+	it: 'should rename data-testid to testId (default import)',
+	original: `
     import DSButton from '@atlaskit/button';
 
     function App() {
       return <DSButton data-testid="my-testid">click me</DSButton>;
     }
   `,
-  expected: `
+	expected: `
     import DSButton from '@atlaskit/button';
 
     function App() {
@@ -23,16 +23,16 @@ check({
 });
 
 check({
-  transformer,
-  it: 'should rename data-testid to testId (standard button)',
-  original: `
+	transformer,
+	it: 'should rename data-testid to testId (standard button)',
+	original: `
     import Button from '@atlaskit/button/standard-button';
 
     function App() {
       return <Button data-testid="my-testid">click me</Button>;
     }
   `,
-  expected: `
+	expected: `
     import Button from '@atlaskit/button/standard-button';
 
     function App() {
@@ -42,16 +42,16 @@ check({
 });
 
 check({
-  transformer,
-  it: 'should rename data-testid to testId (loading button)',
-  original: `
+	transformer,
+	it: 'should rename data-testid to testId (loading button)',
+	original: `
     import Button from '@atlaskit/button/loading-button';
 
     function App() {
       return <Button data-testid="my-testid">click me</Button>;
     }
   `,
-  expected: `
+	expected: `
     import Button from '@atlaskit/button/loading-button';
 
     function App() {
@@ -61,16 +61,16 @@ check({
 });
 
 check({
-  transformer,
-  it: 'should rename data-testid to testId (custom theme button)',
-  original: `
+	transformer,
+	it: 'should rename data-testid to testId (custom theme button)',
+	original: `
     import Button from '@atlaskit/button/custom-theme-button';
 
     function App() {
       return <Button data-testid="my-testid">click me</Button>;
     }
   `,
-  expected: `
+	expected: `
     import Button from '@atlaskit/button/custom-theme-button';
 
     function App() {
@@ -80,9 +80,9 @@ check({
 });
 
 check({
-  transformer,
-  it: 'should rename data-testid to testId (named imports)',
-  original: `
+	transformer,
+	it: 'should rename data-testid to testId (named imports)',
+	original: `
     import { StandardButton, LoadingButton, CustomThemeButton } from '@atlaskit/button';
 
     function App() {
@@ -93,7 +93,7 @@ check({
       </>;
     }
   `,
-  expected: `
+	expected: `
     import { StandardButton, LoadingButton, CustomThemeButton } from '@atlaskit/button';
 
     function App() {
@@ -107,9 +107,9 @@ check({
 });
 
 check({
-  transformer,
-  it: 'should rename data-testid to testId (named imports with alias)',
-  original: `
+	transformer,
+	it: 'should rename data-testid to testId (named imports with alias)',
+	original: `
     import { StandardButton as SB, LoadingButton as LB, CustomThemeButton as CTB } from '@atlaskit/button';
 
     function App() {
@@ -120,7 +120,7 @@ check({
       </>;
     }
   `,
-  expected: `
+	expected: `
     import { StandardButton as SB, LoadingButton as LB, CustomThemeButton as CTB } from '@atlaskit/button';
 
     function App() {
@@ -134,16 +134,16 @@ check({
 });
 
 check({
-  transformer,
-  it: 'should add a warning when both the testId and data-testid props are being used',
-  original: `
+	transformer,
+	it: 'should add a warning when both the testId and data-testid props are being used',
+	original: `
     import StandardButton from '@atlaskit/button';
 
     function App() {
       return <StandardButton testId="oh-my" data-testid="oh-dear">Click me</StandardButton>
     }
   `,
-  expected: `
+	expected: `
     /* TODO: (from codemod) Cannot rename data-testid to testId on StandardButton.
     A StandardButton was detected with both data-testid and testId props.
     Please remove the data-testid prop and check your tests */
@@ -156,9 +156,9 @@ check({
 });
 
 check({
-  transformer,
-  it: 'should not touch unrelated testids',
-  original: `
+	transformer,
+	it: 'should not touch unrelated testids',
+	original: `
     import DSButton from '@atlaskit/button';
     import Something from '@atlaskit/foobar';
 
@@ -170,7 +170,7 @@ check({
       </>;
     }
   `,
-  expected: `
+	expected: `
     import DSButton from '@atlaskit/button';
     import Something from '@atlaskit/foobar';
 

package/codemods/__tests__/_framework.tsx

@@ -7,41 +7,41 @@ const applyTransform = require('jscodeshift/dist/testUtils').applyTransform;
 type Transformer = (file: FileInfo, jscodeshift: API, options: Options) => void;
 
 type TestArgs = {
-  it: string;
-  original: string;
-  expected: string;
-  transformer: Transformer;
-  mode?: 'only' | 'skip' | 'standard';
-  before?: () => void;
-  after?: () => void;
+	it: string;
+	original: string;
+	expected: string;
+	transformer: Transformer;
+	mode?: 'only' | 'skip' | 'standard';
+	before?: () => void;
+	after?: () => void;
 };
 
 export function check({
-  it: name,
-  original,
-  expected,
-  transformer,
-  before = noop,
-  after = noop,
-  mode = 'standard',
+	it: name,
+	original,
+	expected,
+	transformer,
+	before = noop,
+	after = noop,
+	mode = 'standard',
 }: TestArgs) {
-  const run = mode === 'only' ? it.only : mode === 'skip' ? it.skip : it;
+	const run = mode === 'only' ? it.only : mode === 'skip' ? it.skip : it;
 
-  run(name, () => {
-    before();
-    try {
-      const output: string = applyTransform(
-        { default: transformer, parser: 'tsx' },
-        {},
-        { source: original },
-      );
-      expect(output).toBe(expected.trim());
-    } catch (e) {
-      // a failed assertion will throw
-      after();
-      throw e;
-    }
-    // will only be hit if we don't throw
-    after();
-  });
+	run(name, () => {
+		before();
+		try {
+			const output: string = applyTransform(
+				{ default: transformer, parser: 'tsx' },
+				{},
+				{ source: original },
+			);
+			expect(output).toBe(expected.trim());
+		} catch (e) {
+			// a failed assertion will throw
+			after();
+			throw e;
+		}
+		// will only be hit if we don't throw
+		after();
+	});
 }

package/codemods/helpers/15.0.0-runner.tsx

@@ -3,167 +3,165 @@ import type { default as core, FileInfo, JSXElement } from 'jscodeshift';
 import { type Collection } from 'jscodeshift/src/Collection';
 
 import {
-  addToImport,
-  changeImportFor,
-  hasImportDeclaration,
-  removeImport,
-  tryCreateImport,
+	addToImport,
+	changeImportFor,
+	hasImportDeclaration,
+	removeImport,
+	tryCreateImport,
 } from './helpers-generic';
 
 export function convertButtonType({
-  j,
-  base,
-  elements,
-  name,
-  newPackageName,
+	j,
+	base,
+	elements,
+	name,
+	newPackageName,
 }: {
-  j: core.JSCodeshift;
-  base: Collection<any>;
-  elements: NodePath<JSXElement, JSXElement>[];
-  name: string;
-  newPackageName: string;
+	j: core.JSCodeshift;
+	base: Collection<any>;
+	elements: NodePath<JSXElement, JSXElement>[];
+	name: string;
+	newPackageName: string;
 }) {
-  // Don't need to do anything if there are no elements of this type
-  if (!elements.length) {
-    return;
-  }
+	// Don't need to do anything if there are no elements of this type
+	if (!elements.length) {
+		return;
+	}
 
-  tryCreateImport({
-    j,
-    base,
-    relativeToPackage: '@atlaskit/button',
-    packageName: newPackageName,
-  });
+	tryCreateImport({
+		j,
+		base,
+		relativeToPackage: '@atlaskit/button',
+		packageName: newPackageName,
+	});
 
-  addToImport({
-    j,
-    base,
-    importSpecifier: j.importDefaultSpecifier(j.identifier(name)),
-    packageName: newPackageName,
-  });
+	addToImport({
+		j,
+		base,
+		importSpecifier: j.importDefaultSpecifier(j.identifier(name)),
+		packageName: newPackageName,
+	});
 
-  elements.forEach((path: NodePath<JSXElement, JSXElement>) => {
-    path.replace(
-      j.jsxElement(
-        j.jsxOpeningElement(
-          // name: new!
-          j.jsxIdentifier(name),
-          // keep the old attributes
-          path.value.openingElement.attributes,
-          // self closing
-          path.value.openingElement.selfClosing,
-        ),
-        // Only create a closing element if the original had one
-        // <Button /> => <Button /> (rather than <Button></Button>)
-        path.value.closingElement
-          ? j.jsxClosingElement(j.jsxIdentifier(name))
-          : null,
-        path.value.children,
-      ),
-    );
-  });
+	elements.forEach((path: NodePath<JSXElement, JSXElement>) => {
+		path.replace(
+			j.jsxElement(
+				j.jsxOpeningElement(
+					// name: new!
+					j.jsxIdentifier(name),
+					// keep the old attributes
+					path.value.openingElement.attributes,
+					// self closing
+					path.value.openingElement.selfClosing,
+				),
+				// Only create a closing element if the original had one
+				// <Button /> => <Button /> (rather than <Button></Button>)
+				path.value.closingElement ? j.jsxClosingElement(j.jsxIdentifier(name)) : null,
+				path.value.children,
+			),
+		);
+	});
 }
 
 export function changeType({
-  j,
-  base,
-  oldName,
-  newName,
-  fallbackNameAlias,
+	j,
+	base,
+	oldName,
+	newName,
+	fallbackNameAlias,
 }: {
-  j: core.JSCodeshift;
-  base: Collection<any>;
-  oldName: string;
-  newName: string;
-  fallbackNameAlias: string;
+	j: core.JSCodeshift;
+	base: Collection<any>;
+	oldName: string;
+	newName: string;
+	fallbackNameAlias: string;
 }) {
-  changeImportFor({
-    j,
-    base,
-    option: {
-      type: 'change-name',
-      oldName,
-      newName,
-      fallbackNameAlias,
-    },
-    oldPackagePath: '@atlaskit/button',
-    newPackagePath: '@atlaskit/button/types',
-  });
-  changeImportFor({
-    j,
-    base,
-    option: {
-      type: 'change-name',
-      oldName,
-      newName,
-      fallbackNameAlias,
-    },
-    oldPackagePath: '@atlaskit/button/types',
-    newPackagePath: '@atlaskit/button/types',
-  });
+	changeImportFor({
+		j,
+		base,
+		option: {
+			type: 'change-name',
+			oldName,
+			newName,
+			fallbackNameAlias,
+		},
+		oldPackagePath: '@atlaskit/button',
+		newPackagePath: '@atlaskit/button/types',
+	});
+	changeImportFor({
+		j,
+		base,
+		option: {
+			type: 'change-name',
+			oldName,
+			newName,
+			fallbackNameAlias,
+		},
+		oldPackagePath: '@atlaskit/button/types',
+		newPackagePath: '@atlaskit/button/types',
+	});
 }
 
 export function transformButton({
-  j,
-  file,
-  custom: fn,
+	j,
+	file,
+	custom: fn,
 }: {
-  j: core.JSCodeshift;
-  file: FileInfo;
-  custom: (base: Collection<any>) => void;
+	j: core.JSCodeshift;
+	file: FileInfo;
+	custom: (base: Collection<any>) => void;
 }) {
-  const base: Collection<any> = j(file.source);
+	const base: Collection<any> = j(file.source);
 
-  // Exit early if not relevant
-  // We are doing this so we don't touch the formatting of unrelated files
-  const willChange: boolean =
-    hasImportDeclaration(j, file.source, '@atlaskit/button') ||
-    hasImportDeclaration(j, file.source, '@atlaskit/button/types');
+	// Exit early if not relevant
+	// We are doing this so we don't touch the formatting of unrelated files
+	const willChange: boolean =
+		hasImportDeclaration(j, file.source, '@atlaskit/button') ||
+		hasImportDeclaration(j, file.source, '@atlaskit/button/types');
 
-  if (!willChange) {
-    return file.source;
-  }
+	if (!willChange) {
+		return file.source;
+	}
 
-  changeType({
-    j,
-    base,
-    oldName: 'ButtonAppearances',
-    newName: 'Appearance',
-    fallbackNameAlias: 'DSButtonAppearance',
-  });
+	changeType({
+		j,
+		base,
+		oldName: 'ButtonAppearances',
+		newName: 'Appearance',
+		fallbackNameAlias: 'DSButtonAppearance',
+	});
 
-  changeImportFor({
-    j,
-    base,
-    // Not changing the name
-    option: {
-      type: 'keep-name',
-      name: 'Theme',
-      behaviour: 'keep-as-named-import',
-    },
-    oldPackagePath: '@atlaskit/button',
-    newPackagePath: '@atlaskit/button/custom-theme-button',
-  });
+	changeImportFor({
+		j,
+		base,
+		// Not changing the name
+		option: {
+			type: 'keep-name',
+			name: 'Theme',
+			behaviour: 'keep-as-named-import',
+		},
+		oldPackagePath: '@atlaskit/button',
+		newPackagePath: '@atlaskit/button/custom-theme-button',
+	});
 
-  changeImportFor({
-    j,
-    base,
-    option: {
-      type: 'keep-name',
-      name: 'ButtonGroup',
-      behaviour: 'move-to-default-import',
-    },
-    oldPackagePath: '@atlaskit/button',
-    newPackagePath: '@atlaskit/button/button-group',
-  });
+	changeImportFor({
+		j,
+		base,
+		option: {
+			type: 'keep-name',
+			name: 'ButtonGroup',
+			behaviour: 'move-to-default-import',
+		},
+		oldPackagePath: '@atlaskit/button',
+		newPackagePath: '@atlaskit/button/button-group',
+	});
 
-  fn(base);
+	fn(base);
 
-  removeImport({
-    j,
-    base,
-    packageName: '@atlaskit/button',
-  });
+	removeImport({
+		j,
+		base,
+		packageName: '@atlaskit/button',
+	});
 
-  return base.toSource({ quote: 'single' });
+	return base.toSource({ quote: 'single' });
 }