Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expose parse result #773

Merged
merged 6 commits into from
Sep 29, 2021

Conversation

connectdotz
Copy link
Collaborator

@connectdotz connectdotz commented Sep 14, 2021

expose test blocks parse results to TestExplorer when match result is not available or match failed.

motivation

  • Make the TestExplorer status menu always accessible regardless of whether tests have been run. More info see comment.

  • enable us to retire debug codeLens in the future: to simplify the code base and reduce editor interference (Codelense toggles while editing test case #772)


fixes #760

vscode-jest-4.2.0-rc.1.vsix.zip

@coveralls
Copy link

Pull Request Test Coverage Report for Build 1231953453

  • 29 of 29 (100.0%) changed or added relevant lines in 4 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.05%) to 94.288%

Files with Coverage Reduction New Missed Lines %
src/test-provider/test-item-data.ts 1 94.97%
Totals Coverage Status
Change from base Build 1226917357: 0.05%
Covered Lines: 2463
Relevant Lines: 2562

💛 - Coveralls

@connectdotz
Copy link
Collaborator Author

connectdotz commented Sep 29, 2021

@stephtr looks like you are busy, I will merge this but won't deprecate debug codeLens, we can decide what to do with it later...

@connectdotz connectdotz merged commit cebc7e8 into jest-community:master Sep 29, 2021
@connectdotz connectdotz deleted the expose-parse-result branch September 29, 2021 20:47
legend1202 pushed a commit to legend1202/vscode-jest that referenced this pull request Jun 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add inline button to run test
2 participants