Open
Conversation
* Features: Created two new APIs to simplify the initialization process for mobile: - initializeAsync - initializeSync Also Added three unit tests in OptimizelyManager: -InitializeAsync -InitializeSync -InitializeSyncWithEmptyDataFile * Changes made to catch null pointer exception if given R.raw path of data file is null added assertnull for optimizelyStartListener in initializeAsync test that on completion it should be null * Added Empty datafile to pass unit test of check sync with empty datafile * Changes made to download file on initializing optimizely manager * - Renamed functions of inititalize sync and async - Refactored code to look more like ios Initialize methods (Except reading files in SDK instead of asking user to do that) * Removed unused functions and test cases Added comments did code refactoring * Removed initialize just from cache method. * - Changes made for asynchronus initialize to take downloaded file first than cache file - Added new test cases for empty datafile - refactored code to get datafile from cache if not available than from Raw data file * Commented unused loaddatafile variable * Removed unused function of loaddatafilefromcachetask * 1) Added boolean variable in initialized method to check if update using download should take place after initialization 2)updated comment in datafileloader.java 3) updated testcases 4) handled if datafile is null * 1) Added boolean variable in initialized method to check if update using download should take place after initialization (optimizely#11) 2)updated comment in datafileloader.java 3) updated testcases 4) handled if datafile is null * Updated Init test cases and resolved datafile issue of not found resource id * Bug fix of esspresso test of test app not running * Updated test cases and resolved bug of Raw id not found on release (optimizely#14) * Updated Init test cases and resolved datafile issue of not found resource id * Bug fix of esspresso test of test app not running * Updated comments and Allowed null resource ID in Initialize(Context,Integer,OptimizelyStartListener)
* use api 24 instead of 25. cirlceci is having issues right now.
…ariables (optimizely#168) * - Updated Java sdk version from 1.8 to 2.0.0-beta2 (optimizely#9) * - Updated Java sdk version from 1.8 to 2.0.0-beta2 - Removed getVariableString getVariableBoolean getVariableInteger getVariableDouble -Remove track with revenue as a parameter. Pass the revenue value as an event tag instead track(String, String, long) track(String, String, Map<String, String>, long) * added multidex enabled support * make common Constant of $multidex_ver to use in all gradle * Upgraded Java sdk (optimizely#13) * Excluded group: 'com.google.code.findbugs' from dependency of com.optimizely.ab:core-api removed Multidex enabling from sdk and test app * Updated Test unit tests * included Collection library which was removed due to merging of master * undo comment space
…tTest on android version 8.1.0. (optimizely#16) (optimizely#174)
optimizely#177) * Parameterized OptimizelyClientTest.java and Is feature enabled changes * Updated Test unit tests * parameterized optimizelyClientTest.java * updated getdatafile url method and removed Static (because its only usage was in unit tests where we already had optimizelymanager object) * space removed * Added forced variation unitTest and some refactoring * Code refactoring * Updated copyrights
* - Updated Java sdk version from 1.8 to 2.0.0-beta2 (optimizely#9) * - Updated Java sdk version from 1.8 to 2.0.0-beta2 - Removed getVariableString getVariableBoolean getVariableInteger getVariableDouble -Remove track with revenue as a parameter. Pass the revenue value as an event tag instead track(String, String, long) track(String, String, Map<String, String>, long) * added multidex enabled support * make common Constant of $multidex_ver to use in all gradle * Added Feature variables and its testcases (optimizely#12) * Added Feature Variables in optimizely client * Updated Client Test cases of feature Flags * Updated test cases of all Feature variables * Upgraded Java sdk (optimizely#13) * Excluded group: 'com.google.code.findbugs' from dependency of com.optimizely.ab:core-api removed Multidex enabling from sdk and test app * Updated Test unit tests * Feature variable changes asked (optimizely#22) * parameterized optimizelyClientTest.java * Updated test cases of feature variables and updated validProjectconfigv4 * Code refactoring * Feature variable changes asked * parameterized optimizelyClientTest.java * Updated test cases of feature variables and updated validProjectconfigv4 * Code refactoring * Refactored code
* integrate with optimizely java sdk 1.9 * update with bucketing id and notification listener tests * cleanup unused imports and comment on event payload case * added a couple more of listener unit tests
* upgrade gradle and build tools * cleanup lint errors * update to include android annotations
* update changelog to match release notes * update changelog to correct format for newly added releases
Added its unit tests updated java version to 2.0.0-beta6 updated datafile as it was not passing all unittests of due to featureEnabled was not set
* wrap dequeue with try catch in case of exception * fix for issue with Android O JobWorkService failing on dequeue * add a null pointer check
…ptimizely#195) * fix for blue apron by providing default logger if none is supplied. * add a lightweight slf4j logger that uses android native logging. use only if we get failures to load the logger from the LoggerFactory * add a use of lite logger if we fail with Error as well.
optimizely#196) * update to load the user profile service asynch and synch. remove invalid experiments if started asynch (latest datafile) or used cached datafile. * use a worker thread to clean up the user profile service * add try catch, cleanup unused imports in OptimizelyClient
* update for patch release 1.6.1 * update for changes to change log * don't patch user profile service start
* update for 2.0.0 release * add bug fix for JobWorkService * add license header * add other bug fixes to release 2.0.0 changelog * add github issue and fix date * use optimizely java sdk 2.0.1
optimizely#199) * fix for datafile download start when datafile has not changed or when starting synchronously * to avoid multiple starts, stop any service that is running before starting. * added some unit tests to make sure that backgrounding is started when interval is set
* bring back getVariable(String,Integer,Double,Boolean) * remove throws * revert * check for valid input to getVariable calls * log error on invalid user id or variable key for getVariable
* first version of environments for android * remove unused code * add comments and license. * add unit tests for environments * cleanup from Mike's comments. Still need to think of a better name than ProjectId * slight refactor to make it simpler and use data file config * add new datafile config * use environmentUrl * remove unused strings * update extra definition * update to latest design using project id or sdk key. sdk key trumps project id if both are provided. * update to reflect feedback and to cleanup code and comments * fix copyright * remove getCacheKey and just use getKey for DatafileConfig * update to test datafile config with environments * cleanup datafile config comments and add unit tests for initializing the manager with environment instead of project id * add new builder method that takes project id and sdk key. fix tests to use sdk key * have a builder with no arguments * rename some local variables * fix compiler erors after merging master * fix local variable name
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
18 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.