feat: testing no import statement

Author: YuriSS

__tests__/optimize-imports-mocks.ts

@@ -111,3 +111,8 @@ import { MatDialogRef } from '@angular/material/dialog';
 import { Observable } from 'rxjs';
 import { AboutDialogBloc, AboutState } from './about-dialog.bloc';`,
 };
+
+export const noImportStatement: TestCase = {
+  input: `const x = 2;`,
+  expected: `const x = 2;`,
+};

__tests__/optimize-imports.spec.ts

@@ -2,7 +2,7 @@ import { actions, organizeImports, organizeImportsForFile } from '@ic/conductor/
 import * as config from '@ic/config';
 import fs from 'fs';
 import { Config } from '@ic/types';
-import { readmeExample, comments, TestCase, codeBetweenImports, emptyNewLineSeparator } from './optimize-imports-mocks';
+import { readmeExample, comments, TestCase, codeBetweenImports, emptyNewLineSeparator, noImportStatement } from './optimize-imports-mocks';
 import { defaultConfig } from '@ic/defaultConfig';
 
 jest.mock('fs');
@@ -72,4 +72,12 @@ describe('optimizeImports', () => {
       expect(fs.writeFileSync).not.toHaveBeenCalled();
     }
   });
+
+  it('should do nothing if the file has no import', async () => {
+    (fs.readFileSync as any).mockReturnValue(Buffer.from(noImportStatement.input));
+    const file = 'test.ts';
+    const result = await organizeImportsForFile(file);
+    expect(result).toBe(actions.none);
+    expect(fs.writeFileSync).not.toHaveBeenCalled();
+  });
 });

src/conductor/organize-imports.ts

@@ -45,7 +45,7 @@ export async function organizeImportsForFile(filePath: string): Promise<string>
   const { staged, autoAdd, dryRun } = getConfig();
   const fileWithOrganizedImports = await organizeImports(fileContent);
   const fileHasChanged = fileWithOrganizedImports !== fileContent;
-  const isValidAction = [actions.none, actions.skipped].every(action => action !== fileWithOrganizedImports);
+  const isValidAction = [actions.none, actions.skipped].every((action) => action !== fileWithOrganizedImports);
 
   if (fileHasChanged && isValidAction) {
     !dryRun && writeFileSync(filePath, fileWithOrganizedImports);
@@ -55,11 +55,11 @@ export async function organizeImportsForFile(filePath: string): Promise<string>
       msg += ', added to git';
     }
     log('green', msg, filePath);
-  } else {
-    log('gray', 'no change needed', filePath);
+    return actions.reordered;
   }
 
-  return fileHasChanged ? actions.reordered : actions.none;
+  log('gray', 'no change needed', filePath);
+  return actions.none;
 }
 
 export async function organizeImports(fileContent: string): Promise<string> {