Skip to content

Commit 0855922

Browse files
committed
Migrate TestKillDifferentUserContainer to api test
This fix migrates TestKillDifferentUserContainer to api test Signed-off-by: Yong Tang <[email protected]>
1 parent 7e7f816 commit 0855922

2 files changed

Lines changed: 18 additions & 25 deletions

File tree

integration-cli/docker_cli_kill_test.go

Lines changed: 0 additions & 25 deletions
This file was deleted.

integration/container/kill_test.go

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -175,3 +175,21 @@ func TestKillStoppedContainerAPIPre120(t *testing.T) {
175175
err = client.ContainerKill(ctx, c.ID, "SIGKILL")
176176
require.NoError(t, err)
177177
}
178+
179+
func TestKillDifferentUserContainer(t *testing.T) {
180+
// TODO Windows: Windows does not yet support -u (Feb 2016).
181+
skip.If(t, testEnv.OSType != "linux", "User containers (container.Config.User) are not yet supported on %q platform", testEnv.OSType)
182+
183+
defer setupTest(t)()
184+
ctx := context.Background()
185+
client := request.NewAPIClient(t, client.WithVersion("1.19"))
186+
187+
cID := runSimpleContainer(ctx, t, client, "", func(config *container.Config, hostConfig *container.HostConfig, networkingConfig *network.NetworkingConfig) {
188+
config.User = "daemon"
189+
})
190+
poll.WaitOn(t, containerIsInState(ctx, client, cID, "running"), poll.WithDelay(100*time.Millisecond))
191+
192+
err := client.ContainerKill(ctx, cID, "SIGKILL")
193+
require.NoError(t, err)
194+
poll.WaitOn(t, containerIsInState(ctx, client, cID, "exited"), poll.WithDelay(100*time.Millisecond))
195+
}

0 commit comments

Comments
 (0)