remove success info from slack message and only include failed lanes (#…

…539) ## Motivation ## Solution
smartcontractkit · Feb 21, 2024 · cd89e45 · cd89e45
1 parent f219100
commit cd89e45
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 13 deletions.
diff --git a/integration-tests/ccip-tests/testreporters/ccip.go b/integration-tests/ccip-tests/testreporters/ccip.go
@@ -241,22 +241,19 @@ func (r *CCIPTestReporter) SendSlackNotification(t *testing.T, slackClient *slac
 					"Run Duration = %.0fm "+
 						"\nNumber of ccip-send= %d"+
 						"\nNo of failed requests = %d", r.duration.Minutes(), lane.TotalRequests, lane.FailedCountsByPhase[E2E]))
-		} else {
-			msgTexts = append(msgTexts,
-				fmt.Sprintf("lane %s :white_check_mark:", name),
-				fmt.Sprintf(
-					"Run Duration = %.0fm "+
-						"\nNumber of ccip-send= %d", r.duration.Minutes(), lane.TotalRequests))
 		}
 	}
 
 	msgTexts = append(msgTexts, fmt.Sprintf(
 		"\nTest Run Summary created on _remote-test-runner_ at _%s_\nNotifying <@%s>",
 		r.reportFilePath, testreporters.SlackUserID))
+	if r.namespace == "" {
+		r.SetNamespace("ccip")
+	}
 	messageBlocks := testreporters.SlackNotifyBlocks(headerText, r.namespace, msgTexts)
 	ts, err := testreporters.SendSlackMessage(slackClient, slack.MsgOptionBlocks(messageBlocks...))
 	if err != nil {
-		return err
+		return fmt.Errorf("failed to send slack message: %w messageBlocks = %v", err, messageBlocks)
 	}
 
 	return testreporters.UploadSlackFile(slackClient, slack.FileUploadParameters{

diff --git a/integration-tests/ccip-tests/testsetups/ccip.go b/integration-tests/ccip-tests/testsetups/ccip.go
@@ -695,12 +695,6 @@ func CCIPDefaultTestSetUp(
 			}
 		}
 	}
-	if namespace == "" {
-		if value, set := os.LookupEnv(config.EnvVarNamespace); set && value != "" {
-			namespace = value
-		}
-	}
-
 	setUpArgs.Cfg.TestGroupInput.SetTestRunName(namespace)
 	_, err = os.Stat(setUpArgs.LaneConfigFile)
 	if err == nil {