Skip to content

Networking: add regression test for networks endpoint with trailing slash#41729

Merged
tiborvass merged 1 commit into
moby:masterfrom
thaJeztah:add_network_list_regression_test
Dec 15, 2020
Merged

Networking: add regression test for networks endpoint with trailing slash#41729
tiborvass merged 1 commit into
moby:masterfrom
thaJeztah:add_network_list_regression_test

Conversation

@thaJeztah

@thaJeztah thaJeztah commented Nov 30, 2020

Copy link
Copy Markdown
Member

relates to #24595
relates to #24634
closes #41728

This adds a regression test for #24595

make DOCKER_GRAPHDRIVER=vfs TEST_FILTER='TestNetworkList' test-integration

INFO: Testing against a local daemon
=== RUN   TestNetworkList
=== RUN   TestNetworkList//networks
=== PAUSE TestNetworkList//networks
=== RUN   TestNetworkList//networks/
=== PAUSE TestNetworkList//networks/
=== CONT  TestNetworkList//networks
=== CONT  TestNetworkList//networks/
--- PASS: TestNetworkList (0.05s)
    --- PASS: TestNetworkList//networks/ (0.01s)
    --- PASS: TestNetworkList//networks (0.01s)
PASS

…lash

This adds a regression test for moby#24595

    make DOCKER_GRAPHDRIVER=vfs TEST_FILTER='TestNetworkList' test-integration

    INFO: Testing against a local daemon
    === RUN   TestNetworkList
    === RUN   TestNetworkList//networks
    === PAUSE TestNetworkList//networks
    === RUN   TestNetworkList//networks/
    === PAUSE TestNetworkList//networks/
    === CONT  TestNetworkList//networks
    === CONT  TestNetworkList//networks/
    --- PASS: TestNetworkList (0.05s)
        --- PASS: TestNetworkList//networks/ (0.01s)
        --- PASS: TestNetworkList//networks (0.01s)
    PASS

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah

Copy link
Copy Markdown
Member Author

@cpuguy83 @tiborvass PTAL

@tiborvass
tiborvass merged commit 808559d into moby:master Dec 15, 2020
@thaJeztah
thaJeztah deleted the add_network_list_regression_test branch December 16, 2020 01:15
@thaJeztah thaJeztah added this to the 20.10.2 milestone Jan 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants