Merge pull request #109 from IABTechLab/ans-UID2-4347-remove-has-aborted-flag

Remove hasAborted variable

Author: ashleysmithTTD

src/integrationTests/basic.test.ts

@@ -953,10 +953,6 @@ testCookieAndLocalStorage(() => {
       expect(xhrMock.send).toHaveBeenCalledTimes(1);
       expect(xhrMock.abort).toHaveBeenCalledTimes(1);
     });
-    test('should prevent subsequent calls to init()', () => {
-      uid2.abort();
-      expect(() => uid2.init({ callback: () => {} })).toThrow();
-    });
   });
 
   describe('disconnect()', () => {
@@ -989,10 +985,6 @@ testCookieAndLocalStorage(() => {
       expect(xhrMock.send).toHaveBeenCalledTimes(1);
       expect(xhrMock.abort).toHaveBeenCalledTimes(1);
     });
-    test('should prevent subsequent calls to init()', () => {
-      uid2.disconnect();
-      expect(() => uid2.init({ callback: () => {} })).toThrow();
-    });
     test('should switch to unavailable state', () => {
       uid2.init({
         callback: callback,

src/sdkBase.ts

@@ -55,7 +55,6 @@ export abstract class SdkBase {
   private _opts: SdkOptions = {};
   private _identity: Identity | OptoutIdentity | null | undefined;
   private _initComplete = false;
-  private _hasAborted = false;
 
   // Sets up nearly everything, but does not run SdkLoaded callbacks - derived classes must run them.
   protected constructor(existingCallbacks: CallbackHandler[] | undefined, product: ProductDetails) {
@@ -176,7 +175,6 @@ export abstract class SdkBase {
 
   // Note: This doesn't invoke callbacks. It's a hard, silent reset.
   public abort(reason?: string) {
-    this._hasAborted = true;
     this._tokenPromiseHandler.rejectAllPromises(
       reason ?? new Error(`${this._product.name} SDK aborted.`)
     );
@@ -191,10 +189,6 @@ export abstract class SdkBase {
     if (!isSDKOptionsOrThrow(opts))
       throw new TypeError(`Options provided to ${this._product.name} init couldn't be validated.`);
 
-    if (this._hasAborted) {
-      throw new TypeError('Calling init() once aborted or disconnected is not allowed');
-    }
-
     if (this.isInitComplete()) {
       const previousOpts = { ...this._opts };
       Object.assign(this._opts, opts);