search

anfema / integrate

Python integration test framework
BSD 3-Clause "New" or "Revised" License
39 stars 7 forks source link

Added ability to provide optional arguments to the test suite being invoked #7

Closed kensthilaire closed 4 years ago

kensthilaire commented 5 years ago

I have used the integrate framework for some time. In my work, I need to deploy and test the software application in multiple environments (development, integration, and production). Each of these environments requires different configuration parameters based on the environment, and I couldn't find a clean way to use the integrate framework 'out of the box' that allowed me to run the same test suite in each environment.

With this pull request, I have introduced an optional 'args' parameter that is passed to the setup_all() function within the test suite. I use this args parameter to pass the necessary configuration information to the test suite, allowing a single suite to be easily used to test in multiple environments.

I did not update the README, but can do so if you think that this behavior is worth integrating into the base distribution.

dunkelstern commented 5 years ago

Sorry for the late answer, the notification got swamped, will look into this in the next few days.

kensthilaire commented 4 years ago

Hi, I thought I would check back with you on whether you have had a look at this pull request. I have been actively working with my forked version pretty successfully.

dunkelstern commented 4 years ago

Sorry for the delay, looks good, merged and published as 1.3.0. Thanks :)

kensthilaire commented 4 years ago

Awesome, thank you.

On Wed, Oct 16, 2019 at 9:14 AM Johannes Schriewer notifications@github.com wrote:

Sorry for the delay, looks good, merged and published as 1.3.0. Thanks :)

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/anfema/integrate/pull/7?email_source=notifications&email_token=AAY2REMOFEBBNWQCOWWTLQTQO4HSRA5CNFSM4HBEV5NKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBMNWSI#issuecomment-542694217, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAY2REPVMC5GG6BXIVAHNOTQO4HSRANCNFSM4HBEV5NA .