self review

iterative · Apr 27, 2023 · a097c8b · a097c8b
1 parent 408a94c
commit a097c8b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/extension/src/test/suite/setup/util.ts b/extension/src/test/suite/setup/util.ts
@@ -33,7 +33,6 @@ export const buildSetup = (
     messageSpy,
     resourceLocator,
     internalCommands,
-    dvcExecutor,
     dvcReader,
     gitExecutor,
     gitReader
@@ -99,7 +98,6 @@ export const buildSetup = (
 
   return {
     config,
-    dvcExecutor,
     internalCommands,
     messageSpy,
     mockAutoInstallDvc,

diff --git a/extension/src/util/appdirs.test.ts b/extension/src/util/appdirs.test.ts
@@ -2,14 +2,14 @@ import { join } from 'path'
 import { getDVCAppDir, getIterativeAppDir } from './appdirs'
 import { getProcessPlatform } from '../env'
 
-const mockedUserConfigDir = require('appdirs').userConfigDir
-const mockedGetProcessPlatform = jest.mocked(getProcessPlatform)
-const mockedJoin = jest.mocked(join)
-
 jest.mock('appdirs', () => ({ userConfigDir: jest.fn() }))
 jest.mock('../env')
 jest.mock('path')
 
+const mockedUserConfigDir = require('appdirs').userConfigDir
+const mockedGetProcessPlatform = jest.mocked(getProcessPlatform)
+const mockedJoin = jest.mocked(join)
+
 beforeEach(() => {
   jest.resetAllMocks()
 })