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.

Sign up for GitHub

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

Jump to bottom

Bug: Test Context is not cleared after run #266

Closed
wurzelkuchen opened this issue Jul 9, 2024 · 0 comments
Closed

Bug: Test Context is not cleared after run #266

wurzelkuchen opened this issue Jul 9, 2024 · 0 comments
Assignees
@oomelianchuk
Labels
bug codeDone The issue is rechecked for development branch
Milestone
v5.0.1

Comments

@wurzelkuchen
Copy link
Contributor

Quick Description

We keep test data in the context during a test run. If a new test starts in the same thread the context is reused, but the data is kept, so if the new test does not explicitly override the data following tests might use wrong test data.

Actual Behaviour

If a test is reusing a thread test data form previous tests are not removed can bleed through to the current run.

Expected Behaviour

If a test runs it will only use the testdata for this specific test, no matter what happened in previous tests.
@wurzelkuchen wurzelkuchen added bug codeImplementationNeeded This issue requires a change in the code of Neodymium labels Jul 9, 2024
@wurzelkuchen wurzelkuchen modified the milestones: v5.0.0, v5.0.1 Jul 9, 2024
oomelianchuk added a commit that referenced this issue Jul 10, 2024
@oomelianchuk
[#266] clear test context after run
7d48d09
wurzelkuchen added a commit that referenced this issue Jul 10, 2024
@wurzelkuchen
Merge pull request #268 from Xceptance/#266-bug-test-context-is-not-c…
a6eed11 
…leared-after-run

[#266] clear test context after run
@wurzelkuchen wurzelkuchen added codeReadyForRecheck The issue is fixed/implemented AND merged into development, but needs to be rechecked there codeDone The issue is rechecked for development branch and removed codeImplementationNeeded This issue requires a change in the code of Neodymium codeReadyForRecheck The issue is fixed/implemented AND merged into development, but needs to be rechecked there labels Jul 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@oomelianchuk oomelianchuk

Labels
bug codeDone The issue is rechecked for development branch
Projects
None yet
Milestone
v5.0.1
Development

No branches or pull requests
2 participants
@wurzelkuchen @oomelianchuk