Skip to content

Conversation

@pguyot
Copy link
Collaborator

@pguyot pguyot commented Dec 6, 2025

Instead of using computed sleep times which CI doesn't like, implement a callback mechanism to test that indeed timeout messages are sent.

Also fix test_start_name/0 so it doesn't leave an unprocessed EXIT message if trap_exit flag was true.

These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).

SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

Instead of using computed sleep times which CI doesn't like, implement a
callback mechanism to test that indeed timeout messages are sent.

Also fix `test_start_name/0` so it doesn't leave an unprocessed `EXIT`
message if trap_exit flag was true.

Signed-off-by: Paul Guyot <pguyot@kallisys.net>
Copy link
Collaborator

@UncleGrumpy UncleGrumpy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks again for this fix.

@bettio bettio merged commit 02d955b into atomvm:main Dec 7, 2025
131 of 133 checks passed
@pguyot pguyot deleted the w49/fix-test_gen_server-flappiness-case branch December 7, 2025 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants