chore!: remove deprecated method livechat:registerGuest

.changeset/soft-ducks-build.md

@@ -0,0 +1,5 @@
+---
+'@rocket.chat/meteor': major
+---
+
+Removes deprecated method `livechat:registerAgent`. Moving forward, use the endpoint `livechat/visitor`.

apps/meteor/app/livechat/server/index.ts

@@ -22,7 +22,6 @@ import './methods/closeRoom';
 import './methods/getAnalyticsChartData';
 import './methods/getNextAgent';
 import './methods/getRoutingConfig';
-import './methods/registerGuest';
 import './methods/removeAllClosedRooms';
 import './methods/removeCustomField';
 import './methods/removeRoom';

apps/meteor/tests/end-to-end/api/livechat/09-visitors.ts

@@ -5,7 +5,7 @@ import { before, describe, it, after } from 'mocha';
 import moment from 'moment';
 import { type Response } from 'supertest';
 
-import { getCredentials, api, request, credentials, methodCallAnon } from '../../../data/api-data';
+import { getCredentials, api, request, credentials } from '../../../data/api-data';
 import { createCustomField, deleteCustomField } from '../../../data/livechat/custom-fields';
 import {
 	makeAgentAvailable,
@@ -226,19 +226,14 @@ describe('LIVECHAT - visitors', () => {
 				await updateSetting('Livechat_Allow_collect_and_store_HTTP_header_informations', false);
 			});
 
-			// Note: this had to use the meteor method because the endpoint used `req.headers` which we cannot send as empty
-			// method doesn't pass them to the func allowing us to create a test for it
 			it('should allow to create a visitor without passing connectionData when GDPR setting is enabled', async () => {
 				const token = `${new Date().getTime()}-test`;
-				const response = await request
-					.post(methodCallAnon('livechat:registerGuest'))
-					.send({ message: `{"msg":"method","id":"23","method":"livechat:registerGuest","params":[{ "token": "${token}"}]}` });
 
-				expect(response.body).to.have.property('success', true);
-				const r = JSON.parse(response.body.message);
+				const { body } = await request.post(api('livechat/visitor')).send({ visitor: { token } });
 
-				expect(r.result).to.have.property('visitor');
-				expect(r.result.visitor).to.have.property('token', token);
+				expect(body).to.have.property('success', true);
+				expect(body).to.have.property('visitor');
+				expect(body.visitor).to.have.property('token', token);
 			});
 		});
 	});