Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integration Tests should be reorganized #2

Open
choksi81 opened this issue May 26, 2014 · 1 comment
Open

Integration Tests should be reorganized #2

choksi81 opened this issue May 26, 2014 · 1 comment

Comments

@choksi81
Copy link

The current integration tests are structured in such a way that every integration test has its own set of repy dependencies. This makes updating the e-mail list a pain, as well as makes the directories for each test case very cluttered.

We should reorganize it so that the integration tests share the repy dependencies (files generated by preparetest.py). Also, we should only include the actual test cases in their own directories as well.

@choksi81 choksi81 self-assigned this May 26, 2014
@aaaaalbert
Copy link
Contributor

@aot221 is working on this.

updating the e-mail list

is facilitated by freshly reading in the list on every call of integrationtestlib.notify(), see aot221/integrationtests@a7bd41c (further cleanup scheduled before we merge.)

directories [...] cluttered

The repo's subdirectories are clean, so that's a build issue. I suppose we could let all of the tests use the same Repy runtime if we wanted to.

The new question I have is whether we should make this repo only contain the common elements/libraries required to scaffold an integration test, and move the actual tests to the directories of the components they are testing. For example, you wouldn't want to run timeserver integration tests if you are running a custom clearinghouse only.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants