[oadp-1.5] Fix VolumeGroupSnapshotLabelKey test panic

[Joeavaikath]

Summary

Remove the VolumeGroupSnapshotLabelKey test case from TestValidateBackupSpecEnforcedFields that panics at runtime with reflect: call of reflect.Value.Set on zero Value.

Background

The VolumeGroupSnapshotLabelKey field was added to velero's BackupSpec in upstream vmware-tanzu/velero#8938, first appearing in velero v1.17.0. The oadp-1.5 branch uses the openshift/velero v1.16.x fork, which does not carry this field.

A test case referencing this field was introduced in #341 (commit aad3132, cherry-picked from #333). The test uses reflect.ValueOf(...).Elem().FieldByName("VolumeGroupSnapshotLabelKey") to set the field on a BackupSpec struct. Since the field doesn't exist in the v1.16.x fork, FieldByName returns a zero reflect.Value, and calling .Set() on it panics.

This was not caught at the time because CI did not trigger for the two most recent commits on oadp-1.5 (01c7191 on 2026-03-31 and aad3132 on 2026-05-11). The panic first surfaced in PR #355 (the rebase-bot PR), which was the first CI run to exercise this code path.

Test plan

• go test ./internal/common/function/... passes (including the Ensure_all_backup_spec_fields_were_tested guard)
• No other test cases modified

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 72724bd2-8504-450a-a78e-d7a2e7641aa5

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[oadp-snyk]

⛔ Snyk checks have failed. 6 issues have been found so far.

Status	Scan Engine	Critical	High	Medium	Low	Total (6)
⛔	Open Source Security	0	4	2	0	6 issues
✅	Licenses	0	0	0	0	0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

[Joeavaikath]

this will unblock #355 CI failure

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Joeavaikath, shubham-pampattiwar

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [shubham-pampattiwar]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[kaovilai]

For the Golang check, you need to update the GHA workflow.