Merge pull request #16405 from mozilla/FXA-9011

fix(auth): Revise eligibilityFromEligibilityManager - blocked_iap

packages/fxa-auth-server/lib/payments/capability.ts

@@ -383,13 +383,10 @@ export class CapabilityService {
         subscriptionEligibilityResult: SubscriptionEligibilityResult.INVALID,
       };
     const iapProductIds = iapSubscribedPlans.map((p) => p.product_id);
-    const planIds = [
+    const planIds = stripeSubscribedPlans.map((p) => p.plan_id);
-      ...stripeSubscribedPlans.map((p) => p.plan_id),
-      ...iapSubscribedPlans.map((p) => p.plan_id),
-    ];
     const overlaps = await this.eligibilityManager.getOfferingOverlap(
       planIds,
-      [],
+      iapProductIds,
       targetPlan.plan_id
     );
 

packages/fxa-auth-server/test/local/payments/capability.js

@@ -820,6 +820,48 @@ describe('CapabilityService', () => {
         });
       });
     });
+
+    describe('eligibilityManagerResult and stripeEligibilityResult should match', () => {
+      let mockEligibilityManager;
+
+      beforeEach(() => {
+        mockEligibilityManager = {};
+        Container.set(EligibilityManager, mockEligibilityManager);
+        capabilityService = new CapabilityService();
+      });
+
+      it('returns blocked_iap result from both', async () => {
+        mockEligibilityManager.getOfferingOverlap = sinon.fake.resolves([
+          {
+            comparison: 'same',
+            offeringProductId: mockPlanTier1ShortInterval.product_id,
+            type: 'offering',
+          },
+        ]);
+
+        capabilityService.fetchSubscribedPricesFromAppStore =
+          sinon.fake.resolves(['plan_123456']);
+
+        const eligiblityActual =
+          await capabilityService.eligibilityFromEligibilityManager(
+            [],
+            [mockPlanTier1ShortInterval],
+            mockPlanTier1LongInterval
+          );
+
+        const stripeActual =
+          await capabilityService.eligibilityFromStripeMetadata(
+            [],
+            [mockPlanTier1ShortInterval],
+            mockPlanTier1LongInterval
+          );
+
+        assert.deepEqual(
+          eligiblityActual.subscriptionEligibilityResult,
+          stripeActual.subscriptionEligibilityResult
+        );
+      });
+    });
   });
 
   describe('processPriceIdDiff', () => {