Improve detach unit tests

commands_test.go

@@ -404,7 +404,12 @@ func TestRunDetach(t *testing.T) {
 		<-ch
 	})
 
-	setTimeout(t, "Waiting for container to die timedout", 5*time.Second, func() {
+	time.Sleep(500 * time.Millisecond)
+	if !container.State.Running {
+		t.Fatal("The detached container should be still running")
+	}
+
+	setTimeout(t, "Waiting for container to die timed out", 20*time.Second, func() {
 		container.Kill()
 		container.Wait()
 	})
@@ -457,6 +462,11 @@ func TestAttachDetach(t *testing.T) {
 		<-ch
 	})
 
+	time.Sleep(500 * time.Millisecond)
+	if !container.State.Running {
+		t.Fatal("The detached container should be still running")
+	}
+
 	setTimeout(t, "Waiting for container to die timedout", 5*time.Second, func() {
 		container.Kill()
 		container.Wait()

container.go

@@ -957,9 +957,6 @@ func (container *Container) monitor() {
 	}
 	utils.Debugf("Process finished")
 
-	if container.runtime != nil && container.runtime.srv != nil {
-		container.runtime.srv.LogEvent("die", container.ShortID(), container.runtime.repositories.ImageName(container.Image))
-	}
 	exitCode := -1
 	if container.cmd != nil {
 		exitCode = container.cmd.ProcessState.Sys().(syscall.WaitStatus).ExitStatus()
@@ -968,6 +965,10 @@ func (container *Container) monitor() {
 	// Report status back
 	container.State.setStopped(exitCode)
 
+	if container.runtime != nil && container.runtime.srv != nil {
+		container.runtime.srv.LogEvent("die", container.ShortID(), container.runtime.repositories.ImageName(container.Image))
+	}
+
 	// Cleanup
 	container.releaseNetwork()
 	if container.Config.OpenStdin {