[STCOR-835] refactor getUserTenantsPermissions to leverage roles inst…

…ead of permissions (#1543)

* Conditionally use /users-keycloak/_self endpoint when roles interface is present

* Update CHANGELOG

* Update URLs

* CHANGELOG and lint fix

* Match old API respsonse

* Clarify URL code

(cherry picked from commit 49bf361)

CHANGELOG.md

@@ -1,7 +1,11 @@
 # Change history for stripes-core
 
-## [10.2.0](https://github.com/folio-org/stripes-core/tree/v10.2.0) (2024-10-11)
-[Full Changelog](https://github.com/folio-org/stripes-core/compare/v10.1.1...v10.2.0)
+## 10.2.2 IN PROGRESS
+
+* Conditionally use `/users-keycloak/_self` endpoint when `users-keycloak` interface is present. Refs STCOR-835.
+
+## [10.2.1](https://github.com/folio-org/stripes-core/tree/v10.2.1) (2024-10-30)
+[Full Changelog](https://github.com/folio-org/stripes-core/compare/v10.2.0...v10.2.1)
 
 * Wait longer before declaring a rotation request to be stale. Refs STCOR-895.
 

src/queries/getUserTenantsPermissions.js

@@ -8,16 +8,18 @@
  */
 const getUserTenantsPermissions = async (stripes, tenants = []) => {
   const {
-    user: { user: { id } },
     okapi: {
       url,
       token,
     }
   } = stripes;
   const userTenantIds = tenants.map(tenant => tenant.id || tenant);
 
+  const permPath = stripes.hasInterface('users-keycloak') ? 'users-keycloak' : 'bl-users';
+  const permUrl = `${url}/${permPath}/_self?expandPermissions=true`;
+
   const promises = userTenantIds.map(async (tenantId) => {
-    const result = await fetch(`${url}/perms/users/${id}/permissions?full=true&indexField=userId`, {
+    const result = await fetch(permUrl, {
       headers: {
         'X-Okapi-Tenant': tenantId,
         'Content-Type': 'application/json',
@@ -28,7 +30,7 @@ const getUserTenantsPermissions = async (stripes, tenants = []) => {
 
     const json = await result.json();
 
-    return { tenantId, ...json };
+    return { tenantId, permissionNames: json?.permissions?.permissions, ...json };
   });
 
   const userTenantsPermissions = await Promise.allSettled(promises);

src/queries/getUserTenantsPermissions.test.js

@@ -18,6 +18,7 @@ describe('getUserTenantsPermissions', () => {
         url: 'http://okapiUrl',
         token: 'elevensies',
       },
+      hasInterface: jest.fn(),
     };
     mockFetch.mockResolvedValueOnce('non-okapi-success');
 
@@ -38,6 +39,7 @@ describe('getUserTenantsPermissions', () => {
       okapi: {
         url: 'http://okapiUrl',
       },
+      hasInterface: jest.fn(),
     };
     mockFetch.mockResolvedValueOnce('non-okapi-success');
 
@@ -54,6 +56,7 @@ describe('getUserTenantsPermissions', () => {
       okapi: {
         url: 'http://okapiUrl',
       },
+      hasInterface: jest.fn(),
     };
 
     const t1 = { p: ['t1-p1', 't1-p2'] };