add unit test for remote collection

extension/src/setup/collect.test.ts

@@ -0,0 +1,30 @@
+import { collectRemoteList } from './collect'
+import { dvcDemoPath } from '../test/util'
+import { join } from '../test/util/path'
+
+describe('collectRemoteList', () => {
+  it('should return the expected data structure', async () => {
+    const mockedRoot = join('some', 'other', 'root')
+    const remoteList = await collectRemoteList(
+      [dvcDemoPath, 'mockedOtherRoot', mockedRoot],
+      root =>
+        Promise.resolve(
+          {
+            [dvcDemoPath]:
+              'storage s3://dvc-public/remote/mnist-vscode\nbackup gdrive://appDataDir\nurl  https://remote.dvc.org/mnist-vscode',
+            mockedOtherRoot: '',
+            [mockedRoot]: undefined
+          }[root]
+        )
+    )
+    expect(remoteList).toStrictEqual({
+      [dvcDemoPath]: {
+        backup: 'gdrive://appDataDir',
+        storage: 's3://dvc-public/remote/mnist-vscode',
+        url: 'https://remote.dvc.org/mnist-vscode'
+      },
+      mockedOtherRoot: undefined,
+      [mockedRoot]: undefined
+    })
+  })
+})

extension/src/setup/collect.ts

@@ -25,8 +25,8 @@ export const collectSectionCollapsed = (
 export const collectRemoteList = async (
   dvcRoots: string[],
   getRemoteList: (cwd: string) => Promise<string | undefined>
-): Promise<RemoteList> => {
-  const acc: { [dvcRoot: string]: { [alias: string]: string } | undefined } = {}
+): Promise<NonNullable<RemoteList>> => {
+  const acc: NonNullable<RemoteList> = {}
 
   for (const dvcRoot of dvcRoots) {
     const remoteList = await getRemoteList(dvcRoot)