Skip to content

[FFL-1795] Validate rejected dependency exposure not logged - #58

Merged
typotter merged 1 commit into
mainfrom
test/dependent-condition-false-exposure
Oct 30, 2026
Merged

typotter merged 1 commit into
mainfrom
test/dependent-condition-false-exposure

Conversation

@typotter

Copy link
Copy Markdown
Contributor

Summary

  • add telemetry expectations to the existing ONE_OF miss and NOT_ONE_OF hit dependent-flag cases
  • retain prerequisite evaluation events while suppressing prerequisite exposures when that dependency condition rejects the containing allocation
  • require the selected root fallback to be the only exposure via noUnmatchedEvents

This closes the remaining contract gap called out in Helen’s review: the prerequisite is evaluated to determine that the condition is false, but no targeting rule is selected, so its exposure is not committed.

Validation

Cover ONE_OF and NOT_ONE_OF dependency conditions whose evaluated prerequisite variant rejects the containing allocation. Preserve evaluation events while requiring that only the selected root fallback emits an exposure.

Environment: Datadog workspace

Co-Authored-By: Codex GPT-5 <noreply@localhost>
@typotter
typotter requested a review from a team as a code owner October 26, 2026 16:52
@linear-code

linear-code Bot commented Oct 26, 2026

Copy link
Copy Markdown

FFL-1795

@typotter typotter changed the title [FFL-1795] Assert rejected dependency exposure suppression [FFL-1795] Validate rejected dependency exposure not logged Oct 27, 2026
@typotter
typotter requested a review from hhan2024 October 27, 2026 02:30
@typotter
typotter merged commit 33f2f84 into main Oct 30, 2026
1 check passed
@typotter
typotter deleted the test/dependent-condition-false-exposure branch October 30, 2026 14:39
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.

2 participants