-
Notifications
You must be signed in to change notification settings - Fork 300
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
HPCC-18936 Allow a timing unit test to be run individually #10762
Conversation
@richardkchapman please review |
@ghalliday I saw some problem with toxml.ecl before and put a comment about it here: https://track.hpccsystems.com/browse/HPCC-18918?focusedCommentId=76425&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-76425 |
@richardkchapman I think the failure is unrelated. However it looks like it happened after the ifblock fix was merged, so may still be a problem. |
I suspect it's a race condition - I think the ECL both creates and reads that file, and it may be that the codegen cannot spot the dependency. Should raise a Jira to fix it but as you say not related |
roxie/roxiemem/roxiemem.cpp
Outdated
@@ -8716,9 +8716,9 @@ static int compareTiming(const void * pLeft, const void * pRight) | |||
} | |||
|
|||
|
|||
class RoxieMemTimingTests2 : public CppUnit::TestFixture |
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.
Not sure I understand why this name change was made
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.
The name was changed because the filter for running a unit test matches using a substring. If the 2 is on the end of the test name it is impossible to only run the first test - since it will also run this 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.
Hmm. I would have gone for calling them tests1 and tests2 then...
@ghalliday One minor question |
Signed-off-by: Gavin Halliday <gavin.halliday@lexisnexis.com>
@richardkchapman repushed |
Automated Smoketest: ✅ Unit tests result:
Regression test result:
HPCC Stop: OK
|
Signed-off-by: Gavin Halliday gavin.halliday@lexisnexis.com
Type of change:
Checklist:
Testing: