Fix and re-enable TestGetContainersTop

This commit is contained in:
Johannes 'fish' Ziemke 2013-12-16 13:29:06 +01:00
parent 4faba4fae7
commit 5cfcb05486
1 changed files with 5 additions and 6 deletions

View File

@ -432,7 +432,6 @@ func TestGetContainersChanges(t *testing.T) {
} }
func TestGetContainersTop(t *testing.T) { func TestGetContainersTop(t *testing.T) {
t.Skip("Fixme. Skipping test for now. Reported error when testing using dind: 'api_test.go:527: Expected 2 processes, found 0.'")
eng := NewTestEngine(t) eng := NewTestEngine(t)
defer mkRuntimeFromEngine(eng, t).Nuke() defer mkRuntimeFromEngine(eng, t).Nuke()
srv := mkServerFromEngine(eng, t) srv := mkServerFromEngine(eng, t)
@ -475,7 +474,7 @@ func TestGetContainersTop(t *testing.T) {
}) })
r := httptest.NewRecorder() r := httptest.NewRecorder()
req, err := http.NewRequest("GET", "/"+containerID+"/top?ps_args=u", bytes.NewReader([]byte{})) req, err := http.NewRequest("GET", "/containers/"+containerID+"/top?ps_args=aux", nil)
if err != nil { if err != nil {
t.Fatal(err) t.Fatal(err)
} }
@ -498,11 +497,11 @@ func TestGetContainersTop(t *testing.T) {
if len(procs.Processes) != 2 { if len(procs.Processes) != 2 {
t.Fatalf("Expected 2 processes, found %d.", len(procs.Processes)) t.Fatalf("Expected 2 processes, found %d.", len(procs.Processes))
} }
if procs.Processes[0][10] != "/bin/sh" && procs.Processes[0][10] != "cat" { if procs.Processes[0][10] != "/bin/sh -c cat" {
t.Fatalf("Expected `cat` or `/bin/sh`, found %s.", procs.Processes[0][10]) t.Fatalf("Expected `/bin/sh -c cat`, found %s.", procs.Processes[0][10])
} }
if procs.Processes[1][10] != "/bin/sh" && procs.Processes[1][10] != "cat" { if procs.Processes[1][10] != "/bin/sh -c cat" {
t.Fatalf("Expected `cat` or `/bin/sh`, found %s.", procs.Processes[1][10]) t.Fatalf("Expected `/bin/sh -c cat`, found %s.", procs.Processes[1][10])
} }
} }