test: fix broken tests

Author: pranjal-jately-ld

packages/toolbar/src/core/tests/ExpandedToolbarContent.test.tsx

@@ -6,6 +6,7 @@ import { DevServerProvider } from '../ui/Toolbar/context/DevServerProvider';
 import { ToolbarUIProvider } from '../ui/Toolbar/context/ToolbarUIProvider';
 import { SearchProvider } from '../ui/Toolbar/context/SearchProvider';
 import { AnalyticsProvider } from '../ui/Toolbar/context/AnalyticsProvider';
+import { InternalClientProvider } from '../ui/Toolbar/context/InternalClientProvider';
 import { IEventInterceptionPlugin, IFlagOverridePlugin } from '../../types';
 
 // Create mock instances that we can access in tests
@@ -91,9 +92,11 @@ function TestWrapper({
           pollIntervalInMs: 5000,
         }}
       >
-        <AnalyticsProvider>
-          <SearchProvider>{children}</SearchProvider>
-        </AnalyticsProvider>
+        <InternalClientProvider>
+          <AnalyticsProvider>
+            <SearchProvider>{children}</SearchProvider>
+          </AnalyticsProvider>
+        </InternalClientProvider>
       </DevServerProvider>
     </ToolbarUIProvider>
   );
@@ -115,6 +118,8 @@ describe('ExpandedToolbarContent - User Interaction Flows', () => {
     onTabChange: vi.fn(),
     setSearchIsExpanded: vi.fn(),
     defaultActiveTab: 'settings' as const,
+    optInToNewFeatures: false,
+    onToggleOptInToNewFeatures: vi.fn(),
   };
 
   const createMockFlagOverridePlugin = (): IFlagOverridePlugin & {

packages/toolbar/src/core/tests/InternalClientProvider.test.tsx

@@ -85,7 +85,7 @@ describe('InternalClientProvider', () => {
   describe('Initialization', () => {
     test('initializes client successfully with valid clientSideID', async () => {
       render(
-        <InternalClientProvider clientSideID="test-client-id-123">
+        <InternalClientProvider clientSideId="test-client-id-123">
           <TestConsumer />
         </InternalClientProvider>,
       );
@@ -105,7 +105,7 @@ describe('InternalClientProvider', () => {
 
     test('calls setToolbarFlagClient on successful initialization', async () => {
       render(
-        <InternalClientProvider clientSideID="test-client-id-123">
+        <InternalClientProvider clientSideId="test-client-id-123">
           <TestConsumer />
         </InternalClientProvider>,
       );
@@ -117,7 +117,7 @@ describe('InternalClientProvider', () => {
 
     test('initializes with default anonymous context when no initialContext provided', async () => {
       render(
-        <InternalClientProvider clientSideID="test-client-id-123">
+        <InternalClientProvider clientSideId="test-client-id-123">
           <TestConsumer />
         </InternalClientProvider>,
       );
@@ -143,7 +143,7 @@ describe('InternalClientProvider', () => {
       };
 
       render(
-        <InternalClientProvider clientSideID="test-client-id-123" initialContext={customContext}>
+        <InternalClientProvider clientSideId="test-client-id-123" initialContext={customContext}>
           <TestConsumer />
         </InternalClientProvider>,
       );
@@ -159,7 +159,7 @@ describe('InternalClientProvider', () => {
       mockLDClient.waitForInitialization.mockRejectedValueOnce(initError);
 
       render(
-        <InternalClientProvider clientSideID="invalid-client-id">
+        <InternalClientProvider clientSideId="invalid-client-id">
           <TestConsumer />
         </InternalClientProvider>,
       );
@@ -179,7 +179,7 @@ describe('InternalClientProvider', () => {
       mockLDClient.waitForInitialization.mockRejectedValueOnce(new Error('Init failed'));
 
       render(
-        <InternalClientProvider clientSideID="invalid-client-id">
+        <InternalClientProvider clientSideId="invalid-client-id">
           <TestConsumer />
         </InternalClientProvider>,
       );
@@ -197,23 +197,21 @@ describe('InternalClientProvider', () => {
   describe('Custom Base URL Configuration', () => {
     test('configures SDK with custom baseUrl when provided', async () => {
       render(
-        <InternalClientProvider clientSideID="test-client-id-123" baseUrl="https://app.ld.catamorphic.com">
+        <InternalClientProvider clientSideId="test-client-id-123" baseUrl="https://app.ld.catamorphic.com">
           <TestConsumer />
         </InternalClientProvider>,
       );
 
       await waitFor(() => {
         expect(mockInitialize).toHaveBeenCalledWith('test-client-id-123', expect.any(Object), {
           baseUrl: 'https://app.ld.catamorphic.com',
-          streamUrl: 'https://clientstream.ld.catamorphic.com',
-          eventsUrl: 'https://events.ld.catamorphic.com',
         });
       });
     });
 
     test('does not provide options when baseUrl is not specified', async () => {
       render(
-        <InternalClientProvider clientSideID="test-client-id-123">
+        <InternalClientProvider clientSideId="test-client-id-123">
           <TestConsumer />
         </InternalClientProvider>,
       );
@@ -227,7 +225,7 @@ describe('InternalClientProvider', () => {
   describe('Cleanup', () => {
     test('clears singleton and closes client on unmount', async () => {
       const { unmount } = render(
-        <InternalClientProvider clientSideID="test-client-id-123">
+        <InternalClientProvider clientSideId="test-client-id-123">
           <TestConsumer />
         </InternalClientProvider>,
       );
@@ -246,7 +244,7 @@ describe('InternalClientProvider', () => {
   describe('useInternalClient Hook', () => {
     test('returns client, loading, and error state', async () => {
       render(
-        <InternalClientProvider clientSideID="test-client-id-123">
+        <InternalClientProvider clientSideId="test-client-id-123">
           <TestConsumer />
         </InternalClientProvider>,
       );
@@ -276,7 +274,7 @@ describe('InternalClientProvider', () => {
   describe('useInternalClientInstance Hook', () => {
     test('returns client instance when initialized', async () => {
       render(
-        <InternalClientProvider clientSideID="test-client-id-123">
+        <InternalClientProvider clientSideId="test-client-id-123">
           <TestClientInstanceConsumer />
         </InternalClientProvider>,
       );
@@ -288,7 +286,7 @@ describe('InternalClientProvider', () => {
 
     test('returns null before initialization', () => {
       render(
-        <InternalClientProvider clientSideID="test-client-id-123">
+        <InternalClientProvider clientSideId="test-client-id-123">
           <TestClientInstanceConsumer />
         </InternalClientProvider>,
       );