codecov[bot]
commented
1 year ago Codecov Report
Merging #360 (2e880cd) into main (a552487) will decrease coverage by
2.83%. The diff coverage is56.91%.
@@ Coverage Diff @@
## main #360 +/- ##
==========================================
- Coverage 90.29% 87.46% -2.83%
==========================================
Files 50 52 +2
Lines 1422 1516 +94
Branches 357 381 +24
==========================================
+ Hits 1284 1326 +42
- Misses 129 180 +51
- Partials 9 10 +1 | Impacted Files | Coverage Δ | |
|---|---|---|
| src/components/Header/UserAccountMenu.jsx | 0.00% <0.00%> (ø) |
|
| src/components/Login/TermsOfService.jsx | 0.00% <ø> (-100.00%) |
:arrow_down: |
| src/components/Login/ThirdPartyLoginComponents.jsx | 5.88% <5.88%> (ø) |
|
| src/components/Header/Header.jsx | 70.00% <50.00%> (-5.00%) |
:arrow_down: |
| src/utilities/authUtilities.js | 51.85% <50.00%> (-48.15%) |
:arrow_down: |
| src/components/Login/SignIn.jsx | 87.06% <77.55%> (-9.23%) |
:arrow_down: |
| src/App.js | 100.00% <100.00%> (ø) |
|
| src/api.js | 70.00% <100.00%> (+9.47%) |
:arrow_up: |
| src/components/Login/SignInPresentation.jsx | 100.00% <100.00%> (ø) |
|
| src/pages/LoginPage.jsx | 100.00% <100.00%> (ø) |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
What type of PR is this? feature
Closes #355 Closes #351 Closes #359 Which issue does this PR fix:
What does this PR do / Why do we need it: Implements the openid authentication flow (needs project-zot/zot#1381 to be merged) Update signin page ui design
If an issue # is not available please add repro steps and logs from IPAMD/CNI showing the issue:
Testing done on this change:
Automation added to e2e:
Will this break upgrades or downgrades. Has updating a running cluster been tested?:
Does this change require updates to the CNI daemonset config files to work?:
Does this PR introduce any user-facing change?:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.