test: remove unnecessary any eslint rule and clarify TEST_MODE deletion

ricardogarim · ricardogarim · commit ba12cd4ed682 · 2026-02-25T12:20:12.000-03:00
diff --git a/apps/meteor/app/version-check/server/functions/buildVersionUpdateMessage.spec.ts b/apps/meteor/app/version-check/server/functions/buildVersionUpdateMessage.spec.ts
@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-explicit-any */
 import { buildVersionUpdateMessage } from './buildVersionUpdateMessage';
 import { sendMessagesToAdmins } from '../../../../server/lib/sendMessagesToAdmins';
 
@@ -70,6 +69,8 @@ function createAsyncIterableFromArray(items: unknown[]) {
 }
 
 describe('buildVersionUpdateMessage', () => {
+	// Delete the TEST_MODE environment variable so buildVersionUpdateMessage()
+	// doesn't return early (see line 40 in buildVersionUpdateMessage.ts)
 	beforeAll(() => {
 		delete process.env.TEST_MODE;
 	});
