clean up CookieConsentManager constructor and update tests

Author: jaissica12

src/cookieConsentManager.ts

@@ -39,15 +39,11 @@ export interface ICookieConsentManager {
  * unless explicitly opted out by the user.
  */
 export default class CookieConsentManager implements ICookieConsentManager {
-    private flags: ICookieConsentFlags = {
-        noFunctional: false,
-        noTargeting: false,
-    };
-
-    constructor(flags: Partial<ICookieConsentFlags> = {}) {
-        const { noFunctional, noTargeting } = flags;
-        this.flags.noFunctional = noFunctional === true;
-        this.flags.noTargeting = noTargeting === true;
+    constructor(private flags: ICookieConsentFlags) {
+        this.flags = {
+            noFunctional: flags.noFunctional === true,
+            noTargeting: flags.noTargeting === true,
+        };
     }
 
     /**

test/jest/cookieConsentManager.spec.ts

@@ -3,7 +3,7 @@ import CookieConsentManager from '../../src/cookieConsentManager';
 describe('CookieConsentManager', () => {
     describe('#constructor', () => {
         it('should default flags to false when not provided', () => {
-            const cookieConsentManager = new CookieConsentManager();
+            const cookieConsentManager = new CookieConsentManager({ noFunctional: undefined, noTargeting: undefined });
 
             expect(cookieConsentManager.getNoFunctional()).toBe(false);
             expect(cookieConsentManager.getNoTargeting()).toBe(false);
@@ -29,15 +29,15 @@ describe('CookieConsentManager', () => {
 
     describe('#getNoFunctional', () => {
         it('should return the noFunctional flag value', () => {
-            const cookieConsentManager = new CookieConsentManager({ noFunctional: true });
+            const cookieConsentManager = new CookieConsentManager({ noFunctional: true, noTargeting: undefined });
 
             expect(cookieConsentManager.getNoFunctional()).toBe(true);
         });
     });
 
     describe('#getNoTargeting', () => {
         it('should return the noTargeting flag value', () => {
-            const cookieConsentManager = new CookieConsentManager({ noTargeting: true });
+            const cookieConsentManager = new CookieConsentManager({ noTargeting: true, noFunctional: undefined });
 
             expect(cookieConsentManager.getNoTargeting()).toBe(true);
         });

test/jest/cookieSyncManager.spec.ts

@@ -454,7 +454,7 @@ describe('CookieSyncManager', () => {
             });
 
             it('should allow cookie sync when noTargeting is false by default', () => {
-                const cookieConsentManager = new CookieConsentManager(); // Defaults to noTargeting: false
+                const cookieConsentManager = new CookieConsentManager({ noTargeting: undefined, noFunctional: undefined }); // Defaults to noTargeting: false
 
                 const mockMPInstance = ({
                     _Store: {