feat: remove initialize from public API

BREAKING CHANGE: initialize is no longer part of the public API because it was not documented, unlikely to be used and confusing.
trustpilot · Jul 5, 2018 · 6a55505 · 6a55505
1 parent e977818
commit 6a55505
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 14 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -4,7 +4,6 @@ import {
     getCurrentTestVariation,
     getTest,
     getUserAgentInfo,
-    initialize,
     reset,
     setCurrentTestVariation,
     shouldShowUI,
@@ -47,7 +46,6 @@ export {
     reset,
     ui,
     SplitTest,
-    initialize,
 };
 
 export default {
@@ -57,7 +55,6 @@ export default {
     getCurrentTestVariation,
     getTest,
     getUserAgentInfo,
-    initialize,
     reset,
     setCurrentTestVariation,
     tests,

diff --git a/tests/config.spec.ts b/tests/config.spec.ts
@@ -20,15 +20,4 @@ describe('Config', () => {
         expect(config).toBeDefined();
         expect(config.cookieName).toEqual('Test');
     });
-
-    it('should keep configuration after multiple initialization', () => {
-        skift.config({
-            cookieName: 'Test',
-        });
-
-        skift.initialize();
-        skift.initialize();
-        expect(config).toBeDefined();
-        expect(config.cookieName).toEqual('Test');
-    });
 });