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 additional tests for CA implementations and OIDC #602

Merged
merged 2 commits into from May 20, 2022

Conversation

haydentherapper
Copy link
Contributor

Signed-off-by: Hayden Blauzvern hblauzvern@google.com

Summary

Ticket Link

Fixes

Release Note


@codecov-commenter
Copy link

codecov-commenter commented May 20, 2022

Codecov Report

Merging #602 (ac6da4a) into main (00d7e66) will increase coverage by 2.96%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #602      +/-   ##
==========================================
+ Coverage   49.74%   52.70%   +2.96%     
==========================================
  Files          21       22       +1     
  Lines        1548     1588      +40     
==========================================
+ Hits          770      837      +67     
+ Misses        705      670      -35     
- Partials       73       81       +8     
Impacted Files Coverage Δ
pkg/ca/ephemeralca/ephemeral.go 70.00% <0.00%> (ø)
pkg/ca/fileca/load.go 68.96% <0.00%> (+10.34%) ⬆️
pkg/oauthflow/oidc.go 100.00% <0.00%> (+50.00%) ⬆️
pkg/ca/x509ca/common.go 60.52% <0.00%> (+60.52%) ⬆️

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 00d7e66...ac6da4a. Read the comment docs.

Signed-off-by: Hayden Blauzvern <hblauzvern@google.com>
Copy link
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

lgtm

bobcallaway
bobcallaway previously approved these changes May 20, 2022
Copy link
Member

@bobcallaway bobcallaway left a comment

Choose a reason for hiding this comment

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

lgtm minus one nit

pkg/ca/ephemeralca/ephemeral_test.go Outdated Show resolved Hide resolved
dlorenc
dlorenc previously approved these changes May 20, 2022
Signed-off-by: Hayden Blauzvern <hblauzvern@google.com>
@haydentherapper haydentherapper dismissed stale reviews from dlorenc and bobcallaway via 181f7e7 May 20, 2022 16:20
@haydentherapper
Copy link
Contributor Author

Review dismissal is far more aggressive now. I thought it had some tolerance for small nits...

@dlorenc
Copy link
Member

dlorenc commented May 20, 2022

Review dismissal is far more aggressive now. I thought it had some tolerance for small nits...

Allstarbot yelled at me so I had to change the setting to dismiss everything :(

@dlorenc dlorenc merged commit 37f4aa7 into sigstore:main May 20, 2022
@haydentherapper
Copy link
Contributor Author

It's not too much of a nuisance, I just wish there was some small tolerance for single line/few character changes.

@haydentherapper haydentherapper deleted the tests branch May 20, 2022 16:42
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.

None yet

5 participants