test coverage improvements for InstructorResponsesViewBase

src/web/app/components/question-responses/per-question-view-responses/per-question-view-responses.component.spec.ts

@@ -21,6 +21,7 @@ import { CommentTableModel } from '../../comment-box/comment-table/comment-table
 import { RichTextEditorModule } from '../../rich-text-editor/rich-text-editor.module';
 import { TeammatesCommonModule } from '../../teammates-common/teammates-common.module';
 import { SingleResponseModule } from '../single-response/single-response.module';
+import testEventEmission from '../../../../test-helpers/test-event-emitter';
 
 describe('PerQuestionViewResponsesComponent', () => {
   let component: PerQuestionViewResponsesComponent;
@@ -143,4 +144,52 @@ describe('PerQuestionViewResponsesComponent', () => {
     expect(JSON.stringify(component.responsesToShow[0]))
       .toBe(JSON.stringify(responseOutput));
   });
+
+  it('triggerDeleteCommentEvent: should emit correct responseID and index to deleteCommentEvent', () => {
+    let emittedID: string | undefined;
+    let emittedIndex: number | undefined;
+    testEventEmission(component.deleteCommentEvent, (val) => { emittedID = val.responseId; emittedIndex = val.index; });
+
+    component.triggerDeleteCommentEvent('testID', 5);
+    expect(emittedID).toBe('testID');
+    expect(emittedIndex).toBe(5);
+  });
+
+  it('triggerUpdateCommentEvent: should emit correct responseID and index to updateCommentEvent', () => {
+    let emittedID: string | undefined;
+    let emittedIndex: number | undefined;
+    testEventEmission(component.updateCommentEvent, (val) => { emittedID = val.responseId; emittedIndex = val.index; });
+
+    component.triggerUpdateCommentEvent('testID2', 6);
+    expect(emittedID).toBe('testID2');
+    expect(emittedIndex).toBe(6);
+  });
+
+  it('triggerSaveNewCommentEvent: should emit correct responseID to saveNewCommentEvent', () => {
+    let emittedID: string | undefined;
+    testEventEmission(component.saveNewCommentEvent, (responseId) => { emittedID = responseId; });
+
+    component.triggerSaveNewCommentEvent('testID3');
+    expect(emittedID).toBe('testID3');
+  });
+
+  it('triggerModelChangeForSingleResponse: should emit correct instructorCommentTableModel Record to instructorCommentTableModelChange', () => {
+    let emittedRecord: Record<string, CommentTableModel> | undefined;
+    testEventEmission(component.instructorCommentTableModelChange, (record) => { emittedRecord = record; });
+
+    let testRecord: Record<string, CommentTableModel> = {'testID4': commentTableModel};
+
+    component.triggerModelChangeForSingleResponse('testID4', commentTableModel);
+    expect(emittedRecord).toStrictEqual(testRecord);
+  });
+
+  it('triggerModelChange: should emit correct instructorCommentTableModel Record to triggerModelChange', () => {
+    let emittedRecord: Record<string, CommentTableModel> | undefined;
+    testEventEmission(component.instructorCommentTableModelChange, (record) => { emittedRecord = record; });
+
+    let testRecord: Record<string, CommentTableModel> = {};
+
+    component.triggerModelChange(testRecord);
+    expect(emittedRecord).toStrictEqual(testRecord);
+  });
 });