fixes validate command error

Author: Daniel Del Core

community/@atlaskit__textfield/src/5.0.0/__tests__/remove-imports.spec.ts

@@ -15,10 +15,6 @@ function transformer(
   return source.toSource(options.printOptions);
 }
 
-const importToDoComment = `
-/* TODO: (@hypermod) This file uses exports used to help theme @atlaskit/textfield which
-    has now been removed due to its poor performance characteristics. */`;
-
 describe('Remove imports', () => {
   it('should remove theme imports from Textfield and leave a comment', async () => {
     const result = await applyTransform(

community/@atlaskit__textfield/src/5.0.0/__tests__/remove-prop.spec.ts

@@ -15,12 +15,6 @@ function transformer(
   return source.toSource(options.printOptions);
 }
 
-const themeToDoComment = `
-/* TODO: (@hypermod) This file uses the @atlaskit/textfield \`theme\` prop which
-    has now been removed due to its poor performance characteristics. We have not replaced
-    theme with an equivalent API due to minimal usage of the \`theme\` prop.
-    The appearance of TextField will have likely changed. */`;
-
 describe('Remove prop', () => {
   it('should remove theme from Textfield and leave a comment', async () => {
     const result = await applyTransform(

community/hypermod/src/defineInlineTest-to-applyTransform/transform.ts

@@ -1,6 +1,6 @@
 import core, { API, FileInfo, Options, Collection } from 'jscodeshift';
 
-import { hasImportDeclaration, hasImportSpecifier } from '@hypermod/utils';
+import { hasImportSpecifier } from '@hypermod/utils';
 
 export default function transformer(
   file: FileInfo,
@@ -102,8 +102,7 @@ function transformDefineInlineTestCalls(
           path.node.callee.property.name === 'defineInlineTest'),
     )
     .forEach(path => {
-      const [transformerOpts, _configOpts, input, output, description] =
-        path.node.arguments;
+      const [transformerOpts, input, output, description] = path.node.arguments;
 
       const transformer = j(transformerOpts).find(j.ObjectProperty, {
         key: { name: 'default' },

packages/fetcher/src/index.spec.ts

@@ -42,10 +42,10 @@ describe('fetcher', () => {
         { virtual: true },
       );
 
-      const { filePath, config } = await fetchConfig(mockBasePath);
+      const configMeta = await fetchConfig(mockBasePath);
 
-      expect(config).toEqual(mockConfig);
-      expect(filePath).toEqual(mockFilePath);
+      expect(configMeta!.config).toEqual(mockConfig);
+      expect(configMeta!.filePath).toEqual(mockFilePath);
     });
 
     it('fetches config with named export', async () => {
@@ -57,10 +57,12 @@ describe('fetcher', () => {
         },
       );
 
-      const { filePath, config } = await fetchConfig(mockBasePath);
+      const configMeta = await fetchConfig(mockBasePath);
 
-      expect(config).toEqual(mockConfig);
-      expect(filePath).toEqual(path.join(mockBasePath, 'hypermod.config.js'));
+      expect(configMeta!.config).toEqual(mockConfig);
+      expect(configMeta!.filePath).toEqual(
+        path.join(mockBasePath, 'hypermod.config.js'),
+      );
     });
 
     it('fetches first matched config when multiple are found', async () => {
@@ -81,10 +83,10 @@ describe('fetcher', () => {
         path.join(mockBasePath, 'codemods', 'hypermod.config.tsx'),
       ];
 
-      const { config, filePath } = await fetchConfig(mockBasePath);
+      const configMeta = await fetchConfig(mockBasePath);
 
-      expect(config).toEqual(mockConfig);
-      expect(filePath).toEqual(
+      expect(configMeta!.config).toEqual(mockConfig);
+      expect(configMeta!.filePath).toEqual(
         path.join(mockBasePath, 'src', 'hypermod.config.ts'),
       );
     });

packages/fetcher/src/index.ts

@@ -28,7 +28,9 @@ function requireConfig(filePath: string, resolvedPath: string) {
   }
 }
 
-export async function fetchConfig(filePath: string): Promise<ConfigMeta> {
+export async function fetchConfig(
+  filePath: string,
+): Promise<ConfigMeta | undefined> {
   const configs = await fetchConfigs(filePath);
   return configs[0];
 }
@@ -94,7 +96,7 @@ export async function fetchPackage(
 export async function fetchRemotePackage(
   packageName: string,
   packageManager: PluginManager,
-): Promise<ConfigMeta> {
+): Promise<ConfigMeta | undefined> {
   if (['javascript', 'typescript'].includes(packageName)) {
     throw new Error(`'${packageName}' is ignored as a remote package.`);
   }

packages/validator/src/index.spec.ts

@@ -134,6 +134,16 @@ describe('validator', () => {
       );
     });
 
+    it('should error if config file is not found', async () => {
+      const filePath = 'path/to/';
+
+      (fetchConfig as jest.Mock).mockResolvedValue(undefined);
+
+      await expect(isValidConfigAtPath(filePath)).rejects.toThrowError(
+        `Unable to locate config file at path: ${filePath}`,
+      );
+    });
+
     it('should error if config contains multiple invalid properties', async () => {
       (fetchConfig as jest.Mock).mockResolvedValue({
         location: 'path/to/hypermod.config.js',

packages/validator/src/index.ts

@@ -38,25 +38,25 @@ export function isValidConfig(config: CodeshiftConfig) {
 }
 
 export async function isValidConfigAtPath(filePath: string) {
-  const { config } = await fetchConfig(filePath);
+  const configMeta = await fetchConfig(filePath);
 
-  if (!config) {
+  if (!configMeta) {
     throw new Error(`Unable to locate config file at path: ${filePath}`);
   }
 
-  const invalidProperites = getInvalidProperties(config);
+  const invalidProperites = getInvalidProperties(configMeta.config);
   if (invalidProperites.length) {
     throw new Error(
       `Invalid transform ids found: ${invalidProperites.join(', ')}`,
     );
   }
 
-  if (!hasValidTransforms(config)) {
+  if (!hasValidTransforms(configMeta.config)) {
     throw new Error(`Invalid transform ids found for config at "${filePath}".
 Please make sure all transforms are identified by a valid semver version. ie 10.0.0`);
   }
 
-  if (!hasValidPresets(config)) {
+  if (!hasValidPresets(configMeta.config)) {
     throw new Error(`Invalid preset ids found for config at "${filePath}".
 Please make sure all presets are kebab case and contain no spaces or special characters. ie sort-imports-by-scope`);
   }