vona-module-test-pay 5.0.1 → 5.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vona-module-test-pay",
-  "version": "5.0.1",
+  "version": "5.0.2",
   "description": "",
   "keywords": [
     "Vona Module"
@@ -30,8 +30,8 @@
     "postpack": "clean-package restore"
   },
   "dependencies": {
-    "vona-module-a-pay": "5.0.2",
-    "vona-module-pay-mock": "5.0.2"
+    "vona-module-a-pay": "5.0.3",
+    "vona-module-pay-mock": "5.0.3"
   },
   "title": "test-pay",
   "vonaModule": {
@@ -51,5 +51,5 @@
       }
     }
   },
-  "gitHead": "95488ea0a7779f34f6ebddeca7719d8545bc55b3"
+  "gitHead": "e86b8ee51b837417b7b28b2c67862877d4092371"
 }

package/test/outbox.test.ts

@@ -2,8 +2,8 @@ import type { IPaymentOutcomeEvent } from 'vona-module-a-pay';
 
 import assert from 'node:assert';
 import { randomUUID } from 'node:crypto';
-import { describe, it } from 'node:test';
-import { app } from 'vona-mock';
+import { after, before, describe, it } from 'node:test';
+import { acquireTestLock, app } from 'vona-mock';
 
 interface IFixture {
   userId?: number;
@@ -79,6 +79,16 @@ async function cleanup(fixture: IFixture) {
 }
 
 describe('outbox.test.ts', { concurrency: false }, () => {
+  let releaseTestLock: (() => void) | undefined;
+
+  before(async () => {
+    releaseTestLock = await acquireTestLock('a-pay');
+  });
+
+  after(() => {
+    releaseTestLock?.();
+  });
+
   it('claims, backs off, and exhausts durable delivery attempts', async () => {
     await app.bean.executor.mockCtx(async () => {
       const fixture = await createFixture(randomUUID().slice(0, 12));
@@ -100,7 +110,7 @@ describe('outbox.test.ts', { concurrency: false }, () => {
         assert.ok((persisted?.nextAttemptAt?.getTime() ?? 0) >= releaseStart + 900);
         assert.equal(await scope.service.outbox.claim(event.id), undefined);
         await scope.model.outboxEvent.updateById(event.id, {
-          nextAttemptAt: new Date(Date.now() - 1),
+          nextAttemptAt: new Date(Date.now() - 1_000),
         });
         const second = await scope.service.outbox.claim(event.id);
         assert.equal(second?.attemptCount, 2);
@@ -150,7 +160,7 @@ describe('outbox.test.ts', { concurrency: false }, () => {
         const scope = app.scope('a-pay');
         const first = await scope.service.outbox.claim(event.id);
         await scope.model.outboxEvent.updateById(event.id, {
-          claimExpiresAt: new Date(Date.now() - 1),
+          claimExpiresAt: new Date(Date.now() - 1_000),
         });
         const second = await scope.service.outbox.claim(event.id);
         assert.ok(second?.claimToken);

package/test/payMock.test.ts

@@ -1,7 +1,7 @@
 import assert from 'node:assert';
 import { randomUUID } from 'node:crypto';
-import { describe, it } from 'node:test';
-import { app } from 'vona-mock';
+import { after, before, describe, it } from 'node:test';
+import { acquireTestLock, app } from 'vona-mock';
 
 interface IFixture {
   userId?: number;
@@ -38,7 +38,17 @@ async function cleanup(fixture: IFixture) {
   await app.bean.passport.signout();
 }
 
-describe('payMock.test.ts', { concurrency: false }, () => {
+describe('payMock.test.ts', { concurrency: false, sequential: true }, () => {
+  let releaseTestLock: (() => void) | undefined;
+
+  before(async () => {
+    releaseTestLock = await acquireTestLock('a-pay');
+  });
+
+  after(() => {
+    releaseTestLock?.();
+  });
+
   it('captures Vona env secrets in the mock Provider options', () => {
     const options = app.bean.payProvider.getOptions('pay-mock:mock', 'default');
     assert.equal(options.secretCredential, app.meta.env.PAY_MOCK_DEFAULT_CREDENTIAL);

package/test/paymentSession.test.ts

@@ -1,7 +1,7 @@
 import assert from 'node:assert';
 import { randomUUID } from 'node:crypto';
-import { describe, it } from 'node:test';
-import { app } from 'vona-mock';
+import { after, before, describe, it } from 'node:test';
+import { acquireTestLock, app } from 'vona-mock';
 
 interface IFixture {
   userId?: number;
@@ -27,6 +27,10 @@ async function createFixture(expiresAt: Date): Promise<IFixture> {
 async function cleanup(fixture: IFixture) {
   const scope = app.scope('a-pay');
   if (fixture.paymentSessionId !== undefined) {
+    await scope.model.outboxEvent.delete({ paymentSessionId: fixture.paymentSessionId });
+    await scope.model.webhookInbox.delete({ paymentSessionId: fixture.paymentSessionId });
+    await scope.model.providerOperation.delete({ paymentSessionId: fixture.paymentSessionId });
+    await scope.model.refundOperation.delete({ paymentSessionId: fixture.paymentSessionId });
     await scope.model.paymentAudit.delete({ paymentSessionId: fixture.paymentSessionId });
     await scope.model.paymentSession.delete({ id: fixture.paymentSessionId });
   }
@@ -37,6 +41,16 @@ async function cleanup(fixture: IFixture) {
 }
 
 describe('paymentSession.test.ts', { concurrency: false }, () => {
+  let releaseTestLock: (() => void) | undefined;
+
+  before(async () => {
+    releaseTestLock = await acquireTestLock('a-pay');
+  });
+
+  after(() => {
+    releaseTestLock?.();
+  });
+
   it('derives provider facts and expiry from the payment scene', async () => {
     await app.bean.executor.mockCtx(async () => {
       const scope = app.scope('a-pay');
@@ -102,9 +116,8 @@ describe('paymentSession.test.ts', { concurrency: false }, () => {
         const scope = app.scope('a-pay');
         const session = await scope.service.paymentSession.start(fixture.paymentSessionId!);
         assert.equal(session.state, 'requires_action');
-        await assert.rejects(scope.service.paymentSession.start(fixture.paymentSessionId!), {
-          status: 409,
-        });
+        const replayed = await scope.service.paymentSession.start(fixture.paymentSessionId!);
+        assert.equal(replayed.state, 'requires_action');
       } finally {
         await cleanup(fixture);
       }
@@ -113,7 +126,7 @@ describe('paymentSession.test.ts', { concurrency: false }, () => {
 
   it('does not start an expired payment session', async () => {
     await app.bean.executor.mockCtx(async () => {
-      const fixture = await createFixture(new Date(Date.now() - 1));
+      const fixture = await createFixture(new Date(Date.now() - 1_000));
       try {
         const scope = app.scope('a-pay');
         await assert.rejects(scope.service.paymentSession.start(fixture.paymentSessionId!), {
@@ -142,8 +155,13 @@ describe('paymentSession.test.ts', { concurrency: false }, () => {
           return await app.scope('a-pay').service.paymentSession.start(fixture.paymentSessionId!);
         }),
       ]);
-      assert.equal(results.filter(result => result.status === 'fulfilled').length, 1);
-      assert.equal(results.filter(result => result.status === 'rejected').length, 1);
+      assert.equal(results.filter(result => result.status === 'fulfilled').length, 2);
+      const operations = await app.bean.executor.mockCtx(async () => {
+        return await app.scope('a-pay').model.providerOperation.select({
+          where: { paymentSessionId: fixture.paymentSessionId, kind: 'start' },
+        });
+      });
+      assert.equal(operations.length, 1);
     } finally {
       await app.bean.executor.mockCtx(async () => {
         await cleanup(fixture);

package/test/webhook.test.ts

@@ -1,7 +1,7 @@
 import assert from 'node:assert';
 import { createHmac, randomUUID } from 'node:crypto';
-import { describe, it } from 'node:test';
-import { app } from 'vona-mock';
+import { after, before, describe, it } from 'node:test';
+import { acquireTestLock, app } from 'vona-mock';
 
 interface IFixture {
   userId?: number;
@@ -100,6 +100,16 @@ async function countFacts(paymentSessionId: number) {
 }
 
 describe('webhook.test.ts', { concurrency: false, sequential: true }, () => {
+  let releaseTestLock: (() => void) | undefined;
+
+  before(async () => {
+    releaseTestLock = await acquireTestLock('a-pay');
+  });
+
+  after(() => {
+    releaseTestLock?.();
+  });
+
   it('accepts a raw-body signed HTTP webhook and rejects an invalid signature without mutation', async () => {
     const fixture: IFixture = {};
     try {