Add label 'new user' on PRs for new GitGitGadget users

Suggested (among other suggestions) in
gitgitgadget#414 (comment)

Signed-off-by: Johannes Schindelin <[email protected]>

Author: dscho

lib/ci-helper.ts

@@ -796,6 +796,8 @@ export class CIHelper {
             const welcome = (await readFile("res/WELCOME.md")).toString()
                     .replace(/\${username}/g, pr.author);
             await this.github.addPRComment(pullRequestURL, welcome);
+
+            await this.github.addPRLabels(pullRequestURL, ["new user"]);
         }
 
         const commitOkay = await this.checkCommits(pr, addComment);

tests/ci-helper.test.ts

@@ -42,6 +42,7 @@ class TestCIHelper extends CIHelper {
     public ghGlue: GitHubGlue;      // not readonly reference
     public addPRCommentCalls: string[][]; // reference mock.calls
     public updatePRCalls: string[][]; // reference mock.calls
+    public addPRLabelsCalls: any[][]; // reference mock.calls
 
     public constructor(workDir?: string, debug = false, gggDir = ".") {
         super(workDir, debug, gggDir);
@@ -61,6 +62,12 @@ class TestCIHelper extends CIHelper {
         this.ghGlue.updatePR = updatePR;
         this.updatePRCalls = updatePR.mock.calls;
 
+        const addPRLabels = jest.fn( async (_:string, labels: string[]):
+            // eslint-disable-next-line @typescript-eslint/require-await
+            Promise<string[]> => labels );
+        this.ghGlue.addPRLabels = addPRLabels;
+        this.addPRLabelsCalls = addPRLabels.mock.calls;
+
         // need keys to authenticate
         // this.ghGlue.ensureAuthenticated = async (): Promise<void> => {};
     }
@@ -887,6 +894,7 @@ test("handle push/comment too many commits fails", async () => {
 
     expect(ci.addPRCommentCalls[0][1]).toMatch(/Welcome/);
     expect(ci.addPRCommentCalls[1][1]).toMatch(failMsg);
+    expect(ci.addPRLabelsCalls[0][1]).toEqual(["new user"]);
 });
 
 test("handle push/comment merge commits fails", async () => {
@@ -990,6 +998,7 @@ test("handle push/comment merge commits fails", async () => {
 
     expect(ci.addPRCommentCalls[0][1]).toMatch(/Welcome/);
     expect(ci.addPRCommentCalls[1][1]).toMatch(commits[0].commit);
+    expect(ci.addPRLabelsCalls[0][1]).toEqual(["new user"]);
     ci.addPRCommentCalls.length = 0;
 
     // Test Multiple merges