Skip to content

Conversation

PaulRC-ioet
Copy link
Contributor

Remove role field from User model and update the tests.

@codecov
Copy link

codecov bot commented Jan 28, 2021

Codecov Report

Merging #633 (f8fd270) into master (13de42e) will increase coverage by 1.09%.
The diff coverage is 99.22%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #633      +/-   ##
==========================================
+ Coverage   93.09%   94.18%   +1.09%     
==========================================
  Files          85       95      +10     
  Lines        1564     1772     +208     
  Branches      107      116       +9     
==========================================
+ Hits         1456     1669     +213     
+ Misses         67       63       -4     
+ Partials       41       40       -1     
Impacted Files Coverage Δ
src/app/app-routing.module.ts 100.00% <ø> (ø)
...nents/time-range-form/time-range-form.component.ts 100.00% <ø> (ø)
...project-list-hover/project-list-hover.component.ts 88.37% <ø> (ø)
...dules/time-entries/pages/time-entries.component.ts 84.28% <92.85%> (+1.47%) ⬆️
src/app/modules/users/store/user.reducers.ts 94.73% <94.73%> (ø)
src/app/guards/login-guard/login.guard.ts 100.00% <100.00%> (ø)
...nologies-report-guard/technologies-report.guard.ts 100.00% <100.00%> (ø)
src/app/modules/login/login.component.ts 100.00% <100.00%> (+16.66%) ⬆️
...app/modules/login/services/azure.ad.b2c.service.ts 86.95% <100.00%> (+4.60%) ⬆️
...ponents/details-fields/details-fields.component.ts 93.18% <100.00%> (+6.00%) ⬆️
... and 33 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 917e900...f8fd270. Read the comment docs.

@PaulRC-ioet PaulRC-ioet merged commit a2802a1 into master Jan 28, 2021
@PaulRC-ioet PaulRC-ioet deleted the TT-142-RemoveRoleField branch January 28, 2021 17:58
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.

3 participants