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

Expand tests #150

Merged
merged 4 commits into from
Feb 17, 2023
Merged

Expand tests #150

merged 4 commits into from
Feb 17, 2023

Conversation

n8henrie
Copy link
Collaborator

This PR

  • adopts the tests for linux to use the new-style tests
  • adds the CLI to the integration tests
  • adds an example CLI test for the --rekey function

Let me know what you think about this approach -- I'm kind of excited. If there are no major concerns I'll be happy to chip away at adding a few additional tests and seeing whether I can sort out the darwin tests as well.

This test copies the example `secrets.nix` and age files and uses the
user key to rekey them. It compares the hash before and after to ensure
that the age file is actually being changed.
@n8henrie n8henrie requested a review from ryantm February 16, 2023 21:23
Copy link
Owner

@ryantm ryantm left a comment

Choose a reason for hiding this comment

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

looks good!

@ryantm ryantm merged commit de65706 into ryantm:main Feb 17, 2023
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

2 participants