Skip to content

test(java): activate STATIC/SPLIT reason tests (dd-trace-java#11037)#6717

Closed
typotter wants to merge 1 commit intotypo/activate-java-eval-testsfrom
typo/activate-java-eval-reason-tests
Closed

test(java): activate STATIC/SPLIT reason tests (dd-trace-java#11037)#6717
typotter wants to merge 1 commit intotypo/activate-java-eval-testsfrom
typo/activate-java-eval-reason-tests

Conversation

@typotter
Copy link
Copy Markdown
Contributor

@typotter typotter commented Apr 9, 2026

Summary

This activates two tests that were previously marked as bug (FFL-1972) in manifests/java.yml:

  • tests/ffe/test_flag_eval_metrics.py::Test_FFE_Eval_Metric_Basic::test_ffe_eval_metric_basic
  • tests/ffe/test_flag_eval_metrics.py::Test_FFE_Eval_Reason_Split::test_ffe_eval_reason_split

These tests cover STATIC and SPLIT flag evaluation reason reporting in eval metrics. They are fixed by DataDog/dd-trace-java#11037.

Dependencies

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

CODEOWNERS have been resolved as:

manifests/java.yml                                                      @DataDog/asm-java @DataDog/apm-java

@typotter typotter force-pushed the typo/activate-java-eval-reason-tests branch from 75cee22 to 380add0 Compare April 14, 2026 20:54
@typotter typotter force-pushed the typo/activate-java-eval-tests branch from 918d3cd to 0240b32 Compare April 16, 2026 17:11
@typotter typotter force-pushed the typo/activate-java-eval-reason-tests branch from ef3ae83 to fff86af Compare April 16, 2026 18:05
typotter added a commit that referenced this pull request Apr 16, 2026
Removes all remaining bug (FFL-1972) annotations. All 17 tests will
pass on v1.62.0 once the following dd-trace-java PRs ship:
- #11040: core eval metrics implementation (10 tests)
- #11037: STATIC/SPLIT reason (test_ffe_eval_metric_basic, test_ffe_eval_reason_split)
- #11036: type mismatch / parse error (3 tests)
- #11071: invalid regex parse error (1 test)
- #11072: metric count (1 test)

Supersedes #6717, #6718, #6723.
@typotter
Copy link
Copy Markdown
Contributor Author

Superseded by #6759 which consolidates all 17 FFE eval metric test activations into a single PR.

@typotter typotter closed this Apr 16, 2026
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