Page MenuHomePhabricator

Temporarly skip running rpcbind_test on WSL
ClosedPublic

Authored by Fabien on Dec 29 2018, 18:10.

Details

Summary

Skip the test while netstat is not supported in WSL.
Related to T400.

Test Plan

On WSL and native linux run:

./test/functional/test_runner.py rpcbind_test

Check that test_runner outputs the test as skipped on WSL
Check that test_runner outputs a "Passed" status for the test on native linux

Diff Detail

Repository
rABC Bitcoin ABC
Branch
wsl_tmp_fix_rpc_bind
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4391
Build 6847: Bitcoin ABC Buildbot (legacy)
Build 6846: arc lint + arc unit

Event Timeline

deadalnix requested changes to this revision.Dec 30 2018, 12:08

I do not think the test plan appropriately test the change.

This revision now requires changes to proceed.Dec 30 2018, 12:08
Fabien edited the test plan for this revision. (Show Details)
Fabien requested review of this revision.Dec 30 2018, 13:06
Fabien edited the test plan for this revision. (Show Details)

Updated test plan

jasonbcox requested changes to this revision.Dec 30 2018, 18:10
jasonbcox added a subscriber: jasonbcox.
jasonbcox added inline comments.
test/functional/rpcbind_test.py
62 ↗(On Diff #6451)

Please update the comment to point to https://reviews.bitcoinabc.org/T400 and note that this message can be removed once netstat support is provided by Microsoft.

This revision now requires changes to proceed.Dec 30 2018, 18:10
Fabien edited the test plan for this revision. (Show Details)

Complete the comment with details according to feedback

test/functional/rpcbind_test.py
65 ↗(On Diff #6468)

It's not really about removing the message, but reenabling test.

Improve the comment message according to feedback

test/functional/rpcbind_test.py
65 ↗(On Diff #6468)

ya, that's a more accurate way of phrasing it.

jasonbcox added inline comments.
test/functional/rpcbind_test.py
65 ↗(On Diff #6468)

As a nit, you can reword this message. Green-ing this diff since the rest looks good.

This revision is now accepted and ready to land.Jan 1 2019, 00:08
This revision was automatically updated to reflect the committed changes.