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 test framework prefers global binary over current built #1096

Closed
Licenser opened this issue Jun 30, 2021 · 0 comments · Fixed by #1123
Closed

Integration test framework prefers global binary over current built #1096

Licenser opened this issue Jun 30, 2021 · 0 comments · Fixed by #1123
Labels
bug Something isn't working _complexity:low A task with a low complexity that should be easy to understand _size:small A small task that should be quick to do

Comments

@Licenser
Copy link
Member

Problem

The integration tests pick up a global tremor if installed which makes them an extremely confusing experience when running them during development. Running cargo run ... does in fact run the integration test with not the newly built tremor but instead the existing old one a very unexpected behaviour.

Steps

  1. Install tremor
  2. do a code change
  3. run cargo run -p tremor-cli -- test integration tremor-cli/tests to run the tests
  4. The old binary is used

Possible Solution(s)

For spawning sub instances the integration test should always use its own binary.

@Licenser Licenser added bug Something isn't working _complexity:low A task with a low complexity that should be easy to understand _size:small A small task that should be quick to do labels Jun 30, 2021
Licenser added a commit that referenced this issue Jul 9, 2021
Signed-off-by: Heinz N. Gies <heinz@licenser.net>
Licenser added a commit that referenced this issue Jul 9, 2021
Signed-off-by: Heinz N. Gies <heinz@licenser.net>
@Licenser Licenser mentioned this issue Jul 9, 2021
6 tasks
mfelsche pushed a commit that referenced this issue Jul 9, 2021
Signed-off-by: Heinz N. Gies <heinz@licenser.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working _complexity:low A task with a low complexity that should be easy to understand _size:small A small task that should be quick to do
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant