Merge pull request #4 from ThingsO2/BET-3664-auto-close-merge-monom-manifests-p-rs

Bet 3664 auto close merge monom manifests p rs

Author: jilgue

.github/workflows/test.yaml

@@ -0,0 +1,60 @@
+name: Test
+on:
+  pull_request:
+    branches:
+      - main
+
+jobs:
+  test:
+    name: Test
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+
+      - name: Install nodejs
+        uses: actions/setup-node@v2
+        with:
+          node-version: '16'
+
+      - name: Cache dependencies
+        uses: actions/cache@v2
+        with:
+          path: 'node_modules'
+          key: ${{ hashFiles('yarn.lock') }}
+
+      - name: Install
+        run: yarn install --pure-lockfile
+
+      - name: Test
+        run: yarn test --coverage
+        env:
+          GITHUB_TOKEN: ${{ secrets.KINGKONG_GITHUB_TOKEN }}
+
+      - name: Get Package Data
+        id: read-package
+        run: |
+          echo ::set-output name=NAME::$(node -p "require('./package.json').name") | sed 's/@monom\///g'
+          echo ::set-output name=VERSION::$(node -p "require('./package.json').version")
+
+      - name: SonarQube Scan
+        uses: sonarsource/sonarqube-scan-action@master
+        env:
+          SONAR_TOKEN: ${{ secrets.SONARQUBE_TOKEN }}
+          SONAR_HOST_URL: ${{ secrets.SONARQUBE_HOST }}
+        with:
+          args: >
+            -Dsonar.projectKey=front-${{ steps.read-package.outputs.NAME }}
+            -Dsonar.projectName=${{ steps.read-package.outputs.NAME }}
+            -Dsonar.projectVersion=${{ steps.read-package.outputs.VERSION }}
+            -Dsonar.javascript.lcov.reportPaths=coverage/lcov.info
+            -Dsonar.sources=src
+            -Dsonar.tests=__tests__
+            -Dsonar.exclusions=src/__mocks__/**.*
+
+      - name: SonarQube Quality Gate check
+        uses: sonarsource/sonarqube-quality-gate-action@master
+        env:
+          SONAR_TOKEN: ${{ secrets.SONARQUBE_TOKEN }}
+        with:
+          scanMetadataReportFile: .scannerwork/report-task.txt

.gitignore

@@ -0,0 +1 @@
+node_modules

__tests__/getLabels.test.ts

@@ -0,0 +1,39 @@
+jest.mock('@actions/core')
+import { Octokit } from "@octokit/core"
+import { RequestError } from "@octokit/request-error"
+import { getLabels } from '../src/getLabels'
+
+const octokit = new Octokit({
+    auth: process.env.GITHUB_TOKEN
+})
+
+const nooctokit = new Octokit({
+    auth: '1234567890'
+})
+
+const prNumber = 15
+const repoName = 'monom-manifests'
+const owner = 'ThingsO2'
+
+test('get labels', async () => {
+    const labels = await getLabels(octokit, prNumber, repoName, owner)
+    expect(labels).toEqual(['test'])
+})
+
+test('get labels not found', async () => {
+    try {
+        await getLabels(octokit, 999, repoName, owner)
+    } catch (error) {
+        expect(error).toBeInstanceOf(RequestError)
+        expect(error).toHaveProperty('status', 404)
+    }
+})
+
+test('get labels error', async () => {
+    try {
+        await getLabels(nooctokit, prNumber, repoName, owner)
+    } catch (error) {
+        expect(error).toBeInstanceOf(RequestError)
+        expect(error).toHaveProperty('status', 401)
+    }
+})

__tests__/main.test.ts

@@ -0,0 +1,28 @@
+jest.mock('@actions/core')
+import * as core from '@actions/core'
+import { Octokit } from "@octokit/core"
+import { main } from '../src/main'
+
+const octokit = new Octokit({
+    auth: process.env.GITHUB_TOKEN
+})
+
+const nooctokit = new Octokit({
+    auth: '1234567890'
+})
+
+const input = {
+    prNumber: 135,
+    repoName: 'monom-manifests',
+    owner: 'ThingsO2'
+}
+
+test('main', async () => {
+    const mainResult = await main(octokit, input, false)
+    expect(mainResult).toBe(core.ExitCode.Success)
+})
+
+test('main failed', async () => {
+    const mainResult = await main(nooctokit, input, false)
+    expect(mainResult).toBe(core.ExitCode.Failure)
+})

__tests__/mergePR.test.ts

@@ -0,0 +1,41 @@
+jest.mock('@actions/core')
+import { Octokit } from "@octokit/core"
+import { RequestError } from "@octokit/request-error"
+import { mergePR } from '../src/mergePR'
+
+const octokit = new Octokit({
+    auth: process.env.GITHUB_TOKEN
+})
+
+const nooctokit = new Octokit({
+    auth: '1234567890'
+})
+
+const prNumber = 8
+const repoName = 'github-action-close-code-pr'
+const owner = 'ThingsO2'
+
+test('merge PR', async () => {
+    const mergeResult = await mergePR(octokit, prNumber, repoName, owner)
+    expect(mergeResult).toBeDefined()
+    expect(mergeResult.merged).toBeTruthy()
+    expect(mergeResult.message).toEqual('Pull Request successfully merged')
+})
+
+test('merge PR already merged', async () => {
+    try {
+        await mergePR(octokit, 5, repoName, owner)
+    } catch (error) {
+        expect(error).toBeInstanceOf(RequestError)
+        expect(error).toHaveProperty('status', 405)
+    }
+})
+
+test('merge PR no token', async () => {
+    try {
+        await mergePR(nooctokit, prNumber, repoName, owner)
+    } catch (error) {
+        expect(error).toBeInstanceOf(RequestError)
+        expect(error).toHaveProperty('status', 401)
+    }
+})

__tests__/searchCodePR.test.ts

@@ -0,0 +1,38 @@
+jest.mock('@actions/core')
+import { Octokit } from "@octokit/core"
+import { searchCodePR } from '../src/searchCodePR'
+
+const octokit = new Octokit({
+    auth: process.env.GITHUB_TOKEN
+})
+
+const nooctokit = new Octokit({
+    auth: '1234567890'
+})
+
+const prNumber = 135
+const repoName = 'monom-manifests'
+const owner = 'ThingsO2'
+
+test('search code PR', async () => {
+    const res = await searchCodePR(octokit, prNumber, repoName, owner)
+    expect(res).toBeDefined()
+    expect(res?.number).toBe(36)
+    expect(res?.head.repo.name).toBe('dp-monom-2-awm')
+})
+
+test('search code PR not found', async () => {
+    try {
+        await searchCodePR(octokit, 15, repoName, owner)
+    } catch (error) {
+        expect(error).toHaveProperty('status', 404)
+    }
+})
+
+test('search code PR error', async () => {
+    try {
+        await searchCodePR(nooctokit, prNumber, repoName, owner)
+    } catch (error) {
+        expect(error).toHaveProperty('status', 401)
+    }
+})

__tests__/searchPRwithLabels.test.ts

@@ -0,0 +1,32 @@
+jest.mock('@actions/core')
+import { Octokit } from "@octokit/core"
+import { searchPRwithLabels } from '../src/searchPRwithLabels'
+
+const octokit = new Octokit({
+    auth: process.env.GITHUB_TOKEN
+})
+
+const nooctokit = new Octokit({
+    auth: '1234567890'
+})
+
+const repoName = 'monom-manifests'
+const owner = 'ThingsO2'
+
+test('search PR with Labels', async () => {
+    const issue = await searchPRwithLabels(octokit, repoName, owner, ['test'])
+    expect(issue).toEqual([15])
+})
+
+test('search PR with Labels no found', async () => {
+    const issue = await searchPRwithLabels(octokit, repoName, owner, ['test-no-found'])
+    expect(issue).toEqual([])
+})
+
+test('search PR with Labels error', async () => {
+    try {
+        await searchPRwithLabels(nooctokit, repoName, owner, ['test'])
+    } catch (error) {
+        expect(error).toHaveProperty('status', 401)
+    }
+})

action.yml

@@ -0,0 +1,18 @@
+name: 'Close the original code PR when the app is deployed'
+description: 'Close the original code PR when the app is deployed'
+inputs:
+  pr_number:
+    description: 'The PR number to close'
+    required: true
+  repo_name:
+    description: 'The repo name'
+    required: true
+  owner:
+    description: 'The repo owner'
+    required: true
+runs:
+  using: 'node16'
+  main: 'dist/index.js'
+branding:
+  icon: 'lock'
+  color: 'gray-dark'