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

imagehistory tests #50

Merged
merged 8 commits into from
Dec 20, 2024
Merged

imagehistory tests #50

merged 8 commits into from
Dec 20, 2024

Conversation

amritakohli
Copy link
Contributor

@amritakohli amritakohli commented Dec 20, 2024

Added tests for image history.

Test cases:

  • Adding an entry to image history
  • Adding another entry to image history with a different uuid
  • Deep copying config
  • Modifying config
  • Populate / redact functions

Moved redactedString to a constant so it can be reused in the tests.
Added another file in additional dirs for good measure (I found that it helped while locally testing image history).
Added a config for image history that contains several different fields.

Possible future tests:
Test custom MarshalJSON functions that were added for image history
Test omitting JSON fields as selected


Checklist

  • Tests added/updated
  • Documentation updated (if needed)
  • Code conforms to style guidelines

@amritakohli amritakohli requested a review from a team as a code owner December 20, 2024 18:40
@amritakohli amritakohli force-pushed the user/amritakohli/imagehistorytest branch from 4443184 to 652620c Compare December 20, 2024 22:16
cwize1
cwize1 previously approved these changes Dec 20, 2024
@amritakohli amritakohli merged commit 7d539f4 into main Dec 20, 2024
7 checks passed
@amritakohli amritakohli deleted the user/amritakohli/imagehistorytest branch December 20, 2024 22:39
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.

2 participants