Lint

Author: kinyoklion

packages/shared/sdk-client/__tests__/configuration/Configuration.test.ts

@@ -1,5 +1,5 @@
 /* eslint-disable no-console */
-import Configuration from '../../src/configuration/Configuration';
+import ConfigurationImpl from '../../src/configuration/Configuration';
 
 describe('Configuration', () => {
   beforeEach(() => {
@@ -8,7 +8,7 @@ describe('Configuration', () => {
   });
 
   it('has valid default values', () => {
-    const config = new Configuration();
+    const config = new ConfigurationImpl();
 
     expect(config).toMatchObject({
       allAttributesPrivate: false,
@@ -37,21 +37,21 @@ describe('Configuration', () => {
   });
 
   it('allows specifying valid wrapperName', () => {
-    const config = new Configuration({ wrapperName: 'test' });
+    const config = new ConfigurationImpl({ wrapperName: 'test' });
     expect(config).toMatchObject({ wrapperName: 'test' });
   });
 
   it('warns and ignored invalid keys', () => {
     // @ts-ignore
-    const config = new Configuration({ baseballUri: 1 });
+    const config = new ConfigurationImpl({ baseballUri: 1 });
 
     expect(config.baseballUri).toBeUndefined();
     expect(console.error).toHaveBeenCalledWith(expect.stringContaining('unknown config option'));
   });
 
   it('converts boolean types', () => {
     // @ts-ignore
-    const config = new Configuration({ sendEvents: 0 });
+    const config = new ConfigurationImpl({ sendEvents: 0 });
 
     expect(config.sendEvents).toBeFalsy();
     expect(console.error).toHaveBeenCalledWith(
@@ -61,7 +61,7 @@ describe('Configuration', () => {
 
   it('ignores wrong type for number and logs appropriately', () => {
     // @ts-ignore
-    const config = new Configuration({ capacity: true });
+    const config = new ConfigurationImpl({ capacity: true });
 
     expect(config.capacity).toEqual(100);
     expect(console.error).toHaveBeenCalledWith(
@@ -70,7 +70,7 @@ describe('Configuration', () => {
   });
 
   it('enforces minimum flushInterval', () => {
-    const config = new Configuration({ flushInterval: 1 });
+    const config = new ConfigurationImpl({ flushInterval: 1 });
 
     expect(config.flushInterval).toEqual(2);
     expect(console.error).toHaveBeenNthCalledWith(
@@ -80,14 +80,14 @@ describe('Configuration', () => {
   });
 
   it('allows setting a valid maxCachedContexts', () => {
-    const config = new Configuration({ maxCachedContexts: 3 });
+    const config = new ConfigurationImpl({ maxCachedContexts: 3 });
 
     expect(config.maxCachedContexts).toBeDefined();
     expect(console.error).not.toHaveBeenCalled();
   });
 
   it('enforces minimum maxCachedContext', () => {
-    const config = new Configuration({ maxCachedContexts: -1 });
+    const config = new ConfigurationImpl({ maxCachedContexts: -1 });
 
     expect(config.maxCachedContexts).toBeDefined();
     expect(console.error).toHaveBeenNthCalledWith(
@@ -103,15 +103,15 @@ describe('Configuration', () => {
     ['kebab-case-works'],
     ['snake_case_works'],
   ])('allow setting valid payload filter keys', (filter) => {
-    const config = new Configuration({ payloadFilterKey: filter });
+    const config = new ConfigurationImpl({ payloadFilterKey: filter });
     expect(config.payloadFilterKey).toEqual(filter);
     expect(console.error).toHaveBeenCalledTimes(0);
   });
 
   it.each([['invalid-@-filter'], ['_invalid-filter'], ['-invalid-filter']])(
     'ignores invalid filters and logs a warning',
     (filter) => {
-      const config = new Configuration({ payloadFilterKey: filter });
+      const config = new ConfigurationImpl({ payloadFilterKey: filter });
       expect(config.payloadFilterKey).toBeUndefined();
       expect(console.error).toHaveBeenNthCalledWith(
         1,

packages/shared/sdk-client/src/api/LDClient.ts

@@ -1,6 +1,5 @@
 import { LDContext, LDFlagSet, LDFlagValue, LDLogger } from '@launchdarkly/js-sdk-common';
 
-import ConnectionMode from './ConnectionMode';
 import { LDEvaluationDetail, LDEvaluationDetailTyped } from './LDEvaluationDetail';
 import { LDIdentifyOptions } from './LDIdentifyOptions';
 

packages/shared/sdk-client/src/api/LDOptions.ts

@@ -1,7 +1,5 @@
 import type { LDLogger } from '@launchdarkly/js-sdk-common';
 
-import ConnectionMode from './ConnectionMode';
-
 export interface LDOptions {
   /**
    * Whether all context attributes (except the context key) should be marked as private, and

packages/shared/sdk-client/src/context/addAutoEnv.ts

@@ -103,7 +103,11 @@ export const addDeviceInfo = async (platform: Platform) => {
   return undefined;
 };
 
-export const addAutoEnv = async (context: LDContext, platform: Platform, config: ConfigurationImpl) => {
+export const addAutoEnv = async (
+  context: LDContext,
+  platform: Platform,
+  config: ConfigurationImpl,
+) => {
   // LDUser is not supported for auto env reporting
   if (isLegacyUser(context)) {
     return context as LDUser;