Remove allowEviction

Author: aliu39

packages/browser/src/utils/featureFlags.ts

@@ -73,14 +73,12 @@ export function insertFlagToScope(name: string, value: unknown, maxSize: number
  * @param name       Name of the feature flag to insert.
  * @param value      Value of the feature flag.
  * @param maxSize    Max number of flags the buffer should store. Default value should always be used in production.
- * @param allowEviction  If true, the oldest flag is evicted when the buffer is full. Otherwise the new flag is dropped.
  */
 export function insertToFlagBuffer(
   flags: FeatureFlag[],
   name: string,
   value: unknown,
   maxSize: number,
-  allowEviction: boolean = true,
 ): void {
   if (typeof value !== 'boolean') {
     return;
@@ -100,12 +98,8 @@ export function insertToFlagBuffer(
   }
 
   if (flags.length === maxSize) {
-    if (allowEviction) {
-      // If at capacity, pop the earliest flag - O(n)
-      flags.shift();
-    } else {
-      return;
-    }
+    // If at capacity, pop the earliest flag - O(n)
+    flags.shift();
   }
 
   // Push the flag to the end - O(1)

packages/browser/test/utils/featureFlags.test.ts

@@ -59,25 +59,6 @@ describe('flags', () => {
       ]);
     });
 
-    it('drops new entries when allowEviction is false and buffer is full', () => {
-      const buffer: FeatureFlag[] = [];
-      const maxSize = 0;
-      insertToFlagBuffer(buffer, 'feat1', true, maxSize, false);
-      insertToFlagBuffer(buffer, 'feat2', true, maxSize, false);
-      insertToFlagBuffer(buffer, 'feat3', true, maxSize, false);
-
-      expect(buffer).toEqual([]);
-    });
-
-    it('still updates order and values when allowEviction is false and buffer is full', () => {
-      const buffer: FeatureFlag[] = [];
-      const maxSize = 1;
-      insertToFlagBuffer(buffer, 'feat1', false, maxSize, false);
-      insertToFlagBuffer(buffer, 'feat1', true, maxSize, false);
-
-      expect(buffer).toEqual([{ flag: 'feat1', result: true }]);
-    });
-
     it('does not allocate unnecessary space', () => {
       const buffer: FeatureFlag[] = [];
       const maxSize = 1000;