optimizely / go-sdk / 20074736653

92%

push

github

Fix GetDetailedFeatureDecisionUnsafe to include holdout experiment/variation info (#429)

This change fixes a bug where GetDetailedFeatureDecisionUnsafe did not return
experimentKey and variationKey for holdout decisions, only for feature tests.

Holdouts are control groups that need experiment/variation tracking for analytics,
similar to feature tests. This aligns the go-sdk with the Swift SDK reference
implementation, which includes experiment and variation info for all decision
sources except rollouts.

Changes:
- Updated client.go:878 to include decision.Holdout alongside decision.FeatureTest
- Added test case TestGetDetailedFeatureDecisionUnsafeWithHoldoutAndTrackingEnabled
- Updated comment to clarify that holdouts should include experiment/variation info

3 of 3 new or added lines in 1 file covered. (100.0%)

5631 of 6124 relevant lines covered (91.95%)

8522.0 hits per line