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
Fixes #10842: Update write test for 4.1 and add generation for a node #1659
Conversation
val pathComputer = new PathComputerImpl( | ||
SHARE.getParent + "/" | ||
, SHARE.getName | ||
, abstractRoot.getAbsolutePath + "/backup" |
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.
no prob if backup is the same for different label ?
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.
it doesn't seems to care. And at least for the generation part, it does not seems to matter.
|
||
//////////// end init //////////// | ||
|
||
//////////// set-up auto test cleaning //////////// | ||
override def afterAll(): Unit = { | ||
if(System.getProperty("tests.clean.tmp") != "false") { | ||
if(false) { // System.getProperty("tests.clean.tmp") != "false") { |
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.
Remove dead code (or set back condition)
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.
done
PR rebased |
9ef8d0f
to
971dacc
Compare
I see noihing strange or invaluid apart the small remarks, merging |
OK, merging this PR |
https://www.rudder-project.org/redmine/issues/10842