Skip to content

Conversation

@baguilar6174
Copy link
Contributor

Unit test coverage is complete. Currently the coverage is 99.35%.

@codecov
Copy link

codecov bot commented Jan 28, 2022

Codecov Report

Merging #800 (2feb469) into master (95d1ed3) will increase coverage by 1.76%.
The diff coverage is 95.81%.

❗ Current head 2feb469 differs from pull request most recent head 2306e22. Consider uploading reports for the commit 2306e22 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##           master     #800      +/-   ##
==========================================
+ Coverage   95.64%   97.40%   +1.76%     
==========================================
  Files         107      107              
  Lines        2273     2428     +155     
  Branches      175      203      +28     
==========================================
+ Hits         2174     2365     +191     
+ Misses         54       23      -31     
+ Partials       45       40       -5     
Impacted Files Coverage Δ
...er-management/store/customer-management.effects.ts 100.00% <ø> (ø)
src/app/modules/time-clock/store/entry.effects.ts 98.14% <ø> (+1.85%) ⬆️
src/environments/environment.ts 100.00% <ø> (ø)
...eate-project-type/create-project-type.component.ts 96.96% <50.00%> (-3.04%) ⬇️
...es/shared/interceptors/inject.token.interceptor.ts 87.50% <66.66%> (-12.50%) ⬇️
...c/app/modules/time-clock/services/entry.service.ts 96.42% <75.00%> (-3.58%) ⬇️
...p/modules/time-clock/pages/time-clock.component.ts 96.42% <83.33%> (+4.76%) ⬆️
...ts/projects/components/services/project.service.ts 94.11% <85.71%> (+3.20%) ⬆️
...shared/components/dark-mode/dark-mode.component.ts 90.90% <90.90%> (-4.55%) ⬇️
...project-list-hover/project-list-hover.component.ts 90.90% <93.33%> (+0.16%) ⬆️
... and 43 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5d29b8e...2306e22. Read the comment docs.

Copy link
Contributor

@epulla epulla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please do not forget that the tests are written using "it", so if there is a verb, you should write the tests as "it calls..." or "it uses...".

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@baguilar6174 baguilar6174 merged commit d283ed7 into master Jan 28, 2022
@baguilar6174 baguilar6174 deleted the TT-281-UI-Test-Coverage-Second-Part branch January 28, 2022 20:11
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.

4 participants