From 2c3eed8d96fdf62258c9685b0ef3fc317339b4af Mon Sep 17 00:00:00 2001
From: Andrew Rumble <andrew.rumble@overleaf.com>
Date: Fri, 25 Apr 2025 11:29:57 +0100
Subject: [PATCH] Use assert.fail instead of comparing `false` and `true`

GitOrigin-RevId: c6cbaf0a424771cf2a037366fccd790a13d9b1bb
---
 .../web/test/unit/src/Subscription/RecurlyWrapperTests.js     | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/services/web/test/unit/src/Subscription/RecurlyWrapperTests.js b/services/web/test/unit/src/Subscription/RecurlyWrapperTests.js
index 5fd2d11bc3..6f768e680e 100644
--- a/services/web/test/unit/src/Subscription/RecurlyWrapperTests.js
+++ b/services/web/test/unit/src/Subscription/RecurlyWrapperTests.js
@@ -1,4 +1,4 @@
-const { expect } = require('chai')
+const { assert, expect } = require('chai')
 const sinon = require('sinon')
 const modulePath = '../../../../app/src/Features/Subscription/RecurlyWrapper'
 const SandboxedModule = require('sandboxed-module')
@@ -319,7 +319,7 @@ describe('RecurlyWrapper', function () {
           this.recurlyAccountId,
           this.newEmail
         )
-        expect(false).to.equal(true) // Fail if we don't have an error
+        assert.fail('Expected error not thrown')
       } catch (error) {
         expect(error).to.have.property('message')
         expect(error.message.startsWith('Invalid character')).to.be.true