Revert "fun: never gonna give you up"

This reverts commit 37a7880.

Author: kylehoehns

src/bot/__tests__/requestReview.test.ts

@@ -364,7 +364,7 @@ describe('requestReview', () => {
       expect(param.ack).toHaveBeenCalled();
     });
 
-    it.skip('should post a message to the interviewing channel', async () => {
+    it('should post a message to the interviewing channel', async () => {
       const { param } = await callCallback();
 
       expect(param.client.chat.postMessage).toHaveBeenCalledWith({

src/bot/requestReview.ts

@@ -7,7 +7,7 @@ import { App } from '@slack/bolt';
 import { Block, KnownBlock, PlainTextOption, View } from '@slack/types';
 import { blockUtils } from '@utils/blocks';
 import log from '@utils/log';
-import { bold, codeBlock, compose, italic, link, mention, ul } from '@utils/text';
+import { bold, codeBlock, compose, italic, mention, ul } from '@utils/text';
 import { PendingReviewer } from '@models/ActiveReview';
 import {
   ActionId,
@@ -257,7 +257,6 @@ export const requestReview = {
         ul(...languages),
         bold(`Candidate Type: ${candidateTypeDisplay}`),
         bold(`The review is needed by end of day ${deadlineDisplay}`),
-        link(':warning: Plagiarism Flag Detected', 'https://www.youtube.com/watch?v=dQw4w9WgXcQ'),
         candidateIdentifierValue ? italic(`Candidate Identifier: ${candidateIdentifierValue}`) : '',
       ),
     );

src/services/__tests__/ChatService.test.ts

@@ -57,7 +57,7 @@ describe('ChatService', () => {
     });
   });
 
-  describe.skip('sendRequestReviewMessage', () => {
+  describe('sendRequestReviewMessage', () => {
     it('should notify the given user that a request is ready for them to review', async () => {
       const client = buildMockWebClient();
       client.chat.postMessage = jest.fn().mockResolvedValue({

src/utils/RequestBuilder.ts

@@ -1,5 +1,5 @@
 import { ActionId, BlockId } from '@bot/enums';
-import { bold, compose, link, mention, ul } from '@utils/text';
+import { bold, compose, mention, ul } from '@utils/text';
 import { ActionsBlock, Block, SectionBlock } from '@slack/types';
 
 export const requestBuilder = {
@@ -56,7 +56,6 @@ export const requestBuilder = {
           ul(...languages),
           bold(`Candidate Type: ${candidateTypeDisplay}`),
           bold(`The review is needed by end of day ${deadlineDisplay}`),
-          link(':warning: Plagiarism Flag Detected', 'https://www.youtube.com/watch?v=dQw4w9WgXcQ'),
         ),
       },
     };

src/utils/text.ts

@@ -20,10 +20,6 @@ export function li(...items: string[]): string {
   return items.map((item, index) => `  ${index + 1}.  ${item}`).join('\n');
 }
 
-export function link(linkText: string, url: string) {
-  return `<${url}|${linkText}>`;
-}
-
 export function codeBlock(...lines: string[]): string {
   return ['```', ...lines, '```'].join('\n');
 }