Merge pull request #61 from hellofresh/feature/github-actions-pipeline

Github Actions pipeline instead of Travis

Author: vgarvardt

.github/workflows/testing.yml

@@ -0,0 +1,49 @@
+name: Testing
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  lint:
+    name: Lint
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v2
+      - name: golangci-lint
+        uses: golangci/golangci-lint-action@v2
+
+  test:
+    name: Test
+    runs-on: ubuntu-latest
+    needs: [ lint ]
+
+    services:
+      postgres:
+        image: postgres:10
+        ports:
+          - "5432"
+        env:
+          POSTGRES_USER: goengine
+          POSTGRES_PASSWORD: goengine
+          POSTGRES_DB: goengine
+        options: >-
+          --health-cmd pg_isready
+          --health-interval 10s
+          --health-timeout 5s
+          --health-retries 5
+
+    steps:
+      - name: Set up Go
+        uses: actions/setup-go@v2
+        with:
+          go-version: ^1.16
+      - name: Check out code
+        uses: actions/checkout@v2
+      - name: Run tests
+        run: go test -cover ./... -coverprofile=coverage.txt -covermode=atomic
+        env:
+          POSTGRES_DSN: postgres://goengine:goengine@localhost:${{ job.services.postgres.ports[5432] }}/goengine?sslmode=disable

.golangci.yml

@@ -7,17 +7,16 @@ run:
 
 linters:
   enable:
-    - maligned
+    - govet
     - deadcode
     - errcheck
     - gosec
     - goconst
     - gocyclo
     - gofmt
     - goimports
-    - golint
+    - revive
     - ineffassign
-    - interfacer
     - staticcheck
     - structcheck
     - unconvert

.travis.yml

@@ -66,10 +66,6 @@ func TestIDFromString(t *testing.T) {
 				"some string",
 				"some string",
 			},
-			{
-				"UUID with missing minuses",
-				"f4ec75dbc0b04b00a04fa0d9ed18e9fb",
-			},
 		}
 
 		for _, testCase := range testCases {

aggregate/aggregate_test.go

@@ -52,7 +52,7 @@ func isSupportedOperator(value interface{}, operator metadata.Operator) bool {
 	switch operator {
 	case metadata.Equals,
 		metadata.NotEquals:
-		switch value.(type) {	
+		switch value.(type) {
 		{{- range $i, $e := .Types }}
 		{{- if eq $i 0 }}
 		case {{ $e }},
@@ -195,7 +195,7 @@ func main() {
 		panic(err)
 	}
 
-	/* #nosec G302 */
+	/* #nosec */
 	f, err := os.OpenFile(matcherPath, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, 0644)
 	if err != nil {
 		panic(err)

driver/inmemory/internal/generate_matcher/main.go

@@ -3,12 +3,12 @@ package postgres
 import (
 	"context"
 	"database/sql"
+	"errors"
 	"fmt"
 	"strings"
 
 	"github.com/hellofresh/goengine"
 	driverSQL "github.com/hellofresh/goengine/driver/sql"
-	"github.com/pkg/errors"
 )
 
 var _ driverSQL.AggregateProjectorStorage = &AdvisoryLockAggregateProjectionStorage{}

driver/sql/postgres/advisory_lock_aggregate_projection_storage.go

@@ -3,11 +3,11 @@ package sql
 import (
 	"context"
 	"database/sql"
+	"errors"
 	"time"
 
 	"github.com/mailru/easyjson/jlexer"
 	"github.com/mailru/easyjson/jwriter"
-	"github.com/pkg/errors"
 
 	"github.com/hellofresh/goengine"
 )
@@ -37,7 +37,7 @@ type (
 
 	// ProjectionStateSerialization is an interface describing how a projection state can be initialized, serialized/encoded anf deserialized/decoded
 	ProjectionStateSerialization interface {
-		// init initializes the state
+		// Init initializes the state
 		Init(ctx context.Context) (interface{}, error)
 
 		// DecodeState reconstitute the projection state based on the provided state data
@@ -156,7 +156,7 @@ type nopProjectionStateSerialization struct {
 }
 
 // DecodeState reconstitute the projection state based on the provided state data
-func (nopProjectionStateSerialization) DecodeState(data []byte) (interface{}, error) {
+func (nopProjectionStateSerialization) DecodeState([]byte) (interface{}, error) {
 	return nil, nil
 }
 

driver/sql/projection.go

@@ -2,11 +2,11 @@ package sql
 
 import (
 	"context"
+	"errors"
 	"runtime"
 	"sync"
 
 	"github.com/hellofresh/goengine"
-	"github.com/pkg/errors"
 )
 
 type (

driver/sql/projection_notification_processor.go

@@ -3,9 +3,10 @@ package sql
 import (
 	"context"
 	"database/sql"
+	"errors"
+	"fmt"
 
 	"github.com/hellofresh/goengine"
-	"github.com/pkg/errors"
 )
 
 // Ensure the notificationProjector.Execute is a ProjectionTrigger
@@ -210,7 +211,7 @@ func wrapProjectionHandlerToTrapError(handler goengine.MessageHandler) goengine.
 			case error:
 				err = x
 			default:
-				err = errors.Errorf("unknown panic: (%T) %v", x, x)
+				err = fmt.Errorf("unknown panic: (%T) %v", x, x)
 			}
 
 			handlerErr = NewProjectionHandlerError(err)

driver/sql/projector_notification.go

@@ -4,13 +4,14 @@ package sql
 
 import (
 	"context"
+	"errors"
 	"testing"
 
-	"github.com/hellofresh/goengine"
-	"github.com/hellofresh/goengine/aggregate"
-	"github.com/pkg/errors"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
+
+	"github.com/hellofresh/goengine"
+	"github.com/hellofresh/goengine/aggregate"
 )
 
 func TestWrapProjectionHandlerToTrapError(t *testing.T) {