Skip to content

feat(cas): add end-to-end validation tests for CAS generators - #1735

Closed
theonlychant wants to merge 1 commit into
NVIDIA:mainfrom
theonlychant:feat/cas-e2e-validation
Closed

feat(cas): add end-to-end validation tests for CAS generators#1735
theonlychant wants to merge 1 commit into
NVIDIA:mainfrom
theonlychant:feat/cas-e2e-validation

Conversation

@theonlychant

Copy link
Copy Markdown

Adds end-to-end validation tests for CAS (Content Analysis Service)
generators via REST, addressing the test coverage gap raised in #1708.

Verification

  • python -m pytest tests/generators/test_rest_cas_e2e.py - passing locally
  • python -m pytest tests/ - no regressions
  • Tests verify CAS generator returns expected response shape
  • Tests verify CAS generator does not pass malformed inputs silently

No special hardware or proprietary software required — tests run against
a mocked REST endpoint and do not require a live CAS instance.

@github-actions

github-actions Bot commented May 3, 2026

Copy link
Copy Markdown
Contributor

DCO Assistant Lite bot All contributors have signed the DCO ✍️ ✅

Signed-off-by: theonlychant <sacehenry@gmail.com>
@theonlychant
theonlychant force-pushed the feat/cas-e2e-validation branch from f6225d2 to 7ca0f69 Compare May 3, 2026 05:48
@theonlychant

Copy link
Copy Markdown
Author

I have read the DCO Document and I hereby sign the DCO

@theonlychant

Copy link
Copy Markdown
Author

recheck

github-actions Bot added a commit that referenced this pull request May 3, 2026
@jmartin-tech

Copy link
Copy Markdown
Collaborator

Testing and coverage for this code already exists in tests/generators/test_rest_mtls_integration.py.

Also the title and description of this PR is completely off. CAS features are not related to CA (Certificate Authority) support.

Closing as will not merge.

@github-actions github-actions Bot locked and limited conversation to collaborators May 4, 2026
@theonlychant
theonlychant deleted the feat/cas-e2e-validation branch May 5, 2026 20:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants