fix: simplify context lifecycle with defer cancel()

- Use defer cancel() for consistent context cleanup
- Removes manual cancel() calls in error paths
- Ensures context is always properly cleaned up
- Addresses SonarQube reliability concerns

Author: Jai Raj Choudhary

cmd/run.go

@@ -403,11 +403,11 @@ func (o *runOption) runSuite(loader testing.Loader, dataContext map[string]inter
 			o.limiter.Accept()
 
 			ctxWithTimeout, cancel := context.WithTimeout(ctx, o.requestTimeout)
+			defer cancel() // Ensure context is always cancelled when leaving this scope
 			ctxWithTimeout = context.WithValue(ctxWithTimeout, runner.ContextKey("").ParentDir(), loader.GetContext())
 
 			output, err = suiteRunner.RunTestCase(&testCase, dataContext, ctxWithTimeout)
 			if err = util.ErrorWrap(err, "failed to run '%s', %v", testCase.Name, err); err != nil {
-				cancel() // Cancel context before handling error
 				if o.requestIgnoreError {
 					errs = append(errs, err)
 				} else {
@@ -419,12 +419,10 @@ func (o *runOption) runSuite(loader testing.Loader, dataContext map[string]inter
 			reverseRunner.WithTestReporter(runner.NewDiscardTestReporter())
 			if _, err = reverseRunner.RunTestCase(
 				&testCase, dataContext, ctxWithTimeout); err != nil {
-				cancel() // Cancel context before returning error
 				err = fmt.Errorf("got error in reverse test: %w", err)
 				return
 			}
 			suiteRunner.WithTestReporter(o.reporter)
-			cancel() // Cancel context after successful completion
 		}
 		dataContext[testCase.Name] = output
 	}