Merge pull request #195 from streamdevs/179-mr-link

feat: send mr link in mr events

Author: SantiMA10

src/reactions/gitlab/merge-request-merged.ts

@@ -7,7 +7,7 @@ export class MergeRequestMerged extends Reaction<MergeRequestPayload> {
 	}
 
 	getTwitchChatMessage({ payload }: ReactionHandleOptions<MergeRequestPayload>): string {
-		return `The merge request from ${payload.user.username} has been merged into ${payload.repository.homepage}`;
+		return `The merge request from ${payload.user.username} has been merged into ${payload.object_attributes.url}`;
 	}
 
 	canHandle({ payload, event }: ReactionCanHandleOptions<MergeRequestPayload>): boolean {

src/reactions/gitlab/merge-request-opened.ts

@@ -8,7 +8,7 @@ export class MergeRequestOpened extends Reaction<MergeRequestPayload> {
 	}
 
 	getTwitchChatMessage({ payload }: ReactionHandleOptions<MergeRequestPayload>): string {
-		return `${payload.user.username} just opened a merge request in ${payload.repository.homepage}`;
+		return `${payload.user.username} just opened a merge request in ${payload.object_attributes.url}`;
 	}
 
 	canHandle({ payload, event, config }: ReactionCanHandleOptions<MergeRequestPayload>): boolean {

src/schemas/gitlab/merge-request-payload.ts

@@ -14,5 +14,6 @@ export interface MergeRequestPayload {
 	};
 	object_attributes: {
 		state: 'merged' | 'created' | 'updated' | 'closed' | 'opened';
+		url: url;
 	};
 }

test/builders/gitlab/merge-request-payload-builder.ts

@@ -1,5 +1,6 @@
-import { MergeRequestPayload } from '../../../src/schemas/gitlab/merge-request-payload';
 import { merge } from 'lodash';
+import { DeepPartial } from 'utility-types';
+import { MergeRequestPayload } from '../../../src/schemas/gitlab/merge-request-payload';
 
 export class MergeRequestPayloadBuilder {
 	private payload: MergeRequestPayload = {
@@ -17,10 +18,11 @@ export class MergeRequestPayloadBuilder {
 		},
 		object_attributes: {
 			state: 'merged',
+			url: 'http://example.com/diaspora/merge_requests/1',
 		},
 	};
 
-	public with(payload: Partial<MergeRequestPayload>): MergeRequestPayloadBuilder {
+	public with(payload: DeepPartial<MergeRequestPayload>): MergeRequestPayloadBuilder {
 		merge(this.payload, payload);
 
 		return this;

test/reactions/gitlab/merge-request-merged.spec.ts

@@ -55,7 +55,7 @@ describe('MergeRequestMerged', () => {
 				expect.objectContaining({
 					twitchChat: {
 						notified: true,
-						message: `The merge request from ${payload.user.username} has been merged into ${payload.repository.homepage}`,
+						message: `The merge request from ${payload.user.username} has been merged into ${payload.object_attributes.url}`,
 					},
 				}),
 			);

test/reactions/gitlab/merge-request-opened.spec.ts

@@ -92,7 +92,7 @@ describe('MergeRequestOpened', () => {
 
 			expect(response).toEqual({
 				notified: true,
-				message: `${payload.user.username} just opened a merge request in ${payload.repository.homepage}`,
+				message: `${payload.user.username} just opened a merge request in ${payload.object_attributes.url}`,
 			});
 		});