Merge pull request #157 from getpaykit/feat/ci-tests

test(e2e): expand CI workflows and provider coverage

Author: maxktz

.github/workflows/e2e.yml

@@ -13,8 +13,6 @@ on:
           - cli
           - stripe
           - polar
-  schedule:
-    - cron: "0 6 * * *"
 
 permissions: {}
 

e2e/core/webhook/subscription-deleted.test.ts

@@ -22,9 +22,10 @@ describe.skipIf(harness.id !== "stripe")(
     let t: TestPayKit;
     let customerId: string;
     let providerSubscriptionId: string;
-    const stripeClient = new Stripe(env.E2E_STRIPE_SK!, { maxNetworkRetries: 3 });
+    let stripeClient: Stripe;
 
     beforeAll(async () => {
+      stripeClient = new Stripe(env.E2E_STRIPE_SK!, { maxNetworkRetries: 3 });
       t = await createTestPayKit();
       const customer = await createTestCustomerWithPM({
         t,

e2e/test-utils/setup.ts

@@ -57,6 +57,17 @@ export interface CapturedWebhookRequest {
 const activeSubscriptionStatuses = ["active", "trialing", "past_due"] as const;
 const presentSubscriptionStatuses = [...activeSubscriptionStatuses, "scheduled"] as const;
 
+function buildUniqueTestEmail(input: { email: string; suffix: string }): string {
+  const [localPart, domain = ""] = input.email.split("@");
+  if (!localPart || !domain) {
+    throw new Error(`Invalid test email: ${input.email}`);
+  }
+
+  const uniqueLocalPart = `${localPart}+${input.suffix}`;
+
+  return `${uniqueLocalPart}@e2e.paykit.sh`;
+}
+
 export async function createTestPayKit(): Promise<TestPayKit> {
   harness.validateEnv();
 
@@ -143,7 +154,10 @@ export async function createTestCustomer(input: {
 }): Promise<{ customerId: string; providerCustomerId: string }> {
   const suffix = `${Date.now()}_${Math.random().toString(36).slice(2, 8)}`;
   const uniqueId = `${input.customer.id}_${suffix}`;
-  const uniqueEmail = input.customer.email.replace("@", `+${suffix}@`);
+  const uniqueEmail = buildUniqueTestEmail({
+    email: input.customer.email,
+    suffix,
+  });
 
   await input.t.paykit.upsertCustomer({
     ...input.customer,

e2e/vitest.config.ts

@@ -1,11 +1,14 @@
 import { defineConfig } from "vitest/config";
 
+const provider = process.env.PROVIDER;
+const isPolar = provider === "polar";
+
 export default defineConfig({
   test: {
     // Cap parallel workers — Stripe test mode rate-limits at 25 ops/sec; too many
     // workers starting syncProducts simultaneously trips it. Paired with Stripe
     // SDK maxNetworkRetries for headroom.
-    maxWorkers: 6,
+    maxWorkers: isPolar ? 1 : 6,
     projects: [
       {
         test: {
@@ -14,6 +17,7 @@ export default defineConfig({
           globalSetup: ["./test-utils/hub.ts"],
           hookTimeout: 180_000,
           include: ["core/**/*.test.ts"],
+          sequence: isPolar ? { concurrent: false } : undefined,
           testTimeout: 600_000,
         },
       },