Skip to content

Commit fb6b363

Browse files
committed
Fix CodeRabbit review findings
- Remove unused primaryTests assignment (not used in execution flow) - Remove non-idiomatic closing brace comment - Add clarifying comment about primaryTests not being needed
1 parent f314b22 commit fb6b363

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

pkg/test/ginkgo/cmd_runsuite.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -622,7 +622,7 @@ func (o *GinkgoRunSuiteOptions) Run(suite *TestSuite, clusterConfig *clusterdisc
622622
netpolTests = []*testCase{}
623623
buildsTests = []*testCase{}
624624
mustGatherTests = []*testCase{}
625-
primaryTests = udnIterationTests
625+
// Note: primaryTests is not used in execution flow, no need to set it
626626
}
627627

628628
logrus.Infof("Found %d openshift tests", len(openshiftTests))

test/extended/networking/network_segmentation.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -510,7 +510,7 @@ var _ = Describe("[sig-network][OCPFeatureGate:NetworkSegmentation][Feature:User
510510
})),
511511
),
512512
)
513-
} // end iteration loop
513+
}
514514

515515
DescribeTable(
516516
"isolates overlapping CIDRs",

0 commit comments

Comments
 (0)