Skip to content

Commit 764d9be

Browse files
authored
🪳 Fixed refesh handler no longer working with multi root workspace (#181)
1 parent ac2a928 commit 764d9be

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

‎src/pesterTestController.ts

+8-5
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ import {
4343
import { clear, findTestItem, forAll, getTestItems, getUniqueTestItems, isTestItemOptions } from './util/testItemUtils'
4444
import debounce = require('debounce-promise')
4545
import { isDeepStrictEqual } from 'util'
46-
import { getPesterExtensionContext } from './extension'
46+
import { registerDisposable as registerExtensionDisposable, getPesterExtensionContext } from './extension'
4747
import { watchWorkspaceFolder } from './workspaceWatcher'
4848

4949
const defaultControllerLabel = 'Pester'
@@ -329,11 +329,13 @@ export class PesterTestController implements Disposable {
329329
this.log.info("VSCode requested a refresh. Re-initializing the Pester Tests extension")
330330
this.stopPowerShell()
331331
clear(this.testController.items)
332+
this.testFileWatchers.forEach(watcher => {
333+
watcher.dispose()
334+
this.disposables.splice(this.disposables.indexOf(watcher), 1)
335+
})
332336

333-
// TODO: Check settings and refresh if changed
334-
335-
// Reinitialize the monitor which will restart the FileSystemWatchers
336-
this.resolveHandler(undefined, cancelToken)
337+
this.testFileWatchers = new Map<RelativePattern, FileSystemWatcher>()
338+
this.watch()
337339
}
338340

339341
/**
@@ -966,3 +968,4 @@ export class PesterTestController implements Disposable {
966968
this.log.warn(`Test Run Cancelled`)
967969
}
968970
}
971+

0 commit comments

Comments
 (0)