mirror of
https://github.com/moby/moby.git
synced 2022-11-09 12:21:53 -05:00
Merge pull request #36223 from yongtang/02062018-clean
Combine runSimpleContainer with runContainer for rename test
This commit is contained in:
commit
4ba4b4b283
1 changed files with 11 additions and 54 deletions
|
@ -7,22 +7,11 @@ import (
|
|||
"github.com/docker/docker/api/types"
|
||||
"github.com/docker/docker/api/types/container"
|
||||
"github.com/docker/docker/api/types/network"
|
||||
"github.com/docker/docker/api/types/strslice"
|
||||
"github.com/docker/docker/client"
|
||||
"github.com/docker/docker/integration/util/request"
|
||||
"github.com/stretchr/testify/assert"
|
||||
"github.com/stretchr/testify/require"
|
||||
)
|
||||
|
||||
func runContainer(ctx context.Context, t *testing.T, client client.APIClient, cntCfg *container.Config, hstCfg *container.HostConfig, nwkCfg *network.NetworkingConfig, cntName string) string {
|
||||
cnt, err := client.ContainerCreate(ctx, cntCfg, hstCfg, nwkCfg, cntName)
|
||||
require.NoError(t, err)
|
||||
|
||||
err = client.ContainerStart(ctx, cnt.ID, types.ContainerStartOptions{})
|
||||
require.NoError(t, err)
|
||||
return cnt.ID
|
||||
}
|
||||
|
||||
// This test simulates the scenario mentioned in #31392:
|
||||
// Having two linked container, renaming the target and bringing a replacement
|
||||
// and then deleting and recreating the source container linked to the new target.
|
||||
|
@ -32,57 +21,25 @@ func TestRenameLinkedContainer(t *testing.T) {
|
|||
ctx := context.Background()
|
||||
client := request.NewAPIClient(t)
|
||||
|
||||
cntConfig := &container.Config{
|
||||
Image: "busybox",
|
||||
Tty: true,
|
||||
Cmd: strslice.StrSlice([]string{"top"}),
|
||||
}
|
||||
aID := runSimpleContainer(ctx, t, client, "a0")
|
||||
|
||||
var (
|
||||
aID, bID string
|
||||
cntJSON types.ContainerJSON
|
||||
err error
|
||||
)
|
||||
bID := runSimpleContainer(ctx, t, client, "b0", func(config *container.Config, hostConfig *container.HostConfig, networkingConfig *network.NetworkingConfig) {
|
||||
hostConfig.Links = []string{"a0"}
|
||||
})
|
||||
|
||||
aID = runContainer(ctx, t, client,
|
||||
cntConfig,
|
||||
&container.HostConfig{},
|
||||
&network.NetworkingConfig{},
|
||||
"a0",
|
||||
)
|
||||
|
||||
bID = runContainer(ctx, t, client,
|
||||
cntConfig,
|
||||
&container.HostConfig{
|
||||
Links: []string{"a0"},
|
||||
},
|
||||
&network.NetworkingConfig{},
|
||||
"b0",
|
||||
)
|
||||
|
||||
err = client.ContainerRename(ctx, aID, "a1")
|
||||
err := client.ContainerRename(ctx, aID, "a1")
|
||||
require.NoError(t, err)
|
||||
|
||||
runContainer(ctx, t, client,
|
||||
cntConfig,
|
||||
&container.HostConfig{},
|
||||
&network.NetworkingConfig{},
|
||||
"a0",
|
||||
)
|
||||
runSimpleContainer(ctx, t, client, "a0")
|
||||
|
||||
err = client.ContainerRemove(ctx, bID, types.ContainerRemoveOptions{Force: true})
|
||||
require.NoError(t, err)
|
||||
|
||||
bID = runContainer(ctx, t, client,
|
||||
cntConfig,
|
||||
&container.HostConfig{
|
||||
Links: []string{"a0"},
|
||||
},
|
||||
&network.NetworkingConfig{},
|
||||
"b0",
|
||||
)
|
||||
bID = runSimpleContainer(ctx, t, client, "b0", func(config *container.Config, hostConfig *container.HostConfig, networkingConfig *network.NetworkingConfig) {
|
||||
hostConfig.Links = []string{"a0"}
|
||||
})
|
||||
|
||||
cntJSON, err = client.ContainerInspect(ctx, bID)
|
||||
inspect, err := client.ContainerInspect(ctx, bID)
|
||||
require.NoError(t, err)
|
||||
assert.Equal(t, []string{"/a0:/b0/a0"}, cntJSON.HostConfig.Links)
|
||||
assert.Equal(t, []string{"/a0:/b0/a0"}, inspect.HostConfig.Links)
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue