Ensure that file exists before trying to read mtime (#3299)

Author: mattseddon

extension/src/fileSystem/index.test.ts

@@ -172,6 +172,12 @@ describe('getModifiedTime', () => {
     expect(typeof epoch).toBe('number')
     expect(epoch).toBeGreaterThan(1640995200000)
   })
+
+  it('should return -1 for a file that does not exist on the system', () => {
+    const epoch = getModifiedTime('not a path')
+
+    expect(epoch).toStrictEqual(-1)
+  })
 })
 
 describe('findOrCreateDvcYamlFile', () => {

extension/src/fileSystem/index.ts

@@ -36,8 +36,12 @@ export const isDirectory = (path: string): boolean =>
 
 export const isFile = (path: string): boolean => checkStats(path, 'isFile')
 
-export const getModifiedTime = (path: string): number =>
-  lstatSync(path).mtime.getTime()
+export const getModifiedTime = (path: string): number => {
+  if (exists(path)) {
+    return lstatSync(path).mtime.getTime()
+  }
+  return -1
+}
 
 export const findSubRootPaths = async (
   cwd: string,