fix: do not store output in test suite execution result (#4409)

* fix: do not store output in test suite execution result * fix: clean execution result in the BatchResults as well * fix: add apis spec note and check for nil
kubeshop · Oct 2, 2023 · e2341a4 · e2341a4
1 parent 07469be
commit e2341a4
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 1 deletion.
diff --git a/api/v1/testkube.yaml b/api/v1/testkube.yaml
@@ -3432,7 +3432,7 @@ components:
           description: object name and namespace
         execution:
           $ref: "#/components/schemas/Execution"
-          description: test step execution
+          description: "test step execution, NOTE: the execution output will be empty, retrieve it directly form the test execution"
 
     TestSuiteStepExecutionResultV2:
       description: execution result returned from executor

diff --git a/pkg/api/v1/testkube/model_test_suite_execution_extended.go b/pkg/api/v1/testkube/model_test_suite_execution_extended.go
@@ -243,3 +243,17 @@ func (e *TestSuiteExecution) EscapeDots() *TestSuiteExecution {
 func (e *TestSuiteExecution) UnscapeDots() *TestSuiteExecution {
 	return e.convertDots(utils.UnescapeDots)
 }
+
+func (e *TestSuiteExecution) CleanStepsOutput() *TestSuiteExecution {
+	for i := range e.StepResults {
+		if e.StepResults[i].Execution != nil && e.StepResults[i].Execution.ExecutionResult != nil {
+			e.StepResults[i].Execution.ExecutionResult.Output = ""
+			for j := range e.ExecuteStepResults[i].Execute {
+				if e.ExecuteStepResults[i].Execute[j].Execution != nil && e.ExecuteStepResults[i].Execute[j].Execution.ExecutionResult != nil {
+					e.ExecuteStepResults[i].Execute[j].Execution.ExecutionResult.Output = ""
+				}
+			}
+		}
+	}
+	return e
+}
diff --git a/pkg/repository/testresult/mongo.go b/pkg/repository/testresult/mongo.go
@@ -222,12 +222,14 @@ func (r *MongoRepository) GetExecutions(ctx context.Context, filter Filter) (res
 
 func (r *MongoRepository) Insert(ctx context.Context, result testkube.TestSuiteExecution) (err error) {
 	result.EscapeDots()
+	result.CleanStepsOutput()
 	_, err = r.Coll.InsertOne(ctx, result)
 	return
 }
 
 func (r *MongoRepository) Update(ctx context.Context, result testkube.TestSuiteExecution) (err error) {
 	result.EscapeDots()
+	result.CleanStepsOutput()
 	_, err = r.Coll.ReplaceOne(ctx, bson.M{"id": result.Id}, result)
 	return
 }