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

Add unit test covering .dockerconfigjson secrets #1335

Merged
merged 3 commits into from
Jul 13, 2022

Conversation

imjasonh
Copy link
Collaborator

@imjasonh imjasonh commented Apr 7, 2022

This wasn't previously covered, and better tests seem like they might help us narrow down the cause of knative/serving#12761

cc @dprotaso

@codecov-commenter
Copy link

codecov-commenter commented Apr 7, 2022

Codecov Report

Merging #1335 (9d95667) into main (ddd39fb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1335   +/-   ##
=======================================
  Coverage   74.16%   74.16%           
=======================================
  Files         114      114           
  Lines        8459     8459           
=======================================
  Hits         6274     6274           
  Misses       1579     1579           
  Partials      606      606           

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 ddd39fb...9d95667. Read the comment docs.

@github-actions
Copy link

This Pull Request is stale because it has been open for 90 days with
no activity. It will automatically close after 30 more days of
inactivity. Keep fresh with the 'lifecycle/frozen' label.

@imjasonh imjasonh merged commit b7b4ead into google:main Jul 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants