feat(database): add integration testing (#896)

.github/workflows/integration-test.yml

@@ -0,0 +1,49 @@
+# name of the action
+name: integration-test
+
+# trigger on pull_request events that modify this file or any database files
+on:
+  pull_request:
+    paths:
+      - '.github/workflows/integration-test.yml'
+      - 'database/**'
+
+# pipeline to execute
+jobs:
+  database:
+    runs-on: ubuntu-latest
+
+    services:
+      postgres:
+        image: postgres:15-alpine
+        env:
+          POSTGRES_DB: vela
+          POSTGRES_PASSWORD: notARealPassword12345
+          POSTGRES_USER: vela
+        options: >-
+          --health-cmd pg_isready
+          --health-interval 10s
+          --health-timeout 5s
+          --health-retries 5
+        ports:
+          - 5432:5432
+
+    env:
+      POSTGRES_ADDR: postgres://vela:notARealPassword12345@localhost:5432/vela
+      SQLITE_ADDR: vela.db
+
+    steps:
+      - name: clone
+        uses: actions/checkout@v3
+
+      - name: install go
+        uses: actions/setup-go@v4
+        with:
+          # use version from go.mod file
+          go-version-file: 'go.mod'
+          cache: true
+          check-latest: true
+
+      - name: test
+        run: |
+          make integration-test

.github/workflows/test.yml

@@ -25,10 +25,10 @@ jobs:
 
     - name: test
       run: |
-        go test -race -covermode=atomic -coverprofile=coverage.out ./...
+        make test
 
     - name: coverage
       uses: codecov/codecov-action@v3
       with:
         token: ${{ secrets.CODECOV_TOKEN }}
-        file: coverage.out
+        file: coverage.out

Makefile

@@ -91,6 +91,13 @@ fix:
 	@echo "### Fixing Go Code"
 	@go fix ./...
 
+# The `integration-test` target is intended to run all integration tests for the Go source code.
+.PHONY: integration-test
+integration-test:
+	@echo
+	@echo "### Integration Testing"
+	INTEGRATION=1 go test -run TestDatabase_Integration ./...
+
 # The `test` target is intended to run
 # the tests for the Go source code.
 #
@@ -99,18 +106,15 @@ fix:
 test:
 	@echo
 	@echo "### Testing Go Code"
-	@go test -race ./...
+	@go test -race -covermode=atomic -coverprofile=coverage.out ./...
 
 # The `test-cover` target is intended to run
 # the tests for the Go source code and then
 # open the test coverage report.
 #
 # Usage: `make test-cover`
 .PHONY: test-cover
-test-cover:
-	@echo
-	@echo "### Creating test coverage report"
-	@go test -race -covermode=atomic -coverprofile=coverage.out ./...
+test-cover: test
 	@echo
 	@echo "### Opening test coverage report"
 	@go tool cover -html=coverage.out

database/hook/table.go

@@ -51,7 +51,7 @@ hooks (
 	status       TEXT,
 	link         TEXT,
 	webhook_id   INTEGER,
-	UNIQUE(repo_id, build_id)
+	UNIQUE(repo_id, number)
 );
 `
 )