@adobe/spacecat-shared-tier-client 1.0.0 → 1.0.1

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+# [@adobe/spacecat-shared-tier-client-v1.0.1](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-tier-client-v1.0.0...@adobe/spacecat-shared-tier-client-v1.0.1) (2025-09-18)
+
+
+### Bug Fixes
+
+* fix in create org entity ([#965](https://github.com/adobe/spacecat-shared/issues/965)) ([406cbf3](https://github.com/adobe/spacecat-shared/commit/406cbf3cd214f3ec3b332ca48d8a743c123a2ddd))
+
 # @adobe/spacecat-shared-tier-client-v1.0.0 (2025-09-16)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/spacecat-shared-tier-client",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "Shared modules of the Spacecat Services - Tier Client",
   "type": "module",
   "engines": {

package/src/tier-client.js

@@ -58,7 +58,8 @@ class TierClient {
     if (!isNonEmptyObject(context)) {
       throw new Error('Context is required');
     }
-    const organization = await site.getOrganization();
+    const organizationId = await site.getOrganizationId();
+    const organization = await context.dataAccess.Organization.findById(organizationId);
     return new TierClient(context, organization, site, productCode);
   }
 

package/test/tier-client.test.js

@@ -128,7 +128,7 @@ describe('TierClient', () => {
     const testSiteWithOrgRef = Object.create(Site.prototype);
     Object.assign(testSiteWithOrgRef, {
       getId: () => siteId,
-      getOrganization: () => testOrganization,
+      getOrganizationId: () => orgId,
     });
 
     describe('createForOrg', () => {
@@ -165,19 +165,23 @@ describe('TierClient', () => {
 
     describe('createForSite', () => {
       it('should create TierClient for site with getOrganizationId', async () => {
+        mockDataAccess.Organization.findById.resolves(testOrganization);
         const client = await TierClient.createForSite(mockContext, testSite, productCode);
 
         expect(client).to.be.an('object');
         expect(client.checkValidEntitlement).to.be.a('function');
         expect(client.createEntitlement).to.be.a('function');
+        expect(mockDataAccess.Organization.findById).to.have.been.calledWith(orgId);
       });
 
-      it('should create TierClient for site with getOrganization', async () => {
+      it('should create TierClient for site with getOrganizationId (alternative)', async () => {
+        mockDataAccess.Organization.findById.resolves(testOrganization);
         const client = await TierClient.createForSite(mockContext, testSiteWithOrgRef, productCode);
 
         expect(client).to.be.an('object');
         expect(client.checkValidEntitlement).to.be.a('function');
         expect(client.createEntitlement).to.be.a('function');
+        expect(mockDataAccess.Organization.findById).to.have.been.calledWith(orgId);
       });
 
       it('should throw error when site is not provided', async () => {
@@ -199,7 +203,7 @@ describe('TierClient', () => {
 
       it('should throw error when dataAccess is missing', async () => {
         const invalidContext = { log: {} };
-        await expect(TierClient.createForSite(invalidContext, testSite, productCode)).to.be.rejectedWith('Cannot destructure property');
+        await expect(TierClient.createForSite(invalidContext, testSite, productCode)).to.be.rejectedWith('Cannot read properties of undefined');
       });
     });
   });