Merge pull request #128 from juliojimenez/checks-installation-id

fix: Add Installation struct to CheckSuitePayload and CheckRunPayload

.github/workflows/workflow.yml

@@ -0,0 +1,52 @@
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+
+name: Test
+jobs:
+  lint:
+    name: Lint
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v2
+
+      - name: Run linter
+        uses: golangci/golangci-lint-action@v2
+        with:
+          version: v1.35.2
+
+  test:
+    name: Test
+    strategy:
+      matrix:
+        go-version: [1.14.x, 1.15.x]
+        os: [ubuntu-latest, macos-latest]
+    runs-on: ${{ matrix.os }}
+    steps:
+      - name: Install Go
+        uses: actions/setup-go@v2
+        with:
+          go-version: ${{ matrix.go-version }}
+
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Restore cache
+        uses: actions/cache@v2
+        with:
+          path: ~/go/pkg/mod
+          key: ${{ runner.os }}-v1-go-${{ hashFiles('**/go.sum') }}
+          restore-keys: |
+            ${{ runner.os }}-v1-go-
+
+      - name: Run tests
+        run: go test -race -covermode=atomic -coverprofile="coverage.out" ./...
+
+      - name: Upload coverage report to Coveralls
+        if: matrix.os == 'ubuntu-latest' && matrix.go-version == '1.15.x'
+        uses: shogo82148/actions-goveralls@v1
+        with:
+          path-to-profile: coverage.out

.travis.yml

@@ -1,28 +0,0 @@
-language: go
-go:
-  - 1.15.6
-  - tip
-matrix:
-  allow_failures:
-    - go: tip
-
-notifications:
-  email:
-    recipients: dean.karn@gmail.com
-    on_success: change
-    on_failure: always
-
-before_install:
-  - go get -u github.com/mattn/goveralls
-
-before_script:
-  - go mod download
-
-script:
-  - make lint
-  - make test
-
-after_success: |
-  [ $TRAVIS_GO_VERSION = 1.15.6 ] &&
-  go test ./... -covermode=count -coverprofile=coverage.out &&
-  goveralls -coverprofile=coverage.out -service travis-ci -repotoken $COVERALLS_TOKEN

Makefile

@@ -1,5 +1,7 @@
 GOPATH=$(shell go env GOPATH)
 
+all: lint test
+
 linters-install:
 	@echo "+ $@"
 	@$(GOPATH)/bin/golangci-lint --version >/dev/null 2>&1 || { \
@@ -12,6 +14,8 @@ lint: linters-install
 	$(GOPATH)/bin/golangci-lint run ./...
 
 test:
-	GO111MODULE=on go test -cover -race ./...
+	@echo "+ $@"
+	GO111MODULE=on go test -covermode=atomic -race ./...
 
 .PHONY: test lint linters-install
+.DEFAULT_GOAL := all

README.md

@@ -1,8 +1,8 @@
 Library webhooks
 ================
 <img align="right" src="https://raw.githubusercontent.com/go-playground/webhooks/v6/logo.png">![Project status](https://img.shields.io/badge/version-6.0.0-green.svg)
-[![Build Status](https://travis-ci.org/go-playground/webhooks.svg?branch=v6)](https://travis-ci.org/go-playground/webhooks)
+[![Test](https://github.com/go-playground/webhooks/workflows/Test/badge.svg?branch=master)](https://github.com/go-playground/webhooks/actions)
-[![Coverage Status](https://coveralls.io/repos/go-playground/webhooks/badge.svg?branch=v6&service=github)](https://coveralls.io/github/go-playground/webhooks?branch=v6)
+[![Coverage Status](https://coveralls.io/repos/go-playground/webhooks/badge.svg?branch=master&service=github)](https://coveralls.io/github/go-playground/webhooks?branch=master)
 [![Go Report Card](https://goreportcard.com/badge/go-playground/webhooks)](https://goreportcard.com/report/go-playground/webhooks)
 [![GoDoc](https://godoc.org/github.com/go-playground/webhooks/v6?status.svg)](https://godoc.org/github.com/go-playground/webhooks/v6)
 ![License](https://img.shields.io/dub/l/vibe-d.svg)

github/payload.go

@@ -187,6 +187,9 @@ type CheckRunPayload struct {
 		Watchers         int64     `json:"watchers"`
 		DefaultBranch    string    `json:"default_branch"`
 	} `json:"repository"`
+	Installation struct {
+		ID int64 `json:"id"`
+	} `json:"installation,omitempty"`
 	Sender struct {
 		Login             string `json:"login"`
 		ID                int64  `json:"id"`
@@ -359,6 +362,9 @@ type CheckSuitePayload struct {
 		Watchers         int64     `json:"watchers"`
 		DefaultBranch    string    `json:"default_branch"`
 	} `json:"repository"`
+	Installation struct {
+		ID int64 `json:"id"`
+	} `json:"installation,omitempty"`
 	Sender struct {
 		Login             string `json:"login"`
 		ID                int64  `json:"id"`

gitlab/gitlab_test.go

@@ -274,15 +274,6 @@ func TestJobHooks(t *testing.T) {
 		filename string
 		headers  http.Header
 	}{
-		{
-			name:     "JobEvent",
-			events:   []Event{JobEvents},
-			typ:      JobEventPayload{},
-			filename: "../testdata/gitlab/job-event.json",
-			headers: http.Header{
-				"X-Gitlab-Event": []string{"Job Hook"},
-			},
-		},
 		{
 			name:     "JobEvent",
 			events:   []Event{JobEvents, BuildEvents},

gitlab/payload.go

@@ -57,6 +57,7 @@ type MergeRequestEventPayload struct {
 	Project          Project          `json:"project"`
 	Repository       Repository       `json:"repository"`
 	Labels           []Label          `json:"labels"`
+	Assignees        []Assignee       `json:"assignees"`
 }
 
 // PushEventPayload contains the information for GitLab's push event
@@ -449,9 +450,11 @@ type MergeRequest struct {
 
 // Assignee contains all of the GitLab assignee information
 type Assignee struct {
+	ID        int64  `json:"id"`
 	Name      string `json:"name"`
 	Username  string `json:"username"`
 	AvatarURL string `json:"avatar_url"`
+	Email     string `json:"email"`
 }
 
 // StDiff contains all of the GitLab diff information

testdata/gitlab/merge-request-event.json

@@ -136,5 +136,14 @@
         "group_id": 41
       }]
     }
-  }
+  },
+  "assignees": [
+    {
+      "id": 6,
+      "name": "User1",
+      "username": "user1",
+      "avatar_url": "http://www.gravatar.com/avatar/e64c7d89f26bd1972efa854d13d7dd61?s=40\u0026d=identicon",
+      "email": "user1@gmail.com"
+    }
+  ]
 }