Closed danepowell closed 1 week ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 92.23%. Comparing base (
81703a7
) to head (a772a8d
). Report is 1 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Try the dev build for this PR: https://acquia-cli.s3.amazonaws.com/build/pr/1796/acli.phar
curl -OL https://acquia-cli.s3.amazonaws.com/build/pr/1796/acli.phar
chmod +x acli.phar
Motivation
Fixes CLI-1389
Proposed changes
Improve tests to catch validation errors in datastores to prevent regressions like CLI-1389
Calling set() on a keystore doesn't trigger validation. Validation only occurs when the contents of the datastore file are loaded from disk. So test cases should never use set() for configuring a test fixture, but rather write the file and reload the command/datastore.
Testing steps
./bin/acli ckc