fix(migrate): add validation for path after file://

Author: rjabhi

packages/amplify-migration/src/commands/gen2/execute/execute_command.test.ts

@@ -52,6 +52,17 @@ describe('Gen2ExecuteCommand', () => {
     );
   });
 
+  it('should fail command when resourceMappings only has file://', async () => {
+    const parser = yargs().command(new Gen2ExecuteCommand() as unknown as CommandModule);
+    await assert.rejects(
+      () => runCommandAsync(parser, 'execute --from foo --to bar --resourceMappings file://'),
+      (err: Error) => {
+        assert.equal(err.message, 'Expected resourceMap to have a path after file://');
+        return true;
+      },
+    );
+  });
+
   it('should fail command when resourceMappings is invalid JSON', async () => {
     stubReadFile.mockResolvedValue('invalid json');
     const parser = yargs().command(new Gen2ExecuteCommand() as unknown as CommandModule);

packages/amplify-migration/src/commands/gen2/execute/execute_command.ts

@@ -74,6 +74,9 @@ export class Gen2ExecuteCommand implements CommandModule<object, ExecuteCommandO
         throw new Error(`Expected resourceMap to start with ${FILE_PROTOCOL_PREFIX}`);
       }
       const resourceMapPath = resourceMappings.split(FILE_PROTOCOL_PREFIX)[1];
+      if (!resourceMapPath) {
+        throw new Error(`Expected resourceMap to have a path after ${FILE_PROTOCOL_PREFIX}`);
+      }
       const resourceMappingsFromFile = await fs.readFile(resourceMapPath, { encoding: 'utf-8' });
       try {
         parsedResourceMappings = JSON.parse(resourceMappingsFromFile);