fix: add missing t.Helper() to renderSRIOV wrapper

[andrewwhitecdw]

This PR addresses the following issue in pkg/networkoperatorplugin/sriov_render_test.go: add missing t.Helper() to renderSRIOV wrapper.

Changes

• pkg/networkoperatorplugin/sriov_render_test.go: add missing t.Helper() to renderSRIOV wrapper.

Details

--- a/pkg/networkoperatorplugin/sriov_render_test.go
+++ b/pkg/networkoperatorplugin/sriov_render_test.go
@@ -1,3 +1,4 @@
-func renderSRIOV(t *testing.T, dir, fabric, networkTemplate string) map[string]string {
-	return renderSRIOVWithNamespaces(t, dir, fabric, networkTemplate, nil)
-}
+func renderSRIOV(t *testing.T, dir, fabric, networkTemplate string) map[string]string {
+	t.Helper()
+	return renderSRIOVWithNamespaces(t, dir, fabric, networkTemplate, nil)
+}

Tests

Let me know if you want tests added for this fix or not.

Contributor guidelines

Per this repo's CONTRIBUTING.md:

• All commits are signed off (Signed-off-by trailer, DCO).

[greptile-apps]

Greptile Summary

This PR improves Go test failure attribution by marking renderSRIOV as a test helper.

• Adds t.Helper() before delegating to renderSRIOVWithNamespaces.
• Does not alter rendering behavior or production code.

Confidence Score: 5/5

The PR appears safe to merge with no actionable issues identified.

The change only adjusts Go test failure attribution and leaves the helper’s rendering behavior unchanged.

Important Files Changed

Filename	Overview
pkg/networkoperatorplugin/sriov_render_test.go	Correctly marks the renderSRIOV wrapper as a test helper so failures are attributed to its callers.

_{Reviews (1): Last reviewed commit: "fix: add missing t.Helper() to renderSRI..." | Re-trigger Greptile}