timeout_tests: do not call sleep from test thread

[LMESTM]

Description

This is up to scheduler to call sleep when threads are inactive.
So the thread should simply wait for the semaphore for ever (or test timeout).

Pull request type

[ ] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[x ] Test update
[ ] Breaking change

Reviewers

@jeromecoutant @mprse

Release Notes

[ciarmcom]

@LMESTM, thank you for your changes.
@mprse @ARMmbed/mbed-os-core @ARMmbed/mbed-os-test @ARMmbed/mbed-os-maintainers please review.

[kjbracey]

Indeed, but I'm wondering if there is some subtle reason for the code being like that I'm missing.

Following the Semaphore API revision just made, Semaphore::wait should be Semaphore::acquire.

[kjbracey]

Is this fixing a particular issue you've spotted?

[LMESTM]

@kjbracey-arm I'm working on updating LP ticker implementation for STM targets. The changes have highlighted some dependencies to test updates. I'm pushing generic test updates first, and will reference them from the LP ticker change.

According to your comment, should I replace wait by acquire in this PR ?

[kjbracey]

Yes, wait has just been deprecated - the code you're changing has just been changed from wait(0) to try_acquire().

[LMESTM]

#10701 is the change that triggered this test update
I will update the PR with new API

[LMESTM]

PR updated with new Semaphore API and verified on NUCLEO_L476RG

+---------------------+---------------+------------------------------+-------------------------------------------------+--------+--------+--------+--------------------+
| target              | platform_name | test suite                   | test case                                       | passed | failed | result | elapsed_time (sec) |
+---------------------+---------------+------------------------------+-------------------------------------------------+--------+--------+--------+--------------------+
| NUCLEO_L476RG-ARMC6 | NUCLEO_L476RG | tests-mbed_hal-sleep_manager | deep sleep lock/unlock                          | 1      | 0      | OK     | 0.05               |
| NUCLEO_L476RG-ARMC6 | NUCLEO_L476RG | tests-mbed_hal-sleep_manager | deep sleep lock/unlock test_check               | 1      | 0      | OK     | 0.07               |
| NUCLEO_L476RG-ARMC6 | NUCLEO_L476RG | tests-mbed_hal-sleep_manager | deep sleep locked USHRT_MAX times               | 1      | 0      | OK     | 0.12               |
| NUCLEO_L476RG-ARMC6 | NUCLEO_L476RG | tests-mbed_hal-sleep_manager | sleep_auto calls sleep/deep sleep based on lock | 1      | 0      | OK     | 0.14               |
+---------------------+---------------+------------------------------+-------------------------------------------------+--------+--------+--------+--------------------+

[0xc0170]

CI started while waiting for more reviews

[mbed-ci]

Test run: FAILED

Summary: 3 of 3 test jobs failed
Build number : 1
Build artifacts

Failed test jobs:

• jenkins-ci/mbed-os-ci_build-GCC_ARM
• jenkins-ci/mbed-os-ci_build-IAR
• jenkins-ci/mbed-os-ci_build-ARM

[mbed-ci]

Test run: SUCCESS

Summary: 4 of 4 test jobs passed
Build number : 2
Build artifacts