bug(amplify-cli):Solved Windows compatibility issue, made location pa…

packages/amplify-cli/src/__tests__/commands/uninstall.pkg.test.ts

@@ -4,6 +4,8 @@ import execa from 'execa';
 import * as fs from 'fs-extra';
 import { hideSync } from 'hidefile';
 import { setRegPendingDelete } from '../../utils/win-utils';
+import { windowsPathSerializer } from '../testUtils/snapshot-serializer';
+import * as path from 'path';
 
 jest.mock('execa');
 const execa_mock = execa as jest.Mocked<typeof execa>;
@@ -44,6 +46,8 @@ const setRegPendingDelete_mock = setRegPendingDelete as jest.MockedFunction<type
 // save original process.platform
 const originalPlatform = process.platform;
 
+expect.addSnapshotSerializer(windowsPathSerializer);
+
 const context_stub_typed = (context_stub as unknown) as $TSContext;
 
 describe('uninstall packaged CLI on mac / linux', () => {
@@ -60,6 +64,7 @@ describe('uninstall packaged CLI on mac / linux', () => {
       value: originalPlatform,
     });
   });
+
   it('removes the .amplify dir', async () => {
     await run(context_stub_typed);
 
@@ -106,7 +111,7 @@ describe('uninstall packaged CLI on windows', () => {
 
     expect(fs_mock.move.mock.calls[0]).toMatchInlineSnapshot(`
       Array [
-        "homedir/.amplify/bin/amplify.exe",
+        "${path.join('homedir', '.amplify', 'bin', 'amplify.exe')}",
         "a/test/path/.amplify-pending-delete.exe",
         Object {
           "overwrite": true,

packages/amplify-cli/src/__tests__/commands/upgrade.pkg.test.ts

@@ -3,7 +3,8 @@ import { run } from '../../commands/upgrade';
 import fetch, { Response } from 'node-fetch';
 import { $TSContext } from 'amplify-cli-core';
 import * as core from 'amplify-cli-core';
-
+import * as path from 'path';
+import { windowsPathSerializer } from '../testUtils/snapshot-serializer';
 jest.mock('fs-extra');
 const fs_mock = (fs as unknown) as jest.Mocked<typeof fs>;
 
@@ -45,6 +46,7 @@ const context_stub_typed = (context_stub as unknown) as $TSContext;
 
 // save original process.platform
 const originalPlatform = process.platform;
+expect.addSnapshotSerializer(windowsPathSerializer);
 
 describe('run upgrade using packaged CLI', () => {
   beforeEach(() => jest.clearAllMocks());
@@ -53,7 +55,7 @@ describe('run upgrade using packaged CLI', () => {
       value: originalPlatform,
     });
   });
-
+  
   it('exits early if no new packaged version available', async () => {
     // setup
     fetch_mock.mockResolvedValueOnce(({
@@ -98,16 +100,17 @@ describe('run upgrade using packaged CLI', () => {
     // validate
     expect(fs_mock.move.mock.calls[0]).toMatchInlineSnapshot(`
       Array [
-        "homedir/bin/amplify-pkg-linux",
-        "homedir/bin/amplify",
+        "${path.join('homedir', 'bin', 'amplify-pkg-linux')}",
+        "${path.join('homedir', 'bin', 'amplify')}",
         Object {
           "overwrite": true,
         },
       ]
     `);
+
     expect(fs_mock.chmod.mock.calls[0]).toMatchInlineSnapshot(`
       Array [
-        "homedir/bin/amplify",
+        "${path.join('homedir', 'bin', 'amplify')}",
         "700",
       ]
     `);
@@ -149,22 +152,22 @@ describe('run upgrade using packaged CLI', () => {
     // validate
     expect(fs_mock.move.mock.calls[0]).toMatchInlineSnapshot(`
       Array [
-        "homedir/bin/amplify.exe",
-        "homedir/bin/amplify-old.exe",
+        "${path.join('homedir', 'bin', 'amplify.exe')}",
+        "${path.join('homedir', 'bin', 'amplify-old.exe')}",
       ]
     `);
     expect(fs_mock.move.mock.calls[1]).toMatchInlineSnapshot(`
       Array [
-        "homedir/bin/amplify-pkg-win.exe",
-        "homedir/bin/amplify.exe",
+        "${path.join('homedir', 'bin', 'amplify-pkg-win.exe')}",
+        "${path.join('homedir', 'bin', 'amplify.exe')}",
         Object {
           "overwrite": true,
         },
       ]
     `);
     expect(fs_mock.chmod.mock.calls[0]).toMatchInlineSnapshot(`
       Array [
-        "homedir/bin/amplify.exe",
+        "${path.join('homedir', 'bin', 'amplify.exe')}",
         "700",
       ]
     `);

packages/amplify-cli/src/__tests__/testUtils/snapshot-serializer.ts

@@ -0,0 +1,15 @@
+const originalPlatform = process.platform;
+
+// converts inputs to toMatchInlineSnapshot() function into supported format
+export const windowsPathSerializer = {
+  test(val) {
+    return typeof val === 'string' && originalPlatform === 'win32';
+  },
+  print(val) {
+    return `"${(val as string).replace(/\\/g, '\\')}"`;
+  }
+};
+
+test('validateWindowsPathSerializer', () => {
+  expect(windowsPathSerializer).toBeDefined();
+});