Skip to content

Enable the Test Activity in VS Code #4282

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

Merged
merged 3 commits into from
Feb 5, 2019

Conversation

d3r3kk
Copy link

@d3r3kk d3r3kk commented Feb 5, 2019

For #4272

This is a preliminary checkin to get our work rolling on the Test Explorer. We should be able to parallelize the work after this initial (and basic) check-in.

  • Adds a single view
  • Adds a single command (no icon)
  • Adds dummy tree view data

  • Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
  • Title summarizes what is changing
  • Has a news entry file (remember to thank yourself!)
  • Has sufficient logging.
  • Has telemetry for enhancements.
  • Unit tests & system/integration tests are added/updated
  • Test plan is updated as appropriate
  • package-lock.json has been regenerated by running npm install (if dependencies have changed)

For microsoft#4272

- Enables the Test Activity in VS Code
- Adds a single view
- Adds a single command (no icon)
- Adds dummy tree view data
@d3r3kk d3r3kk added the no-changelog No news entry required label Feb 5, 2019
@codecov
Copy link

codecov bot commented Feb 5, 2019

Codecov Report

Merging #4282 into master will increase coverage by 23%.
The diff coverage is 87%.

@@           Coverage Diff            @@
##           master   #4282     +/-   ##
========================================
+ Coverage      57%     79%    +23%     
========================================
  Files         337     414     +77     
  Lines       14284   19106   +4822     
  Branches     2224    3086    +862     
========================================
+ Hits         8085   15084   +6999     
+ Misses       5677    4019   -1658     
+ Partials      522       3    -519
Flag Coverage Δ
#Linux 69% <87%> (?)
#Windows 69% <87%> (?)
#macOS 68% <87%> (?)

@d3r3kk d3r3kk removed the no-changelog No news entry required label Feb 5, 2019
@d3r3kk d3r3kk merged commit 9187479 into microsoft:master Feb 5, 2019
@d3r3kk d3r3kk deleted the 4272_enable_test_explorer_view branch February 5, 2019 23:27
@lock lock bot locked as resolved and limited conversation to collaborators Jul 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants