Merge branch 'dev' into AN-373-remove-terra-workflow-repo-feature-flag

src/auth/oidc-broker.ts

@@ -40,6 +40,7 @@ export const getOidcConfig = () => {
     // Leo's setCookie interval is currently 5 min, set refresh auth then 5 min 30 seconds to guarantee that setCookie's token won't expire between 2 setCookie api calls
     accessTokenExpiringNotificationTimeInSeconds: 330,
     includeIdTokenInSilentRenew: true,
+    includeIdTokenInSilentSignout: true,
     extraQueryParams: { access_type: 'offline' },
     redirect_uri: '', // this field is not being used currently, but is expected from UserManager
   };

src/auth/signout/sign-out.test.ts

@@ -52,6 +52,7 @@ jest.mock('src/libs/state', (): StateExports => {
       get: jest.fn().mockReturnValue({
         userManager: {
           signoutRedirect: jest.fn(() => 'Default signOutRedirectFn'),
+          getUser: jest.fn().mockReturnValue('not null'),
         },
       }),
     },
@@ -91,7 +92,7 @@ describe('sign-out', () => {
     const link = '/signout';
     const expectedState = btoa(JSON.stringify({ signOutRedirect: currentRoute, signOutCause: 'unspecified' }));
     asMockedFn(oidcStore.get).mockReturnValue({
-      userManager: { signoutRedirect: signOutRedirectFn },
+      userManager: { signoutRedirect: signOutRedirectFn, getUser: jest.fn().mockReturnValue('not null') },
     } as unknown as OidcState);
     asMockedFn(leoCookieProvider.unsetCookies).mockImplementation(unsetCookiesFn);
     asMockedFn(Nav.getPath).mockReturnValue(link);
@@ -116,6 +117,7 @@ describe('sign-out', () => {
     asMockedFn(oidcStore.get).mockReturnValue({
       userManager: {
         signoutRedirect: signOutRedirectFn,
+        getUser: jest.fn().mockReturnValue('not null'),
       },
     } as unknown as OidcState);
     asMockedFn(removeUserFromLocalState).mockImplementation(removeUserFromLocalStateFn);
@@ -131,4 +133,21 @@ describe('sign-out', () => {
     expect(removeUserFromLocalStateFn).toHaveBeenCalled();
     expect(goToRootFn).toHaveBeenCalledWith('root');
   });
+  it('calls userSignedOut if getUser returns null', async () => {
+    // Arrange
+    const removeUserFromLocalStateFn = jest.fn();
+    const goToRootFn = jest.fn();
+    asMockedFn(oidcStore.get).mockReturnValue({
+      userManager: {
+        getUser: jest.fn().mockReturnValue(null),
+      },
+    } as unknown as OidcState);
+    asMockedFn(removeUserFromLocalState).mockImplementation(removeUserFromLocalStateFn);
+    asMockedFn(goToPath).mockImplementation(goToRootFn);
+    // Act
+    await doSignOut();
+    // Assert
+    expect(removeUserFromLocalStateFn).toHaveBeenCalled();
+    expect(goToRootFn).toHaveBeenCalledWith('root');
+  });
 });

src/auth/signout/sign-out.ts

@@ -47,10 +47,16 @@ export const doSignOut = async (signOutCause: SignOutCause = 'unspecified'): Pro
     const { name, query, params }: SignOutRedirect = Nav.getCurrentRoute();
     const signOutState: SignOutState = { signOutRedirect: { name, query, params }, signOutCause };
     const encodedState = btoa(JSON.stringify(signOutState));
-    userManager!.signoutRedirect({
-      post_logout_redirect_uri: redirectUrl,
-      extraQueryParams: { state: encodedState },
-    });
+    const user = await userManager!.getUser();
+    if (user) {
+      userManager!.signoutRedirect({
+        post_logout_redirect_uri: redirectUrl,
+        extraQueryParams: { state: encodedState },
+      });
+    } else {
+      userSignedOut(signOutCause, true);
+      Nav.goToPath('root');
+    }
   } catch (e: unknown) {
     console.error('Signing out with B2C failed. Falling back on local signout', e);
     userSignedOut(signOutCause, true);

src/groups/Members/EmailSelect.tsx

@@ -74,6 +74,7 @@ export const EmailSelect: React.FC<EmailSelectProps> = ({
         onBlur={handleOnBlur}
         onChange={handleOnChange}
         height={200}
+        noOptionsMessage={() => null}
       />
     </>
   );

src/groups/Members/NewMemberModal.tsx

@@ -44,6 +44,8 @@ export const NewMemberModal = (props: NewMemberModalProps) => {
 
   const signal = useCancellation();
 
+  const enableShareLog = false; // Set "true" to enable suggestions
+
   useOnMount(() => {
     const loadData = withErrorReporting('Error looking up collaborators')(async () => {
       const [shareSuggestions, groups] = await Promise.all([Workspaces(signal).getShareLog(), Groups(signal).list()]);
@@ -52,7 +54,7 @@ export const NewMemberModal = (props: NewMemberModalProps) => {
 
       setSuggestions(suggestions);
     });
-    loadData();
+    enableShareLog && loadData();
   });
 
   const submit = async () => {

src/workspaces/ShareWorkspaceModal/ShareWorkspaceModal.tsx

@@ -59,6 +59,8 @@ const ShareWorkspaceModal: React.FC<ShareWorkspaceModalProps> = (props: ShareWor
 
   const signal = useCancellation();
 
+  const enableShareLog = false; // Set "true" to enable suggestions
+
   // Lifecycle
   useOnMount(() => {
     const load = async () => {
@@ -100,7 +102,7 @@ const ShareWorkspaceModal: React.FC<ShareWorkspaceModalProps> = (props: ShareWor
     _.uniq
   )(groups);
 
-  const remainingSuggestions = _.difference(suggestions, _.map('email', acl));
+  const remainingSuggestions = enableShareLog ? _.difference(suggestions, _.map('email', acl)) : [];
   const addUserReminder =
     'Did you mean to add collaborators? Add them or clear the "User emails" field to save changes.';