fix: return pull number in create-pr (#132)

* log the json result

* extract pull number

* cleanup

* chore: update tests

src/helpers/create-pr.ts

@@ -23,12 +23,14 @@ export const createPr = async ({ title, body }: CreatePR) => {
   const {
     data: { default_branch }
   } = await octokit.repos.get({ ...context.repo });
-  return octokit.pulls.create({
+  const result = await octokit.pulls.create({
     title,
     head: context.ref.replace('refs/heads/', ''),
     base: default_branch,
     body,
     maintainer_can_modify: true,
     ...context.repo
   });
+  const pullNumber = result?.data?.number;
+  return pullNumber;
 };

test/helpers/create-pr.test.ts

@@ -32,23 +32,29 @@ jest.mock('@actions/github', () => ({
     default_branch: 'default branch'
   }
 }));
+(octokit.pulls.create as unknown as Mocktokit).mockImplementation(async () => ({
+  data: {
+    number: 100
+  }
+}));
 
 describe('createPr', () => {
   const title = 'title';
   const body = 'body';
 
-  beforeEach(() => {
-    createPr({
+  it('should call repos get with correct params', async () => {
+    await createPr({
       title,
       body
     });
-  });
-
-  it('should call repos get with correct params', () => {
     expect(octokit.repos.get).toHaveBeenCalledWith({ ...context.repo });
   });
 
-  it('should call create with correct params', () => {
+  it('should call create with correct params', async () => {
+    await createPr({
+      title,
+      body
+    });
     expect(octokit.pulls.create).toHaveBeenCalledWith({
       title,
       head: 'source',
@@ -60,4 +66,12 @@ describe('createPr', () => {
       ...context.repo
     });
   });
+
+  it('should return the pull number', async () => {
+    const result = await createPr({
+      title,
+      body
+    });
+    expect(result).toBe(100);
+  });
 });