Skip to content

test: demonstrate fixture validation failure - #25

Closed
leoromanovsky wants to merge 1 commit into
mainfrom
agent/demo-broken-fixture-validation
Closed

test: demonstrate fixture validation failure#25
leoromanovsky wants to merge 1 commit into
mainfrom
agent/demo-broken-fixture-validation

Conversation

@leoromanovsky

Copy link
Copy Markdown
Collaborator

Motivation

Demonstrate that the static fixture validator introduced by #24 rejects a semantically invalid evaluation case before it can merge.

Changes

  • Add a syntactically valid evaluation case that references a nonexistent flag.
  • Omit the FLAG_NOT_FOUND expectation required for an unknown flag reference.

Decisions

  • Stack this PR directly on agent/informational-downstream-compatibility so the diff contains only the broken fixture.
  • Keep this PR deliberately broken and in draft. It is a CI demonstration and must not be merged.
  • Expect the blocking fixture-validation check to fail; downstream SDK compatibility checks remain informational.

Validation

  • python3 -m json.tool evaluation-cases/test-case-validator-demo-broken-reference.json passes.
  • python3 ci/validate-fixtures.py fails with: case 0 references unknown flag 'validator-demo-missing-flag' without a FLAG_NOT_FOUND expectation.

Add a syntactically valid evaluation case with an unknown flag reference.

Environment: Datadog workspace
Base automatically changed from agent/informational-downstream-compatibility to main August 10, 2026 20:23
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