refactor(tests): update test assertions for OpenAPI migration

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: ggazzo

apps/meteor/tests/end-to-end/api/commands.ts

@@ -124,22 +124,24 @@ describe('[Commands]', () => {
 				.expect(400)
 				.expect((res) => {
 					expect(res.body).to.have.property('success', false);
-					expect(res.body.error).to.be.equal('You must provide a command to run.');
+					expect(res.body.error).to.be.equal("must have required property 'command'");
 				})
 				.end(done);
 		});
+
 		it('should return an error when call the endpoint with the param "params" and it is not a string', (done) => {
 			void request
 				.post(api('commands.run'))
 				.set(credentials)
 				.send({
 					command: 'help',
+					roomId: 'GENERAL',
 					params: true,
 				})
 				.expect(400)
 				.expect((res) => {
 					expect(res.body).to.have.property('success', false);
-					expect(res.body.error).to.be.equal('The parameters for the command must be a single string.');
+					expect(res.body.error).to.include('must be string');
 				})
 				.end(done);
 		});
@@ -154,7 +156,7 @@ describe('[Commands]', () => {
 				.expect(400)
 				.expect((res) => {
 					expect(res.body).to.have.property('success', false);
-					expect(res.body.error).to.be.equal("The room's id where to execute this command must be provided and be a string.");
+					expect(res.body.error).to.be.equal("must have required property 'roomId'");
 				})
 				.end(done);
 		});
@@ -171,7 +173,7 @@ describe('[Commands]', () => {
 				.expect(400)
 				.expect((res) => {
 					expect(res.body).to.have.property('success', false);
-					expect(res.body.error).to.be.equal('The tmid parameter when provided must be a string.');
+					expect(res.body.error).to.include('must be string');
 				})
 				.end(done);
 		});
@@ -437,11 +439,6 @@ describe('[Commands]', () => {
 					roomId: channel._id,
 					command: 'invite-all-from',
 					params: `#${group.name}`,
-					msg: {
-						_id: Random.id(),
-						rid: channel._id,
-						msg: `invite-all-from #${group.name}`,
-					},
 					triggerId: Random.id(),
 				})
 				.expect(200)
@@ -468,16 +465,11 @@ describe('[Commands]', () => {
 					roomId: group1._id,
 					command: 'invite-all-from',
 					params: `#${group.name}`,
-					msg: {
-						_id: Random.id(),
-						rid: group1._id,
-						msg: `invite-all-from #${group.name}`,
-					},
 					triggerId: Random.id(),
 				})
 				.expect(403)
 				.expect((res) => {
-					expect(res.body).to.have.a.property('error', 'unauthorized');
+					expect(res.body).to.have.a.property('error', 'Not allowed');
 				});
 		});
 
@@ -498,11 +490,6 @@ describe('[Commands]', () => {
 					roomId: channel._id,
 					command: 'invite-all-from',
 					params: `#${group.name}`,
-					msg: {
-						_id: Random.id(),
-						rid: channel._id,
-						msg: `invite-all-from #${group.name}`,
-					},
 					triggerId: Random.id(),
 				})
 				.expect(200)

apps/meteor/tests/end-to-end/api/rooms.ts

@@ -2358,30 +2358,39 @@ describe('[Rooms]', () => {
 				});
 		});
 
-		it('should update group name if user changes name', async () => {
-			await updateSetting('UI_Use_Real_Name', true);
-			await request
-				.post(api('users.update'))
-				.set(credentials)
-				.send({
-					userId: testUser._id,
-					data: {
-						name: `changed.name.${testUser.username}`,
-					},
-				});
+		describe('use real name', () => {
+			before(async () => {
+				await updateSetting('UI_Use_Real_Name', true);
+			});
 
-			// need to wait for the name update finish
-			await sleep(300);
+			after(async () => {
+				await updateSetting('UI_Use_Real_Name', false);
+			});
 
-			await request
-				.get(api('subscriptions.getOne'))
-				.set(credentials)
-				.query({ roomId })
-				.send()
-				.expect((res) => {
-					const { subscription } = res.body;
-					expect(subscription.fname).to.equal(`changed.name.${testUser.username}, ${testUser2.name}`);
-				});
+			it('should update group name if user changes name', async () => {
+				await request
+					.post(api('users.update'))
+					.set(credentials)
+					.send({
+						userId: testUser._id,
+						data: {
+							name: `changed.name.${testUser.username}`,
+						},
+					});
+
+				// need to wait for the name update finish
+				await sleep(300);
+
+				await request
+					.get(api('subscriptions.getOne'))
+					.set(credentials)
+					.query({ roomId })
+					.send()
+					.expect((res) => {
+						const { subscription } = res.body;
+						expect(subscription.fname).to.equal(`changed.name.${testUser.username}, ${testUser2.name}`);
+					});
+			});
 		});
 	});
 

apps/meteor/tests/end-to-end/apps/slash-command-test-simple.ts

@@ -27,7 +27,7 @@ import { IS_EE } from '../../e2e/config/constants';
 				.expect(400)
 				.expect((res) => {
 					expect(res.body).to.have.a.property('success', false);
-					expect(res.body.error).to.be.equal('You must provide a command to run.');
+					expect(res.body.error).to.include('must be string');
 				})
 				.end(done);
 		});