Support for cap-add/cap-drop

[dhuckins]

context from slack

adds cap-add/cap-drop to container request and passes to docker host config

[codecov]

Codecov Report

Merging #555 (537c95e) into main (66fed81) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #555      +/-   ##
==========================================
+ Coverage   69.01%   69.04%   +0.02%     
==========================================
  Files          22       22              
  Lines        2172     2174       +2     
==========================================
+ Hits         1499     1501       +2     
  Misses        535      535              
  Partials      138      138              

Impacted Files	Coverage Δ
container.go	80.72% <ø> (ø)
docker.go	71.73% <100.00%> (+0.05%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[mdelapenya]

This LGTM, although I encourage you to add a test :)

Take this as an example:

testcontainers-go/docker_test.go

Lines 2131 to 2177 in cb9d7c8

	func TestDockerContainerResources(t *testing.T) {
	if providerType == ProviderPodman {
	t.Skip("Rootless Podman does not support setting rlimit")
	}
	ctx := context.Background()
	expected := []*units.Ulimit{
	{
	Name: "memlock",
	Hard: -1,
	Soft: -1,
	},
	{
	Name: "nofile",
	Hard: 65536,
	Soft: 65536,
	},
	}
	nginxC, err := GenericContainer(ctx, GenericContainerRequest{
	ProviderType: providerType,
	ContainerRequest: ContainerRequest{
	Image: nginxAlpineImage,
	ExposedPorts: []string{nginxDefaultPort},
	WaitingFor: wait.ForListeningPort(nginxDefaultPort),
	Resources: container.Resources{
	Ulimits: expected,
	},
	},
	Started: true,
	})
	require.NoError(t, err)
	terminateContainerOnEnd(t, ctx, nginxC)
	c, err := client.NewClientWithOpts(client.FromEnv)
	require.NoError(t, err)
	c.NegotiateAPIVersion(ctx)
	containerID := nginxC.GetContainerID()
	resp, err := c.ContainerInspect(ctx, containerID)
	require.NoError(t, err)
	assert.Equal(t, expected, resp.HostConfig.Ulimits)
	}

[dhuckins]

@mdelapenya added test

[mdelapenya]

Thanks taking the time to contribute this ❤️

LGTM!