Skip to content

Remove duplicate TestTransformValidator and fix test placement#2383

Open
Pritiks23 wants to merge 1 commit intoNVIDIA:mainfrom
Pritiks23:add-validator-transform-test
Open

Remove duplicate TestTransformValidator and fix test placement#2383
Pritiks23 wants to merge 1 commit intoNVIDIA:mainfrom
Pritiks23:add-validator-transform-test

Conversation

@Pritiks23
Copy link
Copy Markdown

Description

This PR began as an effort to add a new unit test for the [TransformValidator]function. During implementation, I discovered that a test with the same name already existed, causing a redeclaration error. This PR resolves the duplication, ensures the test is correctly placed, and confirms that the validator logic is covered by unit tests.

Checklist

  • No secrets, sensitive information, or unrelated changes
  • Lint checks passing (make lint)
  • Generated assets in-sync (make validate-generated-assets)
  • Go mod artifacts in-sync (make validate-modules)
  • Test cases are added for new code paths

Testing

Ran go test //controllers -run TestTransformValidator -v to verify the validator test passes
Ran the full test suit to confirm no regressions

@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Apr 20, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant