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
Fix reruns of full stack tests on specified temp dir #4524
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wait, I think deleting this directory is wrong
sub setup_fullstack_temp_dir { | ||
my ($test_name) = @_; | ||
my $tempdir = $ENV{OPENQA_FULLSTACK_TEMP_DIR} ? path($ENV{OPENQA_FULLSTACK_TEMP_DIR}) : tempdir; | ||
my $tempdir = _test_dir(); | ||
my $basedir = $tempdir->child($test_name); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This $basedir
is the directory which should be deleted.
e.g. full-stack.d is doing:
my $tempdir = setup_fullstack_temp_dir('full-stack.d');
so it would be $OPENQA_FULLSTACK_TEMP_DIR/full-stack.d
Otherwise it could delete a previous dir from another test.
Codecov Report
@@ Coverage Diff @@
## master #4524 +/- ##
==========================================
- Coverage 97.96% 97.96% -0.01%
==========================================
Files 374 374
Lines 34063 34067 +4
==========================================
+ Hits 33371 33373 +2
- Misses 692 694 +2
Continue to review full report at Codecov.
|
See #4527 |
awesome, yours looks good |
No description provided.