Still more sso test

CHANGELOG.md

@@ -14,6 +14,7 @@
 * Opt-in: disable login when cookies are disabled. Refs STCOR-762.
 * Add arial-label for `ProfileDropdown.js`. Refs STCOR-753.
 * Move `localforage.clear()` to `afterEach` for test suite. Refs STCOR-801.
+* Avoid calling `map` on `undefined` via optional-chaining. Refs STCOR-793.
 
 ## [10.0.0](https://github.com/folio-org/stripes-core/tree/v10.0.0) (2023-10-11)
 [Full Changelog](https://github.com/folio-org/stripes-core/compare/v9.0.0...v10.0.0)

src/RootWithIntl.js

@@ -77,6 +77,7 @@
     const connect = connectFor('@folio/core', this.props.stripes.epics, this.props.stripes.logger);
     const stripes = this.props.stripes.clone({ connect });
 
+    console.log(`isAuthenticated: ${isAuthenticated}, token: ${token}`);
     return (
       <StripesContext.Provider value={stripes}>
         <CalloutContext.Provider value={this.state.callout}>

src/components/SSOLanding.test.js

@@ -0,0 +1,71 @@
+import { render, screen } from '@folio/jest-config-stripes/testing-library/react';
+import { useLocation } from 'react-router-dom';
+import { useCookies } from 'react-cookie';
+import { requestUserWithPerms } from '../loginServices';
+
+import SSOLanding from './SSOLanding';
+
+jest.mock('lodash', () => ({
+  debounce: jest.fn(fn => fn)
+}));
+
+jest.mock('../loginServices', () => ({
+  requestUserWithPerms: jest.fn(() => {})
+}));
+
+jest.mock('react-router-dom', () => ({
+  useLocation: jest.fn()
+}));
+
+jest.mock('stripes-config', () => ({
+  okapi: {
+    url: 'okapiUrl',
+    tenant: 'okapiTenant'
+  }
+}),
+{ virtual: true });
+
+jest.mock('react-cookie', () => ({
+  useCookies: jest.fn()
+}));
+
+jest.mock('react-redux', () => ({
+  useStore: jest.fn()
+}));
+
+describe('SSOLanding', () => {
+  beforeEach(() => {
+    useLocation.mockImplementation(() => ({ search: '' }));
+    useCookies.mockImplementation(() => [{}]);
+  });
+
+  afterEach(() => {
+    jest.resetAllMocks();
+  });
+
+  describe('handles token within query parameters', () => {
+    it('processes login', () => {
+      useLocation.mockImplementation(() => ({ search: 'ssoToken=c0ffee' }));
+      render(<SSOLanding />);
+      expect(requestUserWithPerms.mock.calls).toHaveLength(1);
+      expect(screen.getByText(/Logged in with token.*param\.$/)).toBeInTheDocument();
+    });
+  });
+
+  describe('handles token within a cookie', () => {
+    it('processes login', () => {
+      useCookies.mockImplementation(() => ([{ ssoToken: 'c0ffee-c0ffee' }]));
+      render(<SSOLanding />);
+      expect(requestUserWithPerms.mock.calls).toHaveLength(1);
+      expect(screen.getByText(/Logged in with token.*cookie\.$/)).toBeInTheDocument();
+    });
+  });
+
+  describe('displays error with no token.', () => {
+    it('processes login', () => {
+      render(<SSOLanding />);
+      expect(requestUserWithPerms.mock.calls).toHaveLength(0);
+      expect(screen.getByText('No cookie or query parameter')).toBeInTheDocument();
+    });
+  });
+});

src/loginServices.js

@@ -397,9 +397,11 @@ export function spreadUserWithPerms(userWithPerms) {
     ...userWithPerms?.user?.personal,
   };
 
-  // remap data's array of permission-names to set with
-  // permission-names for keys and `true` for values
-  const perms = Object.assign({}, ...userWithPerms?.permissions?.permissions.map(p => ({ [p.permissionName]: true })));
+  // remap permissions from [foo, bar, bat] to { foo: true, bar: true, ... }
+  const perms = userWithPerms?.permissions?.permissions?.reduce((acc, i) => {
+    acc[i.permissionName] = true;
+    return acc;
+  }, {});
 
   return { user, perms };
 }

test/bigtest/helpers/setup-application.js

@@ -1,4 +1,4 @@
-import { beforeEach } from '@bigtest/mocha';
+import { beforeEach, afterEach } from '@bigtest/mocha';
 import localforage from 'localforage';
 
 import { reset } from '@folio/stripes-connect';

test/bigtest/helpers/setup-core-application.js

@@ -1,6 +1,6 @@
 import setupApplication from './setup-application';
 
 export default function setupCoreApplication(options = {}) {
-  options.mirageOptions = { serverType: 'miragejs' };
+  options.mirageOptions = { ...options.mirageOptions, serverType: 'miragejs' };
   setupApplication(options);
 }

test/bigtest/network/config.js

@@ -68,6 +68,11 @@ export default function configure() {
       },
       'departments': [
       ]
+    },
+    'permissions' : {
+      permissions: [
+        'configuration.entries.collection.get'
+      ]
     }
   });