Page MenuHomePhabricator

Merge #12917: qa: Windows fixups for functional tests
ClosedPublic

Authored by markblundeberg on Feb 6 2020, 02:06.

Details

Summary

PR12917 backport.

fab9095d40 qa: Windows fixups for functional tests (MarcoFalke)

Pull request description:

Just two minor fixups to have less errors when the tests run on native windows.
* Strip whitespace from lines when reading from a notification file
* Instead of clumsily creating a file with weird permissions, just create a folder for the same effect in `mempool_persist.py`
Test Plan

ninja check-all

Diff Detail

Repository
rABC Bitcoin ABC
Branch
pr12917
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 9288
Build 16513: Default Diff Build & Tests
Build 16512: arc lint + arc unit

Event Timeline

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those Bitcoin Core PRs have been inserted into the summary for reference.

This revision is now accepted and ready to land.Feb 6 2020, 02:11

This is rerunning on the highperf3 machine...