@adobe/spacecat-shared-tier-client 1.3.6 → 1.3.7

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+# [@adobe/spacecat-shared-tier-client-v1.3.7](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-tier-client-v1.3.6...@adobe/spacecat-shared-tier-client-v1.3.7) (2025-11-28)
+
+
+### Bug Fixes
+
+* using entity name instead of instanceof ([#1190](https://github.com/adobe/spacecat-shared/issues/1190)) ([eccda60](https://github.com/adobe/spacecat-shared/commit/eccda60d02f1df7d251957430cb7fcef4041f89c))
+
 # [@adobe/spacecat-shared-tier-client-v1.3.6](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-tier-client-v1.3.5...@adobe/spacecat-shared-tier-client-v1.3.6) (2025-11-28)
 
 

package/package.json

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

package/src/tier-client.js

@@ -28,7 +28,7 @@ class TierClient {
    * @returns {TierClient} TierClient instance for organization operations.
    */
   static createForOrg(context, organization, productCode) {
-    if (!(organization instanceof Organization)) {
+    if (!(organization.entityName === Organization.ENTITY_NAME)) {
       throw new Error('Entity must be an instance of Organization');
     }
     if (!hasText(productCode)) {
@@ -48,7 +48,7 @@ class TierClient {
    * @returns {TierClient} TierClient instance for site operations.
    */
   static async createForSite(context, site, productCode) {
-    if (!(site instanceof Site)) {
+    if (!(site.entityName === Site.ENTITY_NAME)) {
       throw new Error('Entity must be an instance of Site');
     }
     if (!hasText(productCode)) {

package/test/tier-client.test.js

@@ -50,7 +50,10 @@ describe('TierClient', () => {
 
   // Create actual Organization instance for instanceof checks
   const organizationInstance = Object.create(Organization.prototype);
-  Object.assign(organizationInstance, mockOrganization);
+  Object.assign(
+    organizationInstance,
+    { entityName: Organization.ENTITY_NAME, ...mockOrganization },
+  );
 
   const mockSite = {
     getId: () => siteId,
@@ -60,7 +63,7 @@ describe('TierClient', () => {
 
   // Create actual Site instance for instanceof checks
   const siteInstance = Object.create(Site.prototype);
-  Object.assign(siteInstance, mockSite);
+  Object.assign(siteInstance, { entityName: Site.ENTITY_NAME, ...mockSite });
 
   const mockDataAccess = {
     Entitlement: {
@@ -118,10 +121,11 @@ describe('TierClient', () => {
 
   describe('Static Factory Methods', () => {
     const testOrganization = Object.create(Organization.prototype);
-    Object.assign(testOrganization, { getId: () => orgId });
+    Object.assign(testOrganization, { entityName: Organization.ENTITY_NAME, getId: () => orgId });
 
     const testSite = Object.create(Site.prototype);
     Object.assign(testSite, {
+      entityName: Site.ENTITY_NAME,
       getId: () => siteId,
       getOrganizationId: () => orgId,
       getOrganization: () => testOrganization,
@@ -129,6 +133,7 @@ describe('TierClient', () => {
 
     const testSiteWithOrgRef = Object.create(Site.prototype);
     Object.assign(testSiteWithOrgRef, {
+      entityName: Site.ENTITY_NAME,
       getId: () => siteId,
       getOrganizationId: () => orgId,
     });
@@ -143,7 +148,7 @@ describe('TierClient', () => {
       });
 
       it('should throw error when organization is not provided', () => {
-        expect(() => TierClient.createForOrg(mockContext, null, productCode)).to.throw('Entity must be an instance of Organization');
+        expect(() => TierClient.createForOrg(mockContext, null, productCode)).to.throw('Cannot read properties of null');
       });
 
       it('should throw error when organization has no getId method', () => {
@@ -194,7 +199,7 @@ describe('TierClient', () => {
       });
 
       it('should throw error when site is not provided', async () => {
-        await expect(TierClient.createForSite(mockContext, null, productCode)).to.be.rejectedWith('Entity must be an instance of Site');
+        await expect(TierClient.createForSite(mockContext, null, productCode)).to.be.rejectedWith('Cannot read properties of null');
       });
 
       it('should throw error when site has no getId method', async () => {