@adobe/spacecat-shared-tier-client 1.3.8 → 1.3.10

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+# [@adobe/spacecat-shared-tier-client-v1.3.10](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-tier-client-v1.3.9...@adobe/spacecat-shared-tier-client-v1.3.10) (2025-12-05)
+
+
+### Bug Fixes
+
+* no override paid tier ([#1221](https://github.com/adobe/spacecat-shared/issues/1221)) ([a528e1e](https://github.com/adobe/spacecat-shared/commit/a528e1e345cce2c542b32dc43c221b49e3f86464))
+
+# [@adobe/spacecat-shared-tier-client-v1.3.9](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-tier-client-v1.3.8...@adobe/spacecat-shared-tier-client-v1.3.9) (2025-11-30)
+
+
+### Bug Fixes
+
+* update data-access ([#1196](https://github.com/adobe/spacecat-shared/issues/1196)) ([c9461e7](https://github.com/adobe/spacecat-shared/commit/c9461e7c25634014088320d2faaf7839f164845c))
+
 # [@adobe/spacecat-shared-tier-client-v1.3.8](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-tier-client-v1.3.7...@adobe/spacecat-shared-tier-client-v1.3.8) (2025-11-28)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/spacecat-shared-tier-client",
-  "version": "1.3.8",
+  "version": "1.3.10",
   "description": "Shared modules of the Spacecat Services - Tier Client",
   "type": "module",
   "engines": {
@@ -36,7 +36,7 @@
   },
   "dependencies": {
     "@adobe/spacecat-shared-utils": "1.81.1",
-    "@adobe/spacecat-shared-data-access": "2.88.5"
+    "@adobe/spacecat-shared-data-access": "2.88.7"
   },
   "devDependencies": {
     "c8": "10.1.3",

package/src/tier-client.js

@@ -148,8 +148,8 @@ class TierClient {
       if (existing.entitlement) {
         const currentTier = existing.entitlement.getTier();
 
-        // If tier doesn't match, update it
-        if (currentTier !== tier) {
+        // If currentTier doesn't match with given tier and is not PAID, update it
+        if (currentTier !== tier && currentTier !== EntitlementModel.TIERS.PAID) {
           existing.entitlement.setTier(tier);
           await existing.entitlement.save();
         }

package/test/tier-client.test.js

@@ -417,7 +417,7 @@ describe('TierClient', () => {
     it('should update tier when entitlement exists with different tier', async () => {
       const mockEntitlementWithDifferentTier = {
         ...mockEntitlement,
-        getTier: () => 'PAID',
+        getTier: () => 'FREE_TRIAL', // current non-PAID
         setTier: sandbox.stub().returnsThis(),
         save: sandbox.stub().resolves(),
       };
@@ -426,9 +426,9 @@ describe('TierClient', () => {
         .findByOrganizationIdAndProductCode.resolves(mockEntitlementWithDifferentTier);
       mockDataAccess.SiteEnrollment.allBySiteId.resolves([mockSiteEnrollment]);
 
-      const result = await tierClient.createEntitlement('FREE_TRIAL');
+      const result = await tierClient.createEntitlement('PAID');
 
-      expect(mockEntitlementWithDifferentTier.setTier).to.have.been.calledWith('FREE_TRIAL');
+      expect(mockEntitlementWithDifferentTier.setTier).to.have.been.calledWith('PAID');
       expect(mockEntitlementWithDifferentTier.save).to.have.been.called;
       expect(result).to.deep.equal({
         entitlement: mockEntitlementWithDifferentTier,
@@ -437,6 +437,28 @@ describe('TierClient', () => {
       expect(mockDataAccess.Entitlement.create).to.not.have.been.called;
     });
 
+    it('should not update tier when entitlement exists with same tier', async () => {
+      const mockEntitlementWithSameTier = {
+        ...mockEntitlement,
+        getTier: () => 'FREE_TRIAL',
+        setTier: sandbox.stub().returnsThis(),
+        save: sandbox.stub().resolves(),
+      };
+
+      mockDataAccess.Entitlement
+        .findByOrganizationIdAndProductCode.resolves(mockEntitlementWithSameTier);
+      mockDataAccess.SiteEnrollment.allBySiteId.resolves([mockSiteEnrollment]);
+
+      const result = await tierClient.createEntitlement('FREE_TRIAL');
+
+      expect(mockEntitlementWithSameTier.setTier).to.not.have.been.called;
+      expect(mockEntitlementWithSameTier.save).to.not.have.been.called;
+      expect(result).to.deep.equal({
+        entitlement: mockEntitlementWithSameTier,
+        siteEnrollment: mockSiteEnrollment,
+      });
+    });
+
     it('should throw error when organization not found', async () => {
       mockDataAccess.Organization.findById.resolves(null);