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
Conflict with signinfo's temporary linguistic node sequence file caused test failures. #4625
Conversation
Still having issues. Looking at it |
@@ -30,7 +30,7 @@ TEST(Signinfo, ExitToTest) { | |||
std::vector<SignInfo> signs; | |||
std::vector<std::string> linguistics; | |||
std::vector<std::pair<std::string, bool>> default_languages; | |||
const std::string linguistic_node_file = "test_linguistic_node.bin"; | |||
const std::string linguistic_node_file = "test_sign_linguistic_node.bin"; |
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 file was the cause of the problem. In the multi-threaded environment the test_linguistic_node.bin
was getting deleted for the graphparser.cc
test.
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.
so this test also builds to the same directory as graphparser.cc
? if so, shouldn't we change that?
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.
Most are all running in the same directory. This test is testing specifically signs and is not building data. This is a temporary file as it is required for CreateSignInfoList
We are not testing linguistic node signage here as it is done in the gurka tests. The real solution is to migrate the old tests to gurka.
reverted back
Issue
graphparser test is randomly failing. Ref: #4606 (comment)
Tasklist
Requirements / Relations
Link any requirements here. Other pull requests this PR is based on?