Page MenuHomePhabricator

Fix abc-invalid-message.py flakiness
ClosedPublic

Authored by Fabien on Oct 12 2021, 08:20.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Commits
rABCb04fcc9242d2: Fix abc-invalid-message.py flakiness
Summary

The test will fail occasionally, especially on OSX. This is due to a peer disconnection happening before the connection check completed.
This diff delays sending the invalid message and prevents this wrong operation ordering.

Note that I couldn't reproduce the issue locally, but at least this doesn't make things worst.

Test Plan

In a loop:

./test/functional/test_runner.py abc-invalid-message

Diff Detail

Repository
rABC Bitcoin ABC
Branch
fix_abc_invalid_message_flakyness
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 16961
Build 33758: Build Diffbuild-diff · build-debug · build-without-wallet · build-clang-tidy · build-clang
Build 33757: arc lint + arc unit

Event Timeline

Fabien requested review of this revision.Oct 12 2021, 08:20
Fabien retitled this revision from Fix abc-invlaid-message.py flakiness to Fix abc-invalid-message.py flakiness.Oct 12 2021, 08:55
This revision is now accepted and ready to land.Oct 12 2021, 11:32
This revision was automatically updated to reflect the committed changes.