Added panic test for res.Failure

Signed-off-by: dhruv0000 <[email protected]>
layer5io · Mar 17, 2021 · a9e71a9 · a9e71a9
1 parent 92d8cea
commit a9e71a9
Showing 1 changed file with 10 additions and 8 deletions.
diff --git a/smi-conformance/grpc/handlers.go b/smi-conformance/grpc/handlers.go
@@ -98,14 +98,16 @@ func (s *Service) RunTest(ctx context.Context, req *conformance.Request) (*confo
 
 			// A hacky way to see the testStep Failed, since KUDO only provides it in Failure.Message
 			re := regexp.MustCompile(`[0-9]+`)
-			stepFailed := re.FindAllString(res.Failure.Message, 1)
-			if len(stepFailed) != 0 {
-				passed, _ := strconv.Atoi(stepFailed[0])
-				passed = passed - 1
-				failures := stepsCount[res.Name] - passed
-				totalFailures += failures
-				if (passed) >= (stepsCount[res.Name] / 2) {
-					d.Capability = conformance.Capability_HALF
+			if res.Failure != nil {
+				stepFailed := re.FindAllString(res.Failure.Message, 1)
+				if len(stepFailed) != 0 {
+					passed, _ := strconv.Atoi(stepFailed[0])
+					passed = passed - 1
+					failures := stepsCount[res.Name] - passed
+					totalFailures += failures
+					if (passed) >= (stepsCount[res.Name] / 2) {
+						d.Capability = conformance.Capability_HALF
+					}
 				}
 			}
 		}