Skip to content

Commit 9a9ec9a

Browse files
firelizzard18hyangah
authored andcommitted
src/goTest: cleanup
Removes redundant code that has been moved to run.ts and resolve.ts. This change was part of CL 344149 but got lost in a merge. Change-Id: I2f771e807eb61e122c63625ff3d10a742449796d Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/348009 Run-TryBot: Hyang-Ah Hana Kim <[email protected]> TryBot-Result: kokoro <[email protected]> Reviewed-by: Hyang-Ah Hana Kim <[email protected]> Trust: Hyang-Ah Hana Kim <[email protected]> Trust: Robert Findley <[email protected]>
1 parent b9bf001 commit 9a9ec9a

File tree

1 file changed

+0
-28
lines changed

1 file changed

+0
-28
lines changed

src/goTest/explore.ts

-28
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import {
1010
TestController,
1111
TestItem,
1212
TestItemCollection,
13-
TestRunProfileKind,
1413
TestRunRequest,
1514
TextDocument,
1615
TextDocumentChangeEvent,
@@ -187,33 +186,6 @@ export class GoTestExplorer {
187186
this.resolver = new GoTestResolver(workspace, ctrl, provideDocumentSymbols);
188187
this.profiler = new GoTestProfiler(this.resolver, workspaceState);
189188
this.runner = new GoTestRunner(workspace, ctrl, this.resolver, this.profiler);
190-
191-
ctrl.resolveHandler = async (item) => {
192-
try {
193-
await this.resolver.resolve(item);
194-
} catch (error) {
195-
if (isInTest()) throw error;
196-
197-
const m = 'Failed to resolve tests';
198-
outputChannel.appendLine(`${m}: ${error}`);
199-
await vscode.window.showErrorMessage(m);
200-
}
201-
};
202-
203-
ctrl.createRunProfile(
204-
'go test',
205-
TestRunProfileKind.Run,
206-
async (request, token) => {
207-
try {
208-
await this.runner.run(request, token);
209-
} catch (error) {
210-
const m = 'Failed to execute tests';
211-
outputChannel.appendLine(`${m}: ${error}`);
212-
await vscode.window.showErrorMessage(m);
213-
}
214-
},
215-
true
216-
);
217189
}
218190

219191
/* ***** Listeners ***** */

0 commit comments

Comments
 (0)