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.

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

testTrigger fails with a race condition #147

Closed
killenb opened this issue Apr 9, 2020 · 3 comments
Closed

testTrigger fails with a race condition #147

killenb opened this issue Apr 9, 2020 · 3 comments

Comments

@killenb
Copy link
Member

killenb commented Apr 9, 2020

The issue occurred when something in the recovery mechanism was changed (#144). This probably changed the timimg. The first test ( testTriggerDevToApp<a>) always fails, but running the test multiple times shows that also following tests sometimes fail.

24/26 Test: testTrigger
Command: "/home/killenb/build-ApplicationCore-Desktop-Debug/testTrigger"
Directory: /home/killenb/build-ApplicationCore-Desktop-Debug
"testTrigger" start time: Apr 08 20:04 CEST
Output:
----------------------------------------------------------
Running 32 test cases...
*********************************************************************************************************************
==> testTriggerDevToApp<a>
/home/killenb/ApplicationCore/tests/executables_src/testTrigger.cc(149): error in "testTriggerDevToApp<a>": check app.testModule.consumingPush == 1 failed
/home/killenb/ApplicationCore/tests/executables_src/testTrigger.cc(152): error in "testTriggerDevToApp<a>": check app.testModule.consumingPush == 1 failed
*********************************************************************************************************************
==> testTriggerDevToApp<h>
*********************************************************************************************************************
==> testTriggerDevToApp<s>
*********************************************************************************************************************
==> testTriggerDevToApp<t>
....
@killenb
Copy link
Member Author

killenb commented Apr 9, 2020

Currently the code is in branch wip-phako-144 because it would break the master. But issue #144 is a blocking key element of the current focus week.

@phako phako self-assigned this Apr 9, 2020
@phako
Copy link
Member

phako commented Apr 9, 2020

73629cd improves the race condition a bit

@mhier
Copy link
Member

mhier commented Sep 16, 2020

See #197

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants