fix: ee not being extended because no default export

strapi · Apr 22, 2024 · 3b831b1 · 3b831b1
1 parent 0e2e3db
commit 3b831b1
Show file tree

Hide file tree

Showing 5 changed files with 8 additions and 11 deletions.
diff --git a/packages/core/admin/server/src/services/__tests__/metrics.test.ts b/packages/core/admin/server/src/services/__tests__/metrics.test.ts
@@ -1,8 +1,4 @@
-import {
-  sendDidInviteUser,
-  sendDidUpdateRolePermissions,
-  sendDidChangeInterfaceLanguage,
-} from '../metrics';
+import metrics from '../metrics';
 
 describe('Metrics', () => {
   test('sendDidInviteUser', async () => {
@@ -19,7 +15,7 @@ describe('Metrics', () => {
       },
     } as any;
 
-    await sendDidInviteUser();
+    await metrics.sendDidInviteUser();
 
     expect(send).toHaveBeenCalledWith('didInviteUser', {
       groupProperties: {
@@ -37,7 +33,7 @@ describe('Metrics', () => {
       telemetry: { send },
     } as any;
 
-    await sendDidUpdateRolePermissions();
+    await metrics.sendDidUpdateRolePermissions();
 
     expect(send).toHaveBeenCalledWith('didUpdateRolePermissions');
   });
@@ -55,7 +51,7 @@ describe('Metrics', () => {
       },
     } as any;
 
-    await sendDidChangeInterfaceLanguage();
+    await metrics.sendDidChangeInterfaceLanguage();
 
     expect(getLanguagesInUse).toHaveBeenCalledWith();
     expect(send).toHaveBeenCalledWith('didChangeInterfaceLanguage', {

diff --git a/packages/core/admin/server/src/services/index.ts b/packages/core/admin/server/src/services/index.ts
@@ -3,9 +3,9 @@ import auth from './auth';
 import user from './user';
 import role from './role';
 import passport from './passport';
+import metrics from './metrics';
 import * as token from './token';
 import * as permission from './permission';
-import * as metrics from './metrics';
 import * as contentType from './content-type';
 import * as constants from './constants';
 import * as condition from './condition';

diff --git a/packages/core/admin/server/src/services/metrics.ts b/packages/core/admin/server/src/services/metrics.ts
@@ -34,7 +34,7 @@ const startCron = (strapi: Strapi) => {
   });
 };
 
-export {
+export default {
   sendDidInviteUser,
   sendDidUpdateRolePermissions,
   sendDidChangeInterfaceLanguage,

diff --git a/packages/core/admin/server/src/utils/index.d.ts b/packages/core/admin/server/src/utils/index.d.ts
@@ -2,9 +2,9 @@ import auth from '../services/auth';
 import role from '../services/role';
 import user from '../services/user';
 import passport from '../services/passport';
+import metrics from '../services/metrics';
 import * as permission from '../services/permission';
 import * as contentType from '../services/content-type';
-import * as metrics from '../services/metrics';
 import * as token from '../services/token';
 import * as apiToken from '../services/api-token';
 import * as projectSettings from '../services/project-settings';

diff --git a/packages/core/admin/strapi-server.js b/packages/core/admin/strapi-server.js
@@ -11,6 +11,7 @@ const mergeRoutes = (a, b, key) => {
 if (strapi.EE) {
   const eeAdmin = require('./dist/ee/server');
   // module.exports = admin;
+  // TODO: change to avoid issue with lodash merging frozen objects
   module.exports = _.mergeWith({}, admin, eeAdmin, mergeRoutes);
 } else {
   module.exports = admin;