fix(workspace): update the logging guard to handle ConnectionString for Application Insights

frontend/packages/shared/src/contexts/LoggerContext.test.tsx

@@ -0,0 +1,46 @@
+import React from 'react';
+import { render } from '@testing-library/react';
+import { LoggerContextProvider, type LoggerContextProviderProps } from './LoggerContext';
+import { ApplicationInsights } from '@microsoft/applicationinsights-web';
+
+jest.mock('@microsoft/applicationinsights-web', () => {
+  const mockTrackException = jest.fn();
+  return {
+    ApplicationInsights: jest.fn().mockImplementation(() => ({
+      loadAppInsights: jest.fn(),
+      trackException: mockTrackException,
+    })),
+    mockTrackException,
+  };
+});
+
+describe('LoggerContextProvider', () => {
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+
+  test('does not initialize ApplicationInsights without connectionString', () => {
+    const emptyConfig = {};
+    renderLoggerContext({ config: emptyConfig });
+
+    expect(ApplicationInsights).not.toHaveBeenCalled();
+  });
+
+  test('does initialize ApplicationInsights when connectionString is provided', () => {
+    renderLoggerContext({
+      config: {
+        connectionString: 'my-unit-test-connection-string',
+      },
+    });
+
+    expect(ApplicationInsights).toHaveBeenCalled();
+  });
+});
+
+const renderLoggerContext = (props?: Pick<LoggerContextProviderProps, 'config'>): void => {
+  render(
+    <LoggerContextProvider config={{ ...props.config }}>
+      <div>child</div>
+    </LoggerContextProvider>,
+  );
+};

frontend/packages/shared/src/contexts/LoggerContext.tsx

@@ -8,7 +8,7 @@ export type LoggerConfig = IConfiguration & IConfig;
 
 const LoggerContext = createContext<ApplicationInsights | null>(null);
 
-type LoggerContextProviderProps = {
+export type LoggerContextProviderProps = {
   config: LoggerConfig;
   children: ReactNode;
 };
@@ -19,8 +19,8 @@ export const LoggerContextProvider = ({
   const reactPlugin = useMemo(() => new ReactPlugin(), []);
 
   const applicationInsights = useMemo(() => {
-    // check if we have a instrumentationKey, if not, don't initialize app insights (we do not want AI to run on localhost)
-    if (!config.instrumentationKey) return null;
+    // check if we have a connectionString, if not, don't initialize app insights (we do not want AI to run on localhost)
+    if (!config.connectionString) return null;
 
     const insights = new ApplicationInsights({
       config: {