test(auditor): re-enable tests

[parkanzky]

Previously disabled some e2e tests that depended on the Auditor image being rebuilt. Re-enabling to test the updated image.

Summary by CodeRabbit

• Tests
  • Re-enabled end-to-end audit job submission coverage.
  • Tests now submit audit jobs, wait for completion, and clean up created jobs automatically.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: ed731fd6-a9b3-4f47-8d38-fb15db49ab38

📥 Commits

Reviewing files that changed from the base of the PR and between 61670b5 and c5bab9e.

📒 Files selected for processing (1)

• e2e/auditor/test_audit_job.py

💤 Files with no reviewable changes (1)

• e2e/auditor/test_audit_job.py

---

📝 Walkthrough

Walkthrough

Two skipped auditor audit-job E2E tests are re-enabled. They submit jobs, poll for terminal status, and cancel/delete jobs during cleanup.

Changes

Auditor audit-job execution

Layer / File(s)	Summary
Enable audit-job submission coverage e2e/auditor/test_audit_job.py	Removes skip markers from blank-probe and entity-reference audit-job tests, enabling submission, polling, and cleanup execution.

Possibly related PRs

• NVIDIA-NeMo/nemo-platform#758: Re-enables auditor audit-job E2E submission tests tied to audit job API wiring.
• NVIDIA-NeMo/nemo-platform#794: Re-enables the same two audit-job submission tests in the same module.

Suggested reviewers: mmogallapalli

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change: re-enabling Auditor tests.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch AIRCORE-831/re-enable-e2e-pparkanzky

---

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

[github-actions]

Suite	Lines Covered	Line Rate	Branch Rate
Unit Tests	27113/34832	77.8%	62.1%
Integration Tests	15891/33443	47.5%	19.9%

[parkanzky]

Digging in to why the tests fail on CI. All of QA's E2E tests have passed with the latest image, so something is different here.