Page MenuHomePhabricator

Disable wallet and address book Qt tests on macOS minimal platform
ClosedPublic

Authored by Fabien on Aug 22 2019, 10:02.

Details

Summary
macOS Qt minimal platform is frequently broken, and these are currently
failing with Qt 5.11.1.

The tests do pass when run on the full cocoa platform
(with `test_bitcoin-qt -platform cocoa`).

Backport of core PR14011
https://github.com/bitcoin/bitcoin/pull/14011/files

This was improperly backported in D2054.
Expected to fix issue #340:
https://github.com/Bitcoin-ABC/bitcoin-abc/issues/340

Test Plan

On OSX with minimal platform (default):

make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
PR14011
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7224
Build 12493: Bitcoin ABC Buildbot (legacy)
Build 12492: arc lint + arc unit

Event Timeline

src/qt/test/wallettests.cpp
116

Why is this removed ? This isn't part of the original PR.

src/qt/test/wallettests.cpp
116

The fix has already been backported in D2054, but not at the same place than the original PR.
I moved it to make it consistent with core, there is no functional change.

This revision is now accepted and ready to land.Aug 23 2019, 15:03