search

aws / amazon-chime-sdk-cpp

Apache License 2.0
13 stars 5 forks source link

Add scripts #4

Open hokyungh opened 1 year ago

hokyungh commented 1 year ago

Description of changes: Add scripts to run the cli easier to parse.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

hokyungh commented 1 year ago

Seems test_cli/scripts/run_cli.py and cli/scripts/run_cli.py are very similar. What is test_cli/scripts/run_cli.py for exactly and why can't we combine these scripts?

One is used for just purely testing - canary - integration tests, so I feel like script can diverge at the end, so separating it out.

hensmi-amazon commented 1 year ago

I'm not convinced that we should have a separate CLI for integration tests; its not like JS SDK where we are using webdrivers and all that stuff which are very sensitive to demo changes. Lets just make that split when we need to.