optimizely / javascript-sdk

98%

master: 87%

push

travis-ci

fix (onReady): Clear ready timeout when instance closed, and other small fixes (#25)

Summary:

Previously, if close was called with pending ready timeouts, those timeouts would remain active. With this change, we clear all pending ready timeouts when close is called.

To guard against Promises returned from onReady remaining permanently pending, in close, we resolve any ready promises associated with pending timeouts with unsuccessful results. 

There are two other small changes in this PR:
- Improve documentation comments for Optimzely onReady and ProjectConfigManager onReady
- Return false from setForcedVariation when no configObj available (was previously returning null)

Test plan:

Updated unit tests, existing unit tests should continue passing

584 of 635 branches covered (91.97%)

Branch coverage included in aggregate %.

5707 of 5812 relevant lines covered (98.19%)

74.27 hits per line