Update test_integration - Githubissues

cyberark / cyberark-conjur-cli

CyberArk Conjur command line interface written in Python

https://www.conjur.org

Apache License 2.0

17 stars 16 forks source link

Update test_integration #394

Closed mbenita-Cyberark closed 2 years ago

mbenita-Cyberark commented 2 years ago

Desired Outcome

Developer could run integration tests with local repo intsalled as a python package in the tests container

Implemented Changes

Added another named argument -s | --sdk-repo-path to add path to local repo
To test this works run ./bin/test_integration -s /path/to/conjur-api/repo -d
inside the container run python and see you can run import conjur_api

Definition of Done

[ ] run integration tests with external python package installed from repo inside the testing container

Changelog

[ ] The CHANGELOG has been updated, or
[x] This PR does not include user-facing changes and doesn't require a CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code changes, or
[x] The changes in this PR do not require tests

Documentation

[x] Docs (e.g. READMEs) were updated in this PR
[ ] A follow-up issue to update official docs has been filed here: [insert issue ID]()
[ ] This PR does not require updating any documentation

Behavior

[ ] This PR changes product behavior and has been reviewed by a PO, or
[ ] These changes are part of a larger initiative that will be reviewed later, or
[x] No behavior was changed with this PR

Security

[x] Security architect has reviewed the changes in this PR,
[ ] These changes are part of a larger initiative with a separate security review, or
[ ] There are no security aspects to these changes