From 665b80ed1e80c760d984fa23e7d70fa2954cbc81 Mon Sep 17 00:00:00 2001
From: Andrew Rumble <andrew.rumble@overleaf.com>
Date: Wed, 27 Aug 2025 15:06:14 +0100
Subject: [PATCH] Fix tests

GitOrigin-RevId: db5505ce356a88cb572c86affebebdd7b9e68ba9
---
 .../src/Subscription/SubscriptionControllerTests.js    |  8 ++++++++
 .../Subscription/SubscriptionGroupController.test.mjs  | 10 +++++-----
 2 files changed, 13 insertions(+), 5 deletions(-)

diff --git a/services/web/test/unit/src/Subscription/SubscriptionControllerTests.js b/services/web/test/unit/src/Subscription/SubscriptionControllerTests.js
index 881d0ef0c5..910657305d 100644
--- a/services/web/test/unit/src/Subscription/SubscriptionControllerTests.js
+++ b/services/web/test/unit/src/Subscription/SubscriptionControllerTests.js
@@ -187,6 +187,14 @@ describe('SubscriptionController', function () {
             refreshFeatures: sinon.stub().resolves({ features: {} }),
           },
         }),
+        celebrate: (this.celebrate = {
+          celebrate: sinon.stub(),
+          errors: sinon.stub(),
+          Joi: {
+            any: sinon.stub(),
+            extend: sinon.stub(),
+          },
+        }),
         './GroupPlansData': (this.GroupPlansData = {}),
         './V1SubscriptionManager': (this.V1SubscriptionManager = {}),
         '../Errors/HttpErrorHandler': (this.HttpErrorHandler = {
diff --git a/services/web/test/unit/src/Subscription/SubscriptionGroupController.test.mjs b/services/web/test/unit/src/Subscription/SubscriptionGroupController.test.mjs
index 4783a44bcb..b326e3f030 100644
--- a/services/web/test/unit/src/Subscription/SubscriptionGroupController.test.mjs
+++ b/services/web/test/unit/src/Subscription/SubscriptionGroupController.test.mjs
@@ -1,4 +1,4 @@
-import { vi } from 'vitest'
+import { beforeEach, describe, it, vi } from 'vitest'
 import sinon from 'sinon'
 
 const modulePath =
@@ -612,14 +612,14 @@ describe('SubscriptionGroupController', function () {
     })
 
     it('should fail previewing "add seats" change', async function (ctx) {
-      await new Promise(resolve => {
-        ctx.SubscriptionGroupHandler.promises.previewAddSeatsSubscriptionChange =
-          sinon.stub().rejects()
+      ctx.SubscriptionGroupHandler.promises.previewAddSeatsSubscriptionChange =
+        sinon.stub().rejects()
+      ctx.req.body = { adding: 2 }
 
+      await new Promise(resolve => {
         const res = {
           status: statusCode => {
             statusCode.should.equal(500)
-
             return {
               end: () => {
                 resolve()