Refactor the testController.ts

Open
#1,636 1 comment 0 reactions 1 assignee View on GitHub

@jdneo is already working on this.

Since Dec 7, 2023.

Assessment

This issue has not been assessed yet.

Description

ai-triaged eng enhancement low-priority test-execution

Currently the runTests() of testController.ts is too complicated. What's the meaning of IRunOption and IRunTestContext

Since the debugger's progress reporter api has released for years, we can assume the API is always available and simplify it.

https://github.com/microsoft/vscode-java-test/blob/22a11be0c6074c536e40b7b2e4c8fdb05c8803dd/src/controller/testController.ts#L145

Dominant language
TypeScript
Stars
340
Forks
173
Avg merge
1d 2h
Merged PRs (30d)
17

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from microsoft/vscode-java-test

All issues in microsoft/vscode-java-test

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.