optimizely / go-sdk / 24850128801

Builds Branch Commit Type Ran Committer Via Coverage
24850128801 master [FSSDK-12368] Remove legacy flag-level holdout fields (#446) Remove deprecated includedFlags and excludedFlags from Holdout entity and simplify mapper logic to treat all holdouts as global. - Removed IncludedFlags and ExcludedFlags fields from e... push web-flow github
91.85
24733908509 ai/mat001/FSSDK-12368-legacy-holdout-cleanup [FSSDK-12368] Remove legacy flag-level holdout fields Remove deprecated includedFlags and excludedFlags from Holdout entity and simplify mapper logic to treat all holdouts as global. - Removed IncludedFlags and ExcludedFlags fields from entities... Pull #446 Mat001 github
91.85
24527143563 ai/mat001/FSSDK-12368-mpirnovar-ai-flow-sdk-fssdk-12368 [FSSDK-12368] Add test coverage for holdout service and config methods - Added TestEvaluateLocalHoldout to test EvaluateLocalHoldout method - Added TestEvaluateLocalHoldoutNotRunning to test non-running holdouts - Added TestGetHoldoutsForRule* te... Pull #445 Mat001 github
91.87
24526653609 ai/mat001/FSSDK-12368-mpirnovar-ai-flow-sdk-fssdk-12368 [FSSDK-12368] Add test coverage for local holdouts in rollout service - Added TestGetDecisionWithLocalHoldout to test local holdout path in rollout rules - Reuses mockProjectConfigWithHoldouts from feature_experiment_service_test.go - Covers loca... Pull #445 Mat001 github
91.68
24526528655 ai/mat001/FSSDK-12368-mpirnovar-ai-flow-sdk-fssdk-12368 [FSSDK-12368] Add test coverage for Holdout.IsGlobal() method - Created experiment_test.go with tests for Holdout.IsGlobal() - Tests nil IncludedRules (global holdout) returns true - Tests non-nil IncludedRules (local holdout) returns false - Tes... Pull #445 Mat001 github
91.57
24526067506 ai/mat001/FSSDK-12368-mpirnovar-ai-flow-sdk-fssdk-12368 [FSSDK-12368] Add test for multiple local holdouts iteration - Added TestGetDecisionWithMultipleLocalHoldouts to test loop iteration through multiple holdouts - Tests that code correctly continues to next holdout when first one doesn't bucket use... Pull #445 Mat001 github
91.53
24525278441 ai/mat001/FSSDK-12368-mpirnovar-ai-flow-sdk-fssdk-12368 [FSSDK-12368] Add test for local holdout not bucketing user - Added TestGetDecisionWithLocalHoldoutNotBucketed to test path where user doesn't bucket into holdout - Tests that code continues to normal experiment bucketing when holdout returns nil... Pull #445 Mat001 github
91.53
24489323638 ai/mat001/FSSDK-12368-mpirnovar-ai-flow-sdk-fssdk-12368 [FSSDK-12368] Add test coverage for local holdout evaluation in feature experiments - Added TestGetDecisionWithLocalHoldout to test local holdout path - Created mockProjectConfigWithHoldouts to override GetHoldoutsForRule - Coverage for feature_e... Pull #445 Mat001 github
91.53
24487028511 ai/mat001/FSSDK-12368-mpirnovar-ai-flow-sdk-fssdk-12368 [FSSDK-12368] Refactor local holdout evaluation to eliminate code duplication - Moved shared local holdout evaluation logic to HoldoutService.EvaluateLocalHoldout() - Updated RolloutService.evaluateHoldout() to delegate to shared method - Updated... Pull #445 Mat001 github
91.34
24486946383 ai/mat001/FSSDK-12368-mpirnovar-ai-flow-sdk-fssdk-12368 [FSSDK-12368] Revert broken local holdout tests Pull #445 Mat001 github
90.49
24480305983 ai/mat001/FSSDK-12368-mpirnovar-ai-flow-sdk-fssdk-12368 [FSSDK-12368] Improve test coverage for local holdout evaluation Added 4 new tests: - TestEvaluateHoldoutAudienceFails: User doesn't meet holdout audience conditions - TestEvaluateHoldoutNotBucketed: User not bucketed (0% traffic) - TestEvaluateH... Pull #445 Mat001 github
90.47
24474736094 ai/mat001/FSSDK-12368-mpirnovar-ai-flow-sdk-fssdk-12368 [FSSDK-12368] Add tests for local holdout evaluation - Added TestEvaluateHoldoutNotRunning: tests non-running holdouts are skipped - Added TestEvaluateHoldoutRunningNoAudience: tests holdout bucketing with 100% traffic - Improved evaluateHoldout ... Pull #445 Mat001 github
90.35
24472200787 ai/mat001/FSSDK-12368-mpirnovar-ai-flow-sdk-fssdk-12368 [FSSDK-12368] Fix gofmt and reduce cyclomatic complexity - Fixed gofmt formatting (removed trailing newline) - Extracted evaluateEveryoneElseRule method to reduce GetDecision complexity from 18 to ≤16 - All tests passing - Decision package lintin... Pull #445 Mat001 github
89.58
24471772103 ai/mat001/FSSDK-12368-mpirnovar-ai-flow-sdk-fssdk-12368 [FSSDK-12368] Fix rollout service test failures - Changed continue to break when user fails bucketing (matches original behavior) - Restored experimentDecisionContext creation before audience check - Used finalFeatureDecision variable instead of ... Pull #445 Mat001 github
89.56
24422173128 ai/mat001/FSSDK-12368-mpirnovar-ai-flow-sdk-fssdk-12368 [FSSDK-12368] Implement Local Holdouts support Add Local Holdouts support to replace legacy flag-level holdouts with rule-level targeting. Changes: - Add IncludedRules field to Holdout struct (replaces IncludedFlags/ExcludedFlags) - Add IsGlobal... Pull #445 Mat001 github
89.56