Skip to content

Conversation

@tanderson-ld
Copy link
Contributor

@tanderson-ld tanderson-ld commented Jan 21, 2026

Requirements

  • I have added test coverage for new or changed functionality
  • I have followed the repository's pull request submission guidelines
  • I have validated my changes against all supported platform versions

Related issues

SDK-1631

Describe the solution you've provided

Ports most of the dotnet logic for the TestService.java. Also adds payload filter query param handling to the FDv2 builders/sources so those filter tests pass.


Note

Introduces FDv2 payload filtering and data system wiring, plus contract test harness tweaks.

  • Adds payloadFilter support end-to-end: new setters on FDv2*Initializer/Synchronizer builders, DataSystemComponents plumbs it through, and DefaultFDv2Requestor now appends a filter= query param (with tests); SdkClientEntity consumes dataSystem.payloadFilter and configures FDv2 polling/streaming initializers/synchronizers accordingly
  • Expands contract test representations with dataSystem config types (initializers, synchronizers, store stubs, cache modes) and adds handling in SdkClientEntity (persistent store code left TODO)
  • Makefile: configurable TEST_SERVICE_PORT, introduce suppression files, update harness runs (v2 with -skip-from, add v3.0.0-alpha.1 run), and add contract-tests/test-suppressions-fdv2.txt

Written by Cursor Bugbot for commit c3e5110. This will update automatically on new commits. Configure here.

tanderson-ld and others added 30 commits January 12, 2026 11:47
…unchdarkly/java-core into rlamb/add-fdv2-data-source-interfaces
@tanderson-ld tanderson-ld requested a review from a team as a code owner January 21, 2026 21:21
@tanderson-ld tanderson-ld changed the title chore: adds fdv2 payload parsing and protocol handling chore: adds DataSystem contract test support Jan 21, 2026
@tanderson-ld tanderson-ld merged commit 342fc0a into main Jan 21, 2026
28 of 29 checks passed
@tanderson-ld tanderson-ld deleted the ta/SDK-1631/fdv2-contract-tests branch January 21, 2026 21:56
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.

3 participants