optimizely / go-sdk / 20041008798

92%

master: 92%

Pull #429

github

Fix GetDetailedFeatureDecisionUnsafe to include holdout experiment/variation info

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%)

5630 of 6124 relevant lines covered (91.93%)

8191.99 hits per line