Merge pull request #190 from streamdevs/179-issue-url

feat: send issue url to Twitch Chat

Author: SantiMA10

src/reactions/github/issue-assigned.ts

@@ -7,7 +7,7 @@ export class IssueAssigned extends Reaction {
 	}
 
 	getTwitchChatMessage({ payload }: ReactionHandleOptions<IssuePayload>): string {
-		return `${payload.assignee?.login} has a new assigned issue in ${payload.repository.html_url}`;
+		return `${payload.assignee?.login} has a new assigned issue in ${payload.issue.html_url}`;
 	}
 
 	canHandle({ payload, event, config }: ReactionCanHandleOptions<IssuePayload>): boolean {

src/reactions/github/issue-opened.ts

@@ -6,7 +6,7 @@ export class IssueOpened extends Reaction<IssuePayload> {
 		return `*${payload.sender.login}* opened a issue in *${payload.repository.full_name}*`;
 	}
 	getTwitchChatMessage({ payload }: ReactionHandleOptions<IssuePayload>): string {
-		return `${payload.sender.login} opened a issue in ${payload.repository.html_url}`;
+		return `${payload.sender.login} opened a issue in ${payload.issue.html_url}`;
 	}
 	canHandle({ payload, event }: ReactionCanHandleOptions<IssuePayload>): boolean {
 		return event === 'issues' && payload.action === 'opened';

src/schemas/github/issue-payload.ts

@@ -1,6 +1,9 @@
 import { RepositoryWebhookPayload } from './repository-webhook-payload';
 
+type url = string;
+
 export interface IssuePayload extends RepositoryWebhookPayload {
 	action: string;
 	assignee?: { login: string };
+	issue: { html_url: url };
 }

test/builders/github/issue-payload-builder.ts

@@ -11,6 +11,7 @@ export class IssuePayloadBuilder {
 		},
 		sender: { login: internet.userName() },
 		assignee: { login: internet.userName() },
+		issue: { html_url: internet.url() },
 	};
 
 	public with(payload: Partial<IssuePayload>): IssuePayloadBuilder {

test/reactions/github/issue-assigned.spec.ts

@@ -37,7 +37,7 @@ describe('IssueAssigned', () => {
 			await subject.handle({ payload });
 
 			expect(twitchChat.send).toHaveBeenCalledWith(
-				`${payload.assignee?.login} has a new assigned issue in ${payload.repository.html_url}`,
+				`${payload.assignee?.login} has a new assigned issue in ${payload.issue.html_url}`,
 			);
 		});
 	});

test/reactions/github/issue-opened.spec.ts

@@ -2,6 +2,7 @@ import { TwitchChat } from '../../../src/services/TwitchChat';
 import { StreamLabs } from '../../../src/services/StreamLabs';
 import { IssueOpened } from '../../../src/reactions/github/issue-opened';
 import { IssuePayload } from '../../../src/schemas/github/issue-payload';
+import { IssuePayloadBuilder } from '../../builders/github/issue-payload-builder';
 
 describe('IssueOpened', () => {
 	let twitchChat: TwitchChat;
@@ -16,16 +17,18 @@ describe('IssueOpened', () => {
 		let payload: IssuePayload;
 
 		beforeEach(() => {
-			payload = {
-				action: 'opened',
-				repository: {
-					html_url: 'https://github.com/streamdevs/webhook',
-					full_name: 'streamdevs/webhook',
-				},
-				sender: {
-					login: 'orestes',
-				},
-			};
+			payload = new IssuePayloadBuilder()
+				.with({
+					action: 'opened',
+					repository: {
+						html_url: 'https://github.com/streamdevs/webhook',
+						full_name: 'streamdevs/webhook',
+					},
+					sender: {
+						login: 'orestes',
+					},
+				})
+				.getInstance();
 		});
 
 		it('calls StreamLabs with the expected message', async () => {
@@ -44,7 +47,7 @@ describe('IssueOpened', () => {
 			await subject.handle({ payload });
 
 			expect(twitchChat.send).toHaveBeenCalledWith(
-				`${payload.sender.login} opened a issue in ${payload.repository.html_url}`,
+				`${payload.sender.login} opened a issue in ${payload.issue.html_url}`,
 			);
 		});
 	});

test/routes/github/issues.spec.ts

@@ -3,6 +3,7 @@ import { IssuePayload } from '../../../src/schemas/github/issue-payload';
 import { initServer } from '../../../src/server';
 import { StreamLabs } from '../../../src/services/StreamLabs';
 import { TwitchChat } from '../../../src/services/TwitchChat';
+import { IssuePayloadBuilder } from '../../builders/github/issue-payload-builder';
 
 describe('POST /github', () => {
 	let payload: IssuePayload;
@@ -17,15 +18,17 @@ describe('POST /github', () => {
 		twitchChatSpy = jest.spyOn(TwitchChat.prototype, 'send');
 		twitchChatSpy.mockImplementationOnce(jest.fn());
 
-		payload = {
-			action: 'assigned',
-			assignee: { login: 'SantiMA10' },
-			sender: { login: 'SantiMA10' },
-			repository: {
-				html_url: 'https://github.com/streamdevs/webhook',
-				full_name: 'streamdevs/webhook',
-			},
-		};
+		payload = new IssuePayloadBuilder()
+			.with({
+				action: 'assigned',
+				assignee: { login: 'SantiMA10' },
+				sender: { login: 'SantiMA10' },
+				repository: {
+					html_url: 'https://github.com/streamdevs/webhook',
+					full_name: 'streamdevs/webhook',
+				},
+			})
+			.getInstance();
 	});
 
 	it('returns a 200 OK', async () => {