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

Added sample unit test cases #694

Merged

Conversation

shreyakhajanchi
Copy link
Collaborator

@shreyakhajanchi shreyakhajanchi commented Nov 22, 2023

Added sample unit test for add-index and workspace components.

Copy link

codecov bot commented Nov 22, 2023

Codecov Report

Merging #694 (f90ec20) into master (3312b6a) will increase coverage by 0.97%.
Report is 1 commits behind head on master.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #694      +/-   ##
==========================================
+ Coverage   46.33%   47.31%   +0.97%     
==========================================
  Files         120      120              
  Lines       16643    16641       -2     
  Branches      473      471       -2     
==========================================
+ Hits         7712     7874     +162     
+ Misses       8443     8273     -170     
- Partials      488      494       +6     
Components Coverage Δ
backend-apis 43.28% <ø> (ø)
backend-library 55.53% <ø> (ø)
cli 0.00% <ø> (ø)
frontend 31.96% <100.00%> (+4.66%) ⬆️
Files Coverage Δ
...rc/app/components/workspace/workspace.component.ts 82.12% <100.00%> (+57.26%) ⬆️

... and 5 files with indirect coverage changes

@manitgupta
Copy link
Member

Unit tests should be run by Github actions automatically. Add to test-coverage.yaml which currently runs the backend unit tests

@shreyakhajanchi shreyakhajanchi merged commit d131a88 into GoogleCloudPlatform:master Dec 8, 2023
10 checks passed
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.

2 participants