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

initial performance test #10025

Merged
merged 11 commits into from
May 17, 2022
Merged

initial performance test #10025

merged 11 commits into from
May 17, 2022

Conversation

amunger
Copy link
Contributor

@amunger amunger commented May 13, 2022

fixes #7437

A single perf test that can be run as a launch configuration, or through npm run testPerformance
It specifies the entire filename for Mocha's filename suffix option so that we can only run one test per launch of vscode. That way we can test perf from start to finish and isolate each test.

@amunger amunger changed the title Dev/aamunger/perf test2 initial performance test May 13, 2022
@amunger amunger force-pushed the dev/aamunger/perfTest2 branch from a0433a6 to bc4dce5 Compare May 13, 2022 19:04
@amunger amunger marked this pull request as ready for review May 13, 2022 19:05
@amunger amunger requested a review from a team as a code owner May 13, 2022 19:05
@codecov-commenter
Copy link

codecov-commenter commented May 13, 2022

Codecov Report

Merging #10025 (0b2bfe3) into main (580dd68) will increase coverage by 0%.
The diff coverage is n/a.

❗ Current head 0b2bfe3 differs from pull request most recent head 53a9fa4. Consider uploading reports for the commit 53a9fa4 to get more accurate results

@@          Coverage Diff           @@
##            main   #10025   +/-   ##
======================================
  Coverage     63%      63%           
======================================
  Files        211      216    +5     
  Lines      10090    10130   +40     
  Branches    1610     1613    +3     
======================================
+ Hits        6442     6479   +37     
  Misses      3131     3131           
- Partials     517      520    +3     
Impacted Files Coverage Δ
src/platform/common/utils/misc.ts 69% <0%> (-3%) ⬇️
src/platform/api/kernelApi.ts 71% <0%> (-2%) ⬇️
src/platform/common/platform/fileUtils.node.ts 50% <0%> (-2%) ⬇️
src/platform/serviceRegistry.node.ts 97% <0%> (-1%) ⬇️
src/platform/export/types.ts 100% <0%> (ø)
src/platform/common/platform/types.ts 100% <0%> (ø)
src/platform/common/platform/fs-paths.ts 84% <0%> (ø)
src/platform/debugger/jupyter/debugger.node.ts
...platform/debugger/jupyter/debuggingManager.node.ts
...atform/debugger/jupyter/kernelDebugAdapter.node.ts
... and 21 more

src/test/index.node.ts Outdated Show resolved Hide resolved
@amunger amunger merged commit c100f73 into main May 17, 2022
@amunger amunger deleted the dev/aamunger/perfTest2 branch May 17, 2022 16:33
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.

Write automated tests to test and ensure expected performance characteristics
4 participants