From f956f23d3780ce92ab6f7e778040e4b0305e93f4 Mon Sep 17 00:00:00 2001 From: James DeFelice Date: Mon, 15 Jun 2015 14:30:48 +0000 Subject: [PATCH] move the container name fix to NewTestPod to avoid breakage in other unit tests in the future --- contrib/mesos/pkg/executor/executor_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/contrib/mesos/pkg/executor/executor_test.go b/contrib/mesos/pkg/executor/executor_test.go index faa79372bc..b94deaee18 100644 --- a/contrib/mesos/pkg/executor/executor_test.go +++ b/contrib/mesos/pkg/executor/executor_test.go @@ -344,7 +344,6 @@ func TestExecutorLaunchAndKillTask(t *testing.T) { } pod := NewTestPod(1) - pod.Spec.Containers[0].Name = "foo" podTask, err := podtask.New(api.NewDefaultContext(), "", *pod, &mesosproto.ExecutorInfo{}) assert.Equal(t, nil, err, "must be able to create a task from a pod") @@ -503,6 +502,7 @@ func NewTestPod(i int) *api.Pod { Spec: api.PodSpec{ Containers: []api.Container{ { + Name: "foo", Ports: []api.ContainerPort{ { ContainerPort: 8000 + i,