Skip to content

feat(flag-decisions): Add support for sending flag decisions along with decision metadata. #244

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
Oct 20, 2020
Merged

feat(flag-decisions): Add support for sending flag decisions along with decision metadata. #244

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
4cf5f43
Implemented flag decision logic
mnoman09 Sep 24, 2020
314f13b
added check that if SendFlagDecision is true then send the event
mnoman09 Sep 25, 2020
1bf73ea
- Added unit tests of featureDecision,
mnoman09 Sep 25, 2020
d82558d
Updated headers
mnoman09 Sep 25, 2020
8183bcc
passing null variation as variable
mnoman09 Sep 27, 2020
94a5821
unit test fix converted to snakecase
mnoman09 Sep 27, 2020
315aa55
removed invalid path decision metadata from cs16proj
mnoman09 Sep 28, 2020
823bdb2
Added test to check sendFlagDecision is false in case if it's missing…
mnoman09 Sep 29, 2020
719eb73
reverted changes made in this file
mnoman09 Sep 30, 2020
c6dc84f
Merge branch 'master' into mnoman/flag-decision
pawels-optimizely Oct 1, 2020
724a86b
Reverted removal of Activating user log and added condition to do n…
mnoman09 Oct 1, 2020
2aa8e8d
Renamed flag_type to rule_type and added rule_key
mnoman09 Oct 12, 2020
e5a4ae8
Added decision to rollout
mnoman09 Oct 13, 2020
e8db3cb
when variation is null then ruleType will be rollout
mnoman09 Oct 13, 2020
6c8d8f1
unit test fix
mnoman09 Oct 13, 2020
8c13a0e
Refact: made experiment constant
mnoman09 Oct 13, 2020
File filter

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
passing null variation as variable
  • Loading branch information
@mnoman09
mnoman09 committed Sep 27, 2020
commit 8183bcc7bc7d0f17b91f22f17fbe44e0bd234d9e
7 changes: 5 additions & 2 deletions OptimizelySDK.Tests/EventTests/EventFactoryTest.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,8 +57,9 @@ public void TestCreateImpressionEventReturnsNullWhenSendFlagDecisionsIsFalseAndI
public void TestCreateImpressionEventReturnsNullWhenSendFlagDecisionsIsFalseAndVariationIsNull()
{
Config.SendFlagDecisions = false;
Variation variation = null;
var impressionEvent = UserEventFactory.CreateImpressionEvent(
Config, Config.GetExperimentFromKey("test_experiment"), variation: null, TestUserId, null, "test_experiment", "experiment");
Config, Config.GetExperimentFromKey("test_experiment"), variation, TestUserId, null, "test_experiment", "experiment");
Assert.IsNull(impressionEvent);
}

Expand Down Expand Up @@ -592,7 +593,9 @@ public void TestCreateImpressionEventRemovesInvalidAttributesFromPayloadRollout(
{ "nan", double.NaN },
{ "invalid_num_value", Math.Pow(2, 53) + 2 },
};
var impressionEvent = UserEventFactory.CreateImpressionEvent(Config, null, variation: null, TestUserId, userAttributes, "test_feature", "rollout");
Variation variation = null;

var impressionEvent = UserEventFactory.CreateImpressionEvent(Config, null, variation, TestUserId, userAttributes, "test_feature", "rollout");
var logEvent = EventFactory.CreateLogEvent(impressionEvent, Logger);

TestData.ChangeGUIDAndTimeStamp(expectedLogEvent.Params, impressionEvent.Timestamp, Guid.Parse(impressionEvent.UUID));
Expand Down