acquia / cli

Acquia CLI
GNU General Public License v2.0
42 stars 47 forks source link

CLI-1307| Integration tests #1773

Closed ashu-taide closed 1 month ago

ashu-taide commented 1 month ago

Motivation Integration tests that will execute on ubuntu that will simulate how user uses acli on ubuntu

Proposed changes Integration tests added in

Alternatives considered

Testing steps

  1. Follow the contribution guide to set up your development environment or download a pre-built acli.phar for this PR.
  2. Clear the kernel cache to pick up new and changed commands: ./bin/acli ckc
  3. (add specific steps for this pr)
  4. install python3
  5. on macos rm ~/.acquia/cloud_api.conf
  6. on linux rm /root/.config/acquia/cloud_api.conf
  7. export APPLICATION_UUID="appuuid"
  8. export APPLICATION_NAME="appname"
  9. export ACLI_AUTH_TOKEN="authtoken"
  10. export ACLI_AUTH_SECRET="authsecret"
  11. Run python3 -m unittest discover -s tests/integration/testcases in the cli directory
github-actions[bot] commented 1 month ago

Try the dev build for this PR: https://acquia-cli.s3.amazonaws.com/build/pr/1773/acli.phar

curl -OL https://acquia-cli.s3.amazonaws.com/build/pr/1773/acli.phar
chmod +x acli.phar
codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 92.02%. Comparing base (12e1dc5) to head (d17389f). Report is 30 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1773 +/- ## ============================================ + Coverage 91.73% 92.02% +0.29% - Complexity 1810 1819 +9 ============================================ Files 121 121 Lines 6506 6838 +332 ============================================ + Hits 5968 6293 +325 - Misses 538 545 +7 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

danepowell commented 1 month ago

@ashu-taide this looks great, thanks. Anything you need before I formally approve and merge this?