fix(payment): show selector only for Philippines

src/app/_components/payment/__tests__/payment-method-selector.test.tsx

@@ -10,6 +10,7 @@ describe("PaymentMethodSelector", () => {
         config={{
           canChoosePaymentMethod: true,
           initialPayChannel: "ezpay",
+          showPaymentMethodSelector: true,
         }}
         value="ezpay"
         caption="GCash by default"
@@ -46,6 +47,7 @@ describe("PaymentMethodSelector", () => {
         config={{
           canChoosePaymentMethod: true,
           initialPayChannel: "stripe",
+          showPaymentMethodSelector: true,
         }}
         value="stripe"
         disabled
@@ -64,8 +66,9 @@ describe("PaymentMethodSelector", () => {
     const html = renderToStaticMarkup(
       <PaymentMethodSelector
         config={{
-          canChoosePaymentMethod: false,
+          canChoosePaymentMethod: true,
           initialPayChannel: "stripe",
+          showPaymentMethodSelector: false,
         }}
         value="stripe"
         analyticsKey="tip.payment_method"

src/app/_components/payment/payment-method-selector.tsx

@@ -40,7 +40,7 @@ export function PaymentMethodSelector({
   analyticsKey,
   onChange,
 }: PaymentMethodSelectorProps) {
-  if (!config.canChoosePaymentMethod) return null;
+  if (!config.showPaymentMethodSelector) return null;
 
   const paymentMethods =
     config.initialPayChannel === "ezpay"

src/app/_hooks/__tests__/use-payment-method-selection.test.tsx

@@ -27,7 +27,11 @@ describe("usePaymentMethodSelection", () => {
   it("forces Stripe when payment method selection is unavailable", () => {
     const onChange = vi.fn();
     renderHarness(root, {
-      config: { canChoosePaymentMethod: false, initialPayChannel: "stripe" },
+      config: {
+        canChoosePaymentMethod: false,
+        initialPayChannel: "stripe",
+        showPaymentMethodSelector: false,
+      },
       currentPayChannel: "ezpay",
       onChange,
     });
@@ -40,6 +44,7 @@ describe("usePaymentMethodSelection", () => {
     const config = {
       canChoosePaymentMethod: true,
       initialPayChannel: "ezpay",
+      showPaymentMethodSelector: true,
     } as const;
     renderHarness(root, {
       config,
@@ -59,13 +64,21 @@ describe("usePaymentMethodSelection", () => {
   it("preserves an explicit return channel and pauses while payment is busy", () => {
     const onChange = vi.fn();
     renderHarness(root, {
-      config: { canChoosePaymentMethod: true, initialPayChannel: "ezpay" },
+      config: {
+        canChoosePaymentMethod: true,
+        initialPayChannel: "ezpay",
+        showPaymentMethodSelector: true,
+      },
       currentPayChannel: "stripe",
       requestedPayChannel: "stripe",
       onChange,
     });
     renderHarness(root, {
-      config: { canChoosePaymentMethod: false, initialPayChannel: "stripe" },
+      config: {
+        canChoosePaymentMethod: false,
+        initialPayChannel: "stripe",
+        showPaymentMethodSelector: false,
+      },
       currentPayChannel: "ezpay",
       isPaymentBusy: true,
       onChange,

src/lib/payment/__tests__/payment_method.test.ts

@@ -34,6 +34,7 @@ describe("payment method rules", () => {
     ).toEqual({
       canChoosePaymentMethod: false,
       initialPayChannel: "stripe",
+      showPaymentMethodSelector: false,
     });
   });
 
@@ -51,7 +52,7 @@ describe("payment method rules", () => {
     ).toBe("stripe");
   });
 
-  it("allows every user and keeps the requested channel outside production", () => {
+  it("keeps test channels but shows the selector only for Philippines", () => {
     vi.stubEnv("NEXT_PUBLIC_APP_ENV", "test");
 
     expect(canChoosePayChannel(null)).toBe(true);
@@ -61,5 +62,27 @@ describe("payment method rules", () => {
         requestedPayChannel: "ezpay",
       }),
     ).toBe("ezpay");
+    expect(
+      getPaymentMethodConfig({
+        countryCode: "HK",
+        requestedPayChannel: "ezpay",
+      }),
+    ).toEqual({
+      canChoosePaymentMethod: true,
+      initialPayChannel: "ezpay",
+      showPaymentMethodSelector: false,
+    });
+    expect(
+      getPaymentMethodConfig({
+        countryCode: " ph ",
+        requestedPayChannel: null,
+      }).showPaymentMethodSelector,
+    ).toBe(true);
+    expect(
+      getPaymentMethodConfig({
+        countryCode: undefined,
+        requestedPayChannel: null,
+      }).showPaymentMethodSelector,
+    ).toBe(false);
   });
 });

src/lib/payment/payment_method.ts

@@ -6,13 +6,14 @@ import { getDefaultPayChannelForCountryCode } from "./default_pay_channel";
 export interface PaymentMethodConfig {
   canChoosePaymentMethod: boolean;
   initialPayChannel: PayChannel;
+  showPaymentMethodSelector: boolean;
 }
 
 export function canChoosePayChannel(
   countryCode: string | null | undefined,
 ): boolean {
   if (!AppEnvUtil.isProduction()) return true;
-  return countryCode?.trim().toUpperCase() === "PH";
+  return isPhilippinesCountryCode(countryCode);
 }
 
 export function resolvePayChannel(input: {
@@ -33,5 +34,12 @@ export function getPaymentMethodConfig(input: {
   return {
     canChoosePaymentMethod: canChoosePayChannel(input.countryCode),
     initialPayChannel: resolvePayChannel(input),
+    showPaymentMethodSelector: isPhilippinesCountryCode(input.countryCode),
   };
 }
+
+function isPhilippinesCountryCode(
+  countryCode: string | null | undefined,
+): boolean {
+  return countryCode?.trim().toUpperCase() === "PH";
+}