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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

src/qt/test/wallettests.cpp
116 ↗(On Diff #10922)

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

src/qt/test/wallettests.cpp
116 ↗(On Diff #10922)

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