23148220ec
- api: move more network-related types to api/types/network
full diff: cd3804655a...c6aaabc9fc
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
66 lines
1.5 KiB
Go
66 lines
1.5 KiB
Go
package swarm
|
|
|
|
import (
|
|
"context"
|
|
"testing"
|
|
|
|
"github.com/docker/cli/internal/test/network"
|
|
networktypes "github.com/docker/docker/api/types/network"
|
|
"github.com/pkg/errors"
|
|
"gotest.tools/v3/assert"
|
|
)
|
|
|
|
type notFound struct {
|
|
error
|
|
}
|
|
|
|
func (n notFound) NotFound() {}
|
|
|
|
func TestValidateExternalNetworks(t *testing.T) {
|
|
testcases := []struct {
|
|
inspectResponse networktypes.Inspect
|
|
inspectError error
|
|
expectedMsg string
|
|
network string
|
|
}{
|
|
{
|
|
inspectError: notFound{},
|
|
expectedMsg: "could not be found. You need to create a swarm-scoped network",
|
|
},
|
|
{
|
|
inspectError: errors.New("Unexpected"),
|
|
expectedMsg: "Unexpected",
|
|
},
|
|
// FIXME(vdemeester) that doesn't work under windows, the check needs to be smarter
|
|
/*
|
|
{
|
|
inspectError: errors.New("host net does not exist on swarm classic"),
|
|
network: "host",
|
|
},
|
|
*/
|
|
{
|
|
network: "user",
|
|
expectedMsg: "is not in the right scope",
|
|
},
|
|
{
|
|
network: "user",
|
|
inspectResponse: networktypes.Inspect{Scope: "swarm"},
|
|
},
|
|
}
|
|
|
|
for _, testcase := range testcases {
|
|
fakeClient := &network.FakeClient{
|
|
NetworkInspectFunc: func(_ context.Context, _ string, _ networktypes.InspectOptions) (networktypes.Inspect, error) {
|
|
return testcase.inspectResponse, testcase.inspectError
|
|
},
|
|
}
|
|
networks := []string{testcase.network}
|
|
err := validateExternalNetworks(context.Background(), fakeClient, networks)
|
|
if testcase.expectedMsg == "" {
|
|
assert.NilError(t, err)
|
|
} else {
|
|
assert.ErrorContains(t, err, testcase.expectedMsg)
|
|
}
|
|
}
|
|
}
|