Chore: Update to 6.5.0

package.json

@@ -1,6 +1,6 @@
 {
         "name": "graph-explorer-v2",
-        "version": "6.4.0",
+        "version": "6.5.0",
         "private": true,
         "dependencies": {
                 "@augloop/types-core": "file:packages/types-core-2.16.189.tgz",

src/app/services/actions/auth-action-creators.spec.ts

@@ -123,7 +123,7 @@ describe('Auth Action Creators', () => {
 
   it('should dispatch LOGOUT_SUCCESS when signOutSuccess() is called', () => {
     // Arrange
-    const response: boolean = false;
+    const response: boolean = true;
     const expectedAction = {
       type: LOGOUT_SUCCESS,
       response

src/app/services/reducers/auth-reducers.ts

@@ -38,7 +38,7 @@ export function consentedScopes(state: string[] = [], action: IAction): any {
     case GET_CONSENTED_SCOPES_SUCCESS:
       return action.response;
     case LOGOUT_SUCCESS:
-      return action.response;
+      return [];
     default:
       return state;
   }

src/app/views/App.spec.tsx

@@ -116,23 +116,6 @@ describe('App rendering', () => {
     expect(screen.getByRole('tab', { name: /expand/i} )).toBeDefined();
   });
 
-  it('should hide the sidebar when \'minimize sidebar button\' is clicked', async () => {
-    const user = userEvent.setup();
-    renderApp({ mobileScreen: false, showSidebar: true });
-    const minimizeButton = screen.getByRole('button', { name: /minimize sidebar/i });
-    await user.click(minimizeButton);
-    expect(screen.getByText(/sidebar minimized/i)).toBeDefined();
-
-    // resources, samples and history tabs are now buttons on the sidebar
-    const samplesButton = screen.getByRole('button', { name: /sample queries/i});
-    expect(screen.getByRole('button', { name: /resources/i})).toBeDefined();
-    expect(screen.getByRole('button', { name: /history/i})).toBeDefined();
-
-    await user.click(samplesButton);
-    expect(screen.getByText(/sidebar maximized/i)).toBeDefined();
-    expect(screen.getByRole('tab',{ name: /sample queries/i}));
-  })
-
   it('should render http methods dropdown when the \'request method button\' is clicked', async () => {
     const user = userEvent.setup();
     renderApp({mobileScreen: false, showSidebar: true});

src/app/views/authentication/profile/Profile.tsx

@@ -59,7 +59,7 @@ const Profile = (props: any) => {
   const labelId = useId('callout-label');
   const descriptionId = useId('callout-description');
   const theme = getTheme();
-  const { personaStyleToken , profileSpinnerStyles, permissionsLabelStyles, personaButtonStyles,
+  const { personaStyleToken, profileSpinnerStyles, permissionsLabelStyles, personaButtonStyles,
     profileContainerStyles } = profileStyles(theme);
 
   useEffect(() => {
@@ -99,7 +99,7 @@ const Profile = (props: any) => {
 
   const onRenderSecondaryText = (prop: IPersonaProps): JSX.Element => {
     return (
-      <span style={{fontSize: FontSizes.small}}>
+      <span style={{ fontSize: FontSizes.small }}>
         {prop.secondaryText}
       </span>
     );
@@ -142,25 +142,26 @@ const Profile = (props: any) => {
           beakWidth={10}
           onDismiss={toggleIsCalloutVisible}
           setInitialFocus
-          styles={{root: {border: '1px solid' + theme.palette.neutralTertiary}}}
+          styles={{ root: { border: '1px solid' + theme.palette.neutralTertiary } }}
         >
-          <Stack horizontal horizontalAlign='space-between' styles={{root:{ paddingBottom: 0}}}>
+          <Stack horizontal horizontalAlign='space-between' styles={{ root: { paddingBottom: 0 } }}>
             {profile &&
-            <ActionButton text={`${profile.tenant}`} disabled={true}/>
+              <ActionButton text={`${profile.tenant}`} disabled={true} />
             }
             <ActionButton key={'sign-out'} onClick={() => handleSignOut()}>
               {translateMessage('sign out')}
             </ActionButton>
           </Stack>
-          <Stack styles={{root:{ paddingLeft: 10 }}}>{fullPersona}</Stack>
+          <Stack styles={{ root: { paddingLeft: 10 } }}>{fullPersona}</Stack>
           {graphExplorerMode === Mode.Complete &&
-          <ActionButton key={'view-all-permissions'} onClick={() => changePanelState()} styles={permissionsLabelStyles}>
-            {translateMessage('view all permissions')}
-          </ActionButton>
+            <ActionButton key={'view-all-permissions'}
+              onClick={() => changePanelState()} styles={permissionsLabelStyles}>
+              {translateMessage('view all permissions')}
+            </ActionButton>
           }
-          <Stack styles={{root:{ background: theme.palette.neutralLighter, padding:10}}}>
+          <Stack styles={{ root: { background: theme.palette.neutralLighter, padding: 10 } }}>
             <ActionButton key={'sign-other-account'} onClick={() => handleSignInOther()}
-              iconProps={{iconName: 'AddFriend'}}
+              iconProps={{ iconName: 'AddFriend' }}
             >
               {translateMessage('sign in other account')}
             </ActionButton>
@@ -175,7 +176,7 @@ const Profile = (props: any) => {
   return (
     <div className={classes.profile} style={profileContainerStyles}>
       {showProfileComponent(persona)}
-      {permissionsPanelOpen && <Permission panel={true}/>}
+      {permissionsPanelOpen && <Permission panel={true} />}
     </div>
   );
 }
@@ -190,4 +191,4 @@ const styles = mergeStyleSets({
 // @ts-ignore
 const styledProfile = styled(Profile, authenticationStyles);
 // @ts-ignore
-export default styledProfile;
+export default styledProfile;

src/app/views/query-response/graph-toolkit/GraphToolkit.tsx

@@ -5,7 +5,6 @@ import {
   getTheme, IStyle, ITheme, Label, Link,
   MessageBar, MessageBarType, styled
 } from '@fluentui/react';
-
 import { componentNames, telemetry } from '../../../../telemetry';
 import { IRootState } from '../../../../types/root';
 import { lookupToolkitUrl } from '../../../utils/graph-toolkit-lookup';

src/telemetry/filters.ts

@@ -1,5 +1,4 @@
 import { ITelemetryItem } from '@microsoft/applicationinsights-web';
-import { errorTypes } from '.';
 import {
   ADAPTIVE_CARD_URL,
   DEVX_API_URL,
@@ -92,31 +91,3 @@ export function sanitizeTelemetryItemUriProperty(envelope: ITelemetryItem) {
   }
   return true;
 }
-
-export function sanitizeStackTrace(envelope: ITelemetryItem) {
-  if (envelope.baseType === 'ExceptionData') {
-    const telemetryItem = envelope.baseData || {};
-    telemetryItem.properties = telemetryItem.properties || {};
-
-    if (telemetryItem.exceptions && telemetryItem.exceptions.length > 0) {
-      const exception = telemetryItem.exceptions[0];
-      const parsedStack = exception.parsedStack[0];
-
-      // Only capture errors coming from our source code and not dependencies, to reduce noise
-      if (!parsedStack.fileName.startsWith('webpack-internal')) {
-        return false;
-      }
-
-      // Add properties for unhandled exceptions only
-      if (!telemetryItem.properties.ComponentName) {
-        telemetryItem.properties.ComponentName = parsedStack.assembly;
-        telemetryItem.properties.Message = exception.stack.split('\n')[0]; // Read first line only
-        exception.message = errorTypes.UNHANDLED_ERROR;
-      }
-
-      exception.hasFullStack = false;
-      exception.stack = null;
-    }
-  }
-  return true;
-}

src/telemetry/telemetry.ts

@@ -24,12 +24,14 @@ import {
   addCommonTelemetryItemProperties,
   filterRemoteDependencyData,
   filterTelemetryTypes,
-  sanitizeStackTrace,
   sanitizeTelemetryItemUriProperty
 } from './filters';
 import ITelemetry from './ITelemetry';
 import { getVersion } from '../app/utils/version';
-import { getBrowserScreenSize, getDeviceScreenScale } from '../app/utils/device-characteristics-telemetry';
+import {
+  getBrowserScreenSize,
+  getDeviceScreenScale
+} from '../app/utils/device-characteristics-telemetry';
 
 class Telemetry implements ITelemetry {
   private appInsights: ApplicationInsights;
@@ -58,7 +60,6 @@ class Telemetry implements ITelemetry {
     this.appInsights.trackPageView();
     this.appInsights.addTelemetryInitializer(filterTelemetryTypes);
     this.appInsights.addTelemetryInitializer(filterRemoteDependencyData);
-    this.appInsights.addTelemetryInitializer(sanitizeStackTrace);
     this.appInsights.addTelemetryInitializer(sanitizeTelemetryItemUriProperty);
     this.appInsights.addTelemetryInitializer(addCommonTelemetryItemProperties);
     this.appInsights.context.application.ver = getVersion().toString();

src/tests/telemetry/filters.spec.ts

@@ -1,10 +1,11 @@
 import { ITelemetryItem } from '@microsoft/applicationinsights-web';
 import {
-  filterTelemetryTypes, filterRemoteDependencyData, addCommonTelemetryItemProperties, sanitizeTelemetryItemUriProperty,
-  sanitizeStackTrace
+  filterTelemetryTypes,
+  filterRemoteDependencyData,
+  addCommonTelemetryItemProperties,
+  sanitizeTelemetryItemUriProperty
 } from '../../../src/telemetry/filters';
 
-
 describe('Telemetry filters should', () => {
   it('ensure telemetry types to include are correct when filterTelemetryTypes() is called', () => {
     // Arrange
@@ -21,7 +22,6 @@ describe('Telemetry filters should', () => {
 
     // Assert
     expect(result).toBe(true);
-
   });
 
   it('return true by default when filterRemoteDependencyData() is called', () => {
@@ -56,7 +56,7 @@ describe('Telemetry filters should', () => {
 
     // Assert
     expect(result).toBe(true);
-  });
+  })
 
   it('sanitize telemetry item uri property', () => {
     // Arrange
@@ -77,35 +77,4 @@ describe('Telemetry filters should', () => {
     // Assert
     expect(result).toBe(true);
   })
-
-  it('sanitize stack trace', () => {
-    // Arrange
-    const envelope: ITelemetryItem = {
-      ver: '1.0',
-      name: 'test',
-      time: '',
-      iKey: '',
-      baseType: 'ExceptionData',
-      baseData: {
-        exceptions: [
-          {
-            parsedStack: [
-              {
-                fileName: 'webpack-internal',
-                assembly: 'Assembly'
-              }
-            ],
-            stack: '\n First line of test, \n Second line of test'
-          }
-        ]
-      }
-    }
-
-    // Act
-    const result = sanitizeStackTrace(envelope);
-
-    // Assert
-    expect(result).toBe(true);
-  })
-
-})
+})