optimizely / javascript-sdk / 862

78%

push

travis-ci

fix (datafile manager): Rename top-level exports to avoid conflict with interface name (#291)

Summary:

The interface DatafileManager, defined in src/datafileManager.ts, is exported from src/index.node.ts and src/index.browser.ts.
However, in each of these top-level entry points, another export statement appears below, which exports something else (the actual implementation of the datafile manager), using the same name.
To avoid the name conflict, we rename the exports of the datafile manager implementations from DatafileManager to HttpPollingDatafileManager.

Test plan:

Updated unit tests

582 of 633 branches covered (91.94%)

Branch coverage included in aggregate %.

5761 of 5866 relevant lines covered (98.21%)

92.52 hits per line