Skip to content

(DO NOT REVIEW) Test decide #412

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

Jump to bottom
Closed
wants to merge 52 commits into from
Closed

(DO NOT REVIEW) Test decide #412

wants to merge 52 commits into from

Conversation

jaeopt
Copy link
Contributor

@jaeopt jaeopt commented Dec 11, 2020

Summary

  • The "what"; a concise description of each logical change
  • Another change

The "why", or other context.

Test plan

Issues

jaeopt added 30 commits October 5, 2020 16:45
@jaeopt
add OptimizelyUserContext
9f6c40c
@jaeopt
add decide api
d476539
@jaeopt
add options and reasons
8bea738
@jaeopt
add decide-all api
6800e66
@jaeopt
fix errors
642f957
@jaeopt
fix merge conflicts
e8745ef
@jaeopt
change user-context nonnull
d4e2b1c
@jaeopt
Merge branch 'jae/user-context' into jae/decide-api
2d426a5
@jaeopt
fix existing tests for decision service
b8a19a7
@jaeopt
add more decide tests
e3cedab
@jaeopt
add more tests
2c83fa2
@jaeopt
change ruleKey to have a copy of experimentKey
bb9f4dc
@jaeopt
add all tests (except for reasons)
3223a64
@jaeopt
add more tests
7792926
@jaeopt
fix per reviews
31cce4f
@jaeopt
fix conflicts
77e09d5
@jaeopt
fix conflicts
4bfa1f7
@jaeopt
fix to nonnull options and reasons
fdbf75a
@jaeopt
remove support for custom decisionservice
c0b6c85
@jaeopt
thread-safe setAttribute
a93fc23
@jaeopt
add option and reasons to condition evaluation
f803cd7
@jaeopt
add more tests for reasons
43f7b01
@jaeopt
pass options to DecisonReasons
7ce768d
@jaeopt
move decide core to optimizely
5b6e798
@jaeopt
change variables in decision nonnull
29d3de4
@jaeopt
change withDefaultDecideOptions per reviews
a5954e4
@jaeopt
clean up addInfo
6f40cb0
@jaeopt
remove unused evaluate method
0ecc4b6
@jaeopt
add toString to OptimizelyUserContext and OptimizelyDecision
28493df
@jaeopt
check null userId for createUserContext
fe6cde5
jaeopt and others added 19 commits October 29, 2020 15:14
@jaeopt
merge flag-decisions
9933238
@jaeopt
clean up decision options
82e9036
@jaeopt
clean up decide
73668a9
@jaeopt
remove options propagation from tests
e0cb8fb
@jaeopt
add DecisionReasons interface
4c68f74
@jaeopt @mnoman09
Update core-api/src/main/java/com/optimizely/ab/Optimizely.java
0fbe851 
Co-authored-by: mnoman09 <Muhammadnoman@folio3.com>
@jaeopt @mnoman09
Update core-api/src/main/java/com/optimizely/ab/OptimizelyUserContext…
b3165f6 
….java

Co-authored-by: mnoman09 <Muhammadnoman@folio3.com>
@jaeopt
Merge branch 'jae/user-context' of github.com:optimizely/java-sdk int…
a1202c3 
…o jae/user-context
@jaeopt
fix convenience methods in user context
de508fb
@jaeopt
Merge branch 'master' into jae/user-context
b9e9e34
@jaeopt
add featureEnabled to metadata for decide-api
dd47439
@jaeopt
add metadata validation for decide-api
af0f1f9
@jaeopt
add more tests for decide-api event validations
4a35f33
@jaeopt
add more tests for decide-api event validations
21c2a9d
@jaeopt
Merge branch 'master' into jae/user-context
f0de871
@jaeopt
fix import
ff67de0
@jaeopt
change decision logs to infos
70373c1
@jaeopt
Merge branch 'master' into jae/user-context
819b7db
@jaeopt
insert errors for testing
342e8ca
@jaeopt jaeopt requested a review from a team as a code owner December 11, 2020 18:20
@jaeopt jaeopt self-assigned this Dec 11, 2020
@jaeopt jaeopt removed the request for review from a team December 11, 2020 19:32
@jaeopt jaeopt closed this Dec 11, 2020
@jaeopt jaeopt reopened this Dec 11, 2020
@jaeopt jaeopt closed this Dec 11, 2020
@jaeopt jaeopt deleted the test-decide branch December 11, 2020 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jaeopt jaeopt

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jaeopt