Merge pull request #42457 from yujuhong/do_not_panic

Automatic merge from submit-queue (batch tested with PRs 42456, 42457, 42414, 42480, 42370) node e2e: apparmor test should fail instead of panicking This doesn't fix #42420, but at least stop the test from panicking.

Merge pull request #42457 from yujuhong/do_not_panic
cb0728c5 · Kubernetes Submit Queue · GitHub · c6d9d9c5 · 1d907dbf · cb0728c5
Commit cb0728c5 authored Mar 04, 2017 by Kubernetes Submit Queue Committed by GitHub Mar 04, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 0 deletions

apparmor_test.go test/e2e_node/apparmor_test.go +2 -0

No files found.
--- a/test/e2e_node/apparmor_test.go
+++ b/test/e2e_node/apparmor_test.go
@@ -60,6 +60,7 @@ var _ = framework.KubeDescribe("AppArmor [Feature:AppArmor]", func() {
 					return
 				}
 				state := status.ContainerStatuses[0].State.Terminated
+				Expect(state).ToNot(BeNil(), "ContainerState: %+v", status.ContainerStatuses[0].State)
 				Expect(state.ExitCode).To(Not(BeZero()), "ContainerStateTerminated: %+v", state)

 			})
@@ -70,6 +71,7 @@ var _ = framework.KubeDescribe("AppArmor [Feature:AppArmor]", func() {
 					return
 				}
 				state := status.ContainerStatuses[0].State.Terminated
+				Expect(state).ToNot(BeNil(), "ContainerState: %+v", status.ContainerStatuses[0].State)
 				Expect(state.ExitCode).To(BeZero(), "ContainerStateTerminated: %+v", state)
 			})
 		})