chore: CI improvements (#88)

* Updated test pipeline

* Updated build pipeline to run on v2 branch

* Added buf, linter update & fixed proto gen

Author: xBlaz3kx

.github/workflows/build-image.yaml

@@ -11,6 +11,7 @@ on:
       - '*.md'
     branches:
       - main
+      - v2
   push:
     paths-ignore:
       - '.run/**'
@@ -20,6 +21,7 @@ on:
       - '*.md'
     branches:
       - main
+      - v2
 
 jobs:
   build:

.github/workflows/lint.yaml

@@ -4,6 +4,7 @@ on:
   push:
     branches:
       - main
+      - v2
     paths-ignore:
       - '.run/**'
       - 'docs/**'
@@ -13,6 +14,7 @@ on:
   pull_request:
     branches:
       - main
+      - v2
     types: [ opened, synchronize ]
     paths-ignore:
       - '.run/**'
@@ -22,18 +24,33 @@ on:
 
 jobs:
   lint:
-    name: "Run linter"
+    name: Lint Go code
     runs-on: ubuntu-latest
+
     steps:
-      - name: Checkout
+      - name: Checkout code
         uses: actions/checkout@v4
 
-      - name: Setup Go
-        uses: actions/setup-go@v5
+      - uses: actions/setup-go@v5
         with:
-          go-version: 1.21.1
+          go-version: 1.23.0
 
       - name: golangci-lint
         uses: golangci/golangci-lint-action@v6
         with:
-          version: latest
+          version: v1.62.0
+          args: --timeout=3m
+
+  buf:
+    name: Run buf
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      pull-requests: write
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - uses: bufbuild/buf-action@v1
+        with:
+          github_token: ${{ github.token }}

.github/workflows/test.yaml

@@ -4,6 +4,7 @@ on:
   push:
     branches:
       - main
+      - v2
     paths-ignore:
       - '.run/**'
       - 'docs/**'
@@ -14,6 +15,7 @@ on:
   pull_request:
     branches:
       - main
+      - v2
     types: [ opened, synchronize ]
     paths-ignore:
       - '.run/**'
@@ -23,21 +25,38 @@ on:
       - '*.md'
 
 jobs:
-  test:
+  # Run unit tests
+  unit:
     name: "Run unit tests"
     runs-on: ubuntu-latest
-
     steps:
-      - name: Checkout code
+      - name: Checkout
         uses: actions/checkout@v4
 
-      - name: Set up Go
+      - name: Setup Go
         uses: actions/setup-go@v5
         with:
-          go-version: "^1.21.1"
+          go-version: 1.23.0
+
+        # todo skip mocks coverage
+      - name: Install dependencies and run tests
+        run: |
+          go mod download
+          go test -v ./... -coverpkg=./... -short -coverprofile=unit_coverage.out
+
+  integration:
+    name: "Run integration tests"
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
 
-      - name: Install dependencies
-        run: go mod download
+      - name: Setup Go
+        uses: actions/setup-go@v5
+        with:
+          go-version: 1.23.0
 
-      - name: Test
-        run: go test --short -v ./...
+      - name: Install dependencies and run tests
+        run: |
+          go mod download
+          go test -v -run 'Integration$' ./... -coverpkg=./... -coverprofile=integration_coverage.out

buf.gen.yaml

@@ -0,0 +1,24 @@
+version: v2
+managed:
+  disable:
+    # Don't modify any file option or field option for googleapis
+    - module: buf.build/googleapis/googleapis
+  enabled: true
+  override:
+    - file_option: go_package_prefix
+      value: github.com/ChargePi/ChargePi-go/gen/proto
+
+inputs:
+  - directory: ./proto
+plugins:
+  #- remote: buf.build/protocolbuffers/go:v1.31.0
+  - local: protoc-gen-go
+    out: gen/proto
+    opt:
+      - paths=source_relative
+  #- remote: buf.build/grpc/go:v1.5.1
+  - local: protoc-gen-go-grpc
+    out: gen/proto
+    opt:
+      - paths=source_relative
+

buf.yaml

@@ -0,0 +1,19 @@
+version: v2
+modules:
+  - path: proto
+    lint:
+      use:
+        - STANDARD
+        - FILE_LOWER_SNAKE_CASE
+        - PACKAGE_NO_IMPORT_CYCLE
+        - PACKAGE_SAME_GO_PACKAGE
+        - FIELD_LOWER_SNAKE_CASE
+        - ENUM_VALUE_UPPER_SNAKE_CASE
+        - PACKAGE_LOWER_SNAKE_CASE
+      except:
+        - FIELD_NOT_REQUIRED
+      enum_zero_value_suffix: _UNSPECIFIED
+      rpc_allow_same_request_response: false
+      rpc_allow_google_protobuf_empty_requests: true
+      rpc_allow_google_protobuf_empty_responses: true
+      service_suffix: Service