actions
/
stale
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add rebuilt index
Code scanning / CodeQL-Build (push) Failing after 35s Details

This commit is contained in:
Ross Brodbeck 2021-02-05 06:56:22 -05:00
parent d21d307fd8
commit 9d6f46564a
2 changed files with 19 additions and 31 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

45
__tests__/main.test.ts
View File

@ -2273,12 +2273,7 @@ test('processing an issue opened since 2 days and with the option "daysBeforeIss
let issueDate = new Date(); let issueDate = new Date();
issueDate.setDate(issueDate.getDate() - 2); issueDate.setDate(issueDate.getDate() - 2);
const TestIssueList: Issue[] = [ const TestIssueList: Issue[] = [
generateIssue( generateIssue(opts, 1, 'An issue with no label', issueDate.toDateString())
opts,
1,
'An issue with no label',
issueDate.toDateString(),
)
]; ];
const processor = new IssueProcessor( const processor = new IssueProcessor(
opts, opts,
@ -2287,10 +2282,10 @@ test('processing an issue opened since 2 days and with the option "daysBeforeIss
async (num, dt) => [], async (num, dt) => [],
async (issue, label) => new Date().toDateString() async (issue, label) => new Date().toDateString()
); );
// process our fake issue list // process our fake issue list
await processor.processIssues(1); await processor.processIssues(1);
expect(processor.staleIssues.length).toEqual(0); expect(processor.staleIssues.length).toEqual(0);
expect(processor.closedIssues.length).toEqual(0); expect(processor.closedIssues.length).toEqual(0);
}); });
@ -2305,12 +2300,7 @@ test('processing an issue opened since 2 days and with the option "daysBeforeIss
let issueDate = new Date(); let issueDate = new Date();
issueDate.setDate(issueDate.getDate() - 2); issueDate.setDate(issueDate.getDate() - 2);
const TestIssueList: Issue[] = [ const TestIssueList: Issue[] = [
generateIssue( generateIssue(opts, 1, 'An issue with no label', issueDate.toDateString())
opts,
1,
'An issue with no label',
issueDate.toDateString(),
)
]; ];
const processor = new IssueProcessor( const processor = new IssueProcessor(
opts, opts,
@ -2319,10 +2309,10 @@ test('processing an issue opened since 2 days and with the option "daysBeforeIss
async (num, dt) => [], async (num, dt) => [],
async (issue, label) => new Date().toDateString() async (issue, label) => new Date().toDateString()
); );
// process our fake issue list // process our fake issue list
await processor.processIssues(1); await processor.processIssues(1);
expect(processor.staleIssues.length).toEqual(1); expect(processor.staleIssues.length).toEqual(1);
expect(processor.closedIssues.length).toEqual(0); expect(processor.closedIssues.length).toEqual(0);
}); });
@ -2337,12 +2327,7 @@ test('processing an issue opened since 2 days and with the option "daysBeforeIss
let issueDate = new Date(); let issueDate = new Date();
issueDate.setDate(issueDate.getDate() - 2); issueDate.setDate(issueDate.getDate() - 2);
const TestIssueList: Issue[] = [ const TestIssueList: Issue[] = [
generateIssue( generateIssue(opts, 1, 'An issue with no label', issueDate.toDateString())
opts,
1,
'An issue with no label',
issueDate.toDateString(),
)
]; ];
const processor = new IssueProcessor( const processor = new IssueProcessor(
opts, opts,
@ -2351,10 +2336,10 @@ test('processing an issue opened since 2 days and with the option "daysBeforeIss
async (num, dt) => [], async (num, dt) => [],
async (issue, label) => new Date().toDateString() async (issue, label) => new Date().toDateString()
); );
// process our fake issue list // process our fake issue list
await processor.processIssues(1); await processor.processIssues(1);
expect(processor.staleIssues.length).toEqual(1); expect(processor.staleIssues.length).toEqual(1);
expect(processor.closedIssues.length).toEqual(0); expect(processor.closedIssues.length).toEqual(0);
}); });
@ -2385,10 +2370,10 @@ test('processing a pull request opened since 2 days and with the option "daysBef
async (num, dt) => [], async (num, dt) => [],
async (issue, label) => new Date().toDateString() async (issue, label) => new Date().toDateString()
); );
// process our fake issue list // process our fake issue list
await processor.processIssues(1); await processor.processIssues(1);
expect(processor.staleIssues.length).toEqual(0); expect(processor.staleIssues.length).toEqual(0);
expect(processor.closedIssues.length).toEqual(0); expect(processor.closedIssues.length).toEqual(0);
}); });
@ -2419,10 +2404,10 @@ test('processing a pull request opened since 2 days and with the option "daysBef
async (num, dt) => [], async (num, dt) => [],
async (issue, label) => new Date().toDateString() async (issue, label) => new Date().toDateString()
); );
// process our fake issue list // process our fake issue list
await processor.processIssues(1); await processor.processIssues(1);
expect(processor.staleIssues.length).toEqual(1); expect(processor.staleIssues.length).toEqual(1);
expect(processor.closedIssues.length).toEqual(0); expect(processor.closedIssues.length).toEqual(0);
}); });
@ -2453,10 +2438,10 @@ test('processing a pull request opened since 2 days and with the option "daysBef
async (num, dt) => [], async (num, dt) => [],
async (issue, label) => new Date().toDateString() async (issue, label) => new Date().toDateString()
); );
// process our fake issue list // process our fake issue list
await processor.processIssues(1); await processor.processIssues(1);
expect(processor.staleIssues.length).toEqual(1); expect(processor.staleIssues.length).toEqual(1);
expect(processor.closedIssues.length).toEqual(0); expect(processor.closedIssues.length).toEqual(0);
}); });

5
dist/index.js vendored
View File

@ -157,13 +157,16 @@ class IssueProcessor {
continue; // don't process exempt milestones continue; // don't process exempt milestones
} }
// should this issue be marked stale? // should this issue be marked stale?
const shouldBeStale = !IssueProcessor._updatedSince(issue.updated_at, this.options.daysBeforeStale); const shouldBeStale = !IssueProcessor._updatedSince(issue.updated_at, daysBeforeStale);
// determine if this issue needs to be marked stale first // determine if this issue needs to be marked stale first
if (!issue.isStale && shouldBeStale && shouldMarkAsStale) { if (!issue.isStale && shouldBeStale && shouldMarkAsStale) {
issueLogger.info(`Marking ${issueType} stale because it was last updated on ${issue.updated_at} and it does not have a stale label`); issueLogger.info(`Marking ${issueType} stale because it was last updated on ${issue.updated_at} and it does not have a stale label`);
yield this._markStale(issue, staleMessage, staleLabel, skipMessage); yield this._markStale(issue, staleMessage, staleLabel, skipMessage);
issue.isStale = true; // this issue is now considered stale issue.isStale = true; // this issue is now considered stale
} }
else if (!issue.isStale) {
issueLogger.info(`Not marking as stale: shouldBeStale=${shouldBeStale}, shouldMarkAsStale=${shouldMarkAsStale}`);
}
// process the issue if it was marked stale // process the issue if it was marked stale
if (issue.isStale) { if (issue.isStale) {
issueLogger.info(`Found a stale ${issueType}`); issueLogger.info(`Found a stale ${issueType}`);