Filter comments by content instead of actor (#519)

* Filter comments by content instead of actor * Remove dead actor code * WIP fix tests * Fix test
actions · Jul 12, 2021 · d901397 · d901397
1 parent 678bfc7
commit d901397
Show file tree

Hide file tree

Showing 11 changed files with 122 additions and 160 deletions.
diff --git a/__tests__/any-of-labels.spec.ts b/__tests__/any-of-labels.spec.ts
@@ -1139,7 +1139,6 @@ class IssuesProcessorBuilder {
   build(): IssuesProcessorMock {
     return new IssuesProcessorMock(
       this._options,
-      async () => 'abot',
       async p => (p === 1 ? this._issues : []),
       async () => [],
       async () => new Date().toDateString()

diff --git a/__tests__/assignees.spec.ts b/__tests__/assignees.spec.ts
@@ -48,7 +48,6 @@ describe('assignees options', (): void => {
   const setProcessor = () => {
     processor = new IssuesProcessorMock(
       opts,
-      async () => 'abot',
       async p => (p === 1 ? testIssueList : []),
       async () => [],
       async () => new Date().toDateString()

diff --git a/__tests__/classes/issues-processor-mock.ts b/__tests__/classes/issues-processor-mock.ts
@@ -6,7 +6,6 @@ import {IIssuesProcessorOptions} from '../../src/interfaces/issues-processor-opt
 export class IssuesProcessorMock extends IssuesProcessor {
   constructor(
     options: IIssuesProcessorOptions,
-    getActor?: () => Promise<string>,
     getIssues?: (page: number) => Promise<Issue[]>,
     listIssueComments?: (
       issueNumber: number,
@@ -19,10 +18,6 @@ export class IssuesProcessorMock extends IssuesProcessor {
   ) {
     super(options);
 
-    if (getActor) {
-      this.getActor = getActor;
-    }
-
     if (getIssues) {
       this.getIssues = getIssues;
     }