-
Notifications
You must be signed in to change notification settings - Fork 281
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
Deployment profile testing with BATS #4991
Conversation
050187b
to
4a32171
Compare
2b7b86e
to
d9c74b1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated the PR with some of the suggested changes; see replies for the rest.
I also had to update the start_app
function in the sample test; not sure why this stopped working. The change is similar to something I did in #5108.
We can later combine it, but once #5104 is merged this may be further simplified again.
8bdbbd7
to
1eec7f8
Compare
Signed-off-by: Jan Dubois <[email protected]>
Signed-off-by: Jan Dubois <[email protected]>
"Declare and assign separately to avoid masking return values" Instead of local foo=$(bar) we need to write local foo foo=$(bar) to make sure we throw an error when `bar` returns a non-zero status Signed-off-by: Jan Dubois <[email protected]>
Changing this to ${FOO-} was done by 3.6.0, but has since been reverted. See mvdan/sh#970 Signed-off-by: Jan Dubois <[email protected]>
This PR contains both the BATS framework for testing deployment profiles, and a sample test to demonstrate how to use it.
It works for macOS using user profiles. Testing system profiles will require password-less sudo.
The code has not been tested on Windows yet. It is only expected to work for the user-legacy location on Windows; every other location requires admin access and is out of scope for this PR.