build(deps): bump probot from 10.9.5 to 10.18.0 (#151)

* build(deps): bump probot from 10.9.5 to 10.18.0

Bumps [probot](https://github.com/probot/probot) from 10.9.5 to 10.18.0.
- [Release notes](https://github.com/probot/probot/releases)
- [Commits](probot/probot@v10.9.5...v10.18.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

* test: fix broken cases

* test: refine deprecation

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Co-authored-by: hi-rustin <rustin.liu@gmail.com>

Author: dependabot-preview[bot]

package-lock.json

@@ -31,7 +31,7 @@
     "http-status-codes": "^2.1.4",
     "lodash": "^4.17.20",
     "mysql": "^2.18.1",
-    "probot": "^10.9.5",
+    "probot": "^10.18.0",
     "probot-commands-pro": "^1.0.1",
     "reflect-metadata": "^0.1.13",
     "typedi": "^0.8.0",

package.json

@@ -32,7 +32,7 @@ async function constructPullFormatQuery(
 ): Promise<PullFormatQuery> {
   const { number } = context.payload.pull_request;
 
-  const { data: filesData } = await context.github.pulls.listFiles({
+  const { data: filesData } = await context.octokit.pulls.listFiles({
     ...context.issue(),
     pull_number: number,
   });
@@ -82,7 +82,7 @@ async function checkPullFormat(context: Context, pullService: PullService) {
         reply.message
       );
       // @ts-ignore
-      await context.github.repos.createStatus({
+      await context.octokit.repos.createStatus({
         ...context.repo(),
         ...status,
       });
@@ -96,7 +96,7 @@ async function checkPullFormat(context: Context, pullService: PullService) {
         reply.message
       );
       // @ts-ignore
-      await context.github.repos.createStatus({
+      await context.octokit.repos.createStatus({
         ...context.repo(),
         ...status,
       });
@@ -111,7 +111,7 @@ async function checkPullFormat(context: Context, pullService: PullService) {
         combineReplay(reply)
       );
       // @ts-ignore
-      await context.github.repos.createStatus({
+      await context.octokit.repos.createStatus({
         ...context.repo(),
         ...status,
       });
@@ -132,7 +132,7 @@ async function createOrUpdateComment(
   body: string
 ) {
   // List all comment.
-  const { data: comments } = await context.github.issues.listComments({
+  const { data: comments } = await context.octokit.issues.listComments({
     ...context.issue(),
   });
 
@@ -141,7 +141,7 @@ async function createOrUpdateComment(
   });
 
   if (botComment === undefined) {
-    await context.github.issues.createComment(context.issue({ body }));
+    await context.octokit.issues.createComment(context.issue({ body }));
   } else {
     // Update.
     botComment.body = body;
@@ -150,7 +150,7 @@ async function createOrUpdateComment(
       ...botComment,
       comment_id: botComment.id,
     };
-    await context.github.issues.updateComment(comment);
+    await context.octokit.issues.updateComment(comment);
   }
 }
 
@@ -167,21 +167,21 @@ async function updateSigInfo(context: Context, sigService: SigService) {
   switch (reply.status) {
     case Status.Failed: {
       context.log.error("Update sig info.", files);
-      await context.github.issues.createComment(
+      await context.octokit.issues.createComment(
         context.issue({ body: reply.message })
       );
       break;
     }
     case Status.Success: {
       context.log.info("Update sig info.", files);
-      await context.github.issues.createComment(
+      await context.octokit.issues.createComment(
         context.issue({ body: reply.message })
       );
       break;
     }
     case Status.Problematic: {
       context.log.warn("Update sig info has some problems.", files);
-      await context.github.issues.createComment(
+      await context.octokit.issues.createComment(
         context.issue({ body: combineReplay(reply) })
       );
     }

src/events/pull/index.ts

@@ -10,16 +10,24 @@ import { PullMessage } from "./services/messages/PullMessage";
 import { getSig } from "./api/sig";
 import { listOwners } from "./api/pull";
 import { handlePullRequestEvents } from "./events/pull";
+import { Router } from "express";
 
 const commands = require("probot-commands-pro");
 const bodyParser = require("body-parser");
 const cors = require("cors");
 
-export = (app: Application) => {
+export = (
+  app: Application,
+  {
+    getRouter,
+  }: {
+    getRouter: (path?: string) => Router;
+  }
+) => {
   useContainer(Container);
 
   // Get an express router to expose new HTTP endpoints.
-  const router = app.route("/ti-community-bot");
+  const router = getRouter("/ti-community-bot");
   router.use(bodyParser.json());
   router.use(cors());
 
@@ -41,7 +49,7 @@ export = (app: Application) => {
 
       // Ping command.
       commands(app, "ping", async (context: Context) => {
-        await context.github.issues.createComment(
+        await context.octokit.issues.createComment(
           context.issue({ body: "pong! I am community bot." })
         );
       });

src/index.ts

@@ -56,13 +56,13 @@ describe("Pull API", () => {
   const team_slug = "bots-test";
   const installationId = 2;
 
-  beforeEach(() => {
+  beforeEach(async () => {
     // Mock the db connection.
     typeorm.createConnection = jest.fn().mockResolvedValue(null);
     nock.disableNetConnect();
     // Create probot.
     const probot = new Probot({
-      id: 123,
+      appId: 123,
       privateKey,
       // disable request throttling and retries for testing
       Octokit: ProbotOctokit.defaults({
@@ -72,13 +72,11 @@ describe("Pull API", () => {
     });
 
     // Get github client.
-    probot.load(async (app) => {
-      github = await app.auth(installationId);
-      github.config.get = jest.fn().mockReturnValue({
-        config: {
-          maintainerTeamSlug: team_slug,
-        },
-      });
+    github = await probot.auth(installationId);
+    github.config.get = jest.fn().mockReturnValue({
+      config: {
+        maintainerTeamSlug: team_slug,
+      },
     });
   });
 

test/api/pull/index.test.ts

@@ -25,7 +25,7 @@ describe("My Probot app", () => {
     typeorm.createConnection = jest.fn().mockResolvedValue(null);
     nock.disableNetConnect();
     probot = new Probot({
-      id: 123,
+      appId: 123,
       privateKey,
       // disable request throttling and retries for testing
       Octokit: ProbotOctokit.defaults({