working on tests

Author: roncohen

packages/browser-sdk/src/client.ts

@@ -775,10 +775,10 @@ export class BucketClient {
       },
       isEnabledOverride: this.featuresClient.getFeatureOverride(key),
       setIsEnabledOverride(isEnabled: boolean | null) {
-        fClient.setFeatureOverride(key, isEnabled);
+        self.featuresClient.setFeatureOverride(key, isEnabled);
       },
       setInUse(inUse: boolean) {
-        fClient.setInUse(key, inUse);
+        self.featuresClient.setInUse(key, inUse);
       },
     };
   }

packages/browser-sdk/test/features.test.ts

@@ -265,6 +265,7 @@ describe("FeaturesClient", () => {
         key: "featureB",
         targetingVersion: 1,
         isEnabledOverride: null,
+        inUse: false,
       } satisfies RawFeature,
     });
 
@@ -304,6 +305,7 @@ describe("FeaturesClient", () => {
           targetingVersion: 1,
           key: "featureA",
           isEnabledOverride: null,
+          inUse: false,
         } satisfies RawFeature,
       }),
     );

packages/browser-sdk/test/mocks/handlers.ts

@@ -36,6 +36,7 @@ export const featuresResult = Object.entries(featureResponse.features).reduce(
       ...feature!,
       config: feature.config,
       isEnabledOverride: null,
+      inUse: false,
     };
     return acc;
   },

packages/browser-sdk/test/usage.test.ts

@@ -80,6 +80,9 @@ describe("usage", () => {
       track: expect.any(Function),
       requestFeedback: expect.any(Function),
       config: { key: undefined, payload: undefined },
+      isEnabledOverride: null,
+      setInUse: expect.any(Function),
+      setIsEnabledOverride: expect.any(Function),
     });
   });
 
@@ -413,6 +416,9 @@ describe(`sends "check" events `, () => {
         config: { key: undefined, payload: undefined },
         track: expect.any(Function),
         requestFeedback: expect.any(Function),
+        isEnabledOverride: null,
+        setInUse: expect.any(Function),
+        setIsEnabledOverride: expect.any(Function),
       });
 
       expect(client.getFeature("featureB")).toStrictEqual({
@@ -426,13 +432,19 @@ describe(`sends "check" events `, () => {
         },
         track: expect.any(Function),
         requestFeedback: expect.any(Function),
+        isEnabledOverride: null,
+        setInUse: expect.any(Function),
+        setIsEnabledOverride: expect.any(Function),
       });
 
       expect(client.getFeature("featureC")).toStrictEqual({
         isEnabled: false,
         config: { key: undefined, payload: undefined },
         track: expect.any(Function),
         requestFeedback: expect.any(Function),
+        isEnabledOverride: null,
+        setInUse: expect.any(Function),
+        setIsEnabledOverride: expect.any(Function),
       });
     });
 
@@ -564,9 +576,12 @@ describe(`sends "check" events `, () => {
       const featureId1 = client.getFeature("featureId1");
       expect(featureId1).toStrictEqual({
         isEnabled: false,
+        isEnabledOverride: null,
         track: expect.any(Function),
         requestFeedback: expect.any(Function),
         config: { key: undefined, payload: undefined },
+        setInUse: expect.any(Function),
+        setIsEnabledOverride: expect.any(Function),
       });
 
       vi.spyOn(client, "track");
@@ -586,6 +601,9 @@ describe(`sends "check" events `, () => {
         track: expect.any(Function),
         requestFeedback: expect.any(Function),
         config: { key: undefined, payload: undefined },
+        isEnabledOverride: null,
+        setIsEnabledOverride: expect.any(Function),
+        setInUse: expect.any(Function),
       });
 
       vi.spyOn(client, "requestFeedback");