Merge remote-tracking branch 'origin/dev'

Author: boris-ilijic

.github/workflows/ci.yml

@@ -38,7 +38,7 @@ jobs:
       PBM_BRANCH: ${{ github.event.inputs.pbm_branch || 'main' }}
       GO_VER: ${{ github.event.inputs.go_ver || '1.22-bullseye' }}
       PR_NUMBER: ${{ github.event.number|| github.event.inputs.pr_ver }}
-
+      MAKE_TARGET: 'build-cover'
     steps:
       - name: Checkout testing repo
         uses: actions/checkout@v4
@@ -58,8 +58,50 @@ jobs:
           docker compose run test pytest -s --junitxml=junit.xml -k ${{ matrix.test }}
         working-directory: psmdb-testing/pbm-functional/pytest
 
+      - name: Fetch coverage files
+        run: |
+          docker compose run --rm golang_reports cp -r /gocoverdir/reports /test
+          sudo chmod -R 777 reports
+        working-directory: psmdb-testing/pbm-functional/pytest
+        if: success() || failure()
+
+      - name: Upload coverage reports
+        uses: actions/upload-artifact@v4
+        with:
+          name: reports-${{ matrix.test }}-${{ matrix.psmdb }}
+          path: psmdb-testing/pbm-functional/pytest/reports/
+        if: success() || failure()
+
       - name: Publish Test Report
         uses: mikepenz/action-junit-report@v4
         if: success() || failure()
         with:
           report_paths: "**/junit.xml"
+
+  coverage:
+    if: ${{ always() }}
+    needs: test
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: Set up Go
+        uses: actions/setup-go@v5
+        with:
+          go-version-file: 'go.mod'
+          check-latest: true
+      - name: Download all coverage reports
+        uses: actions/download-artifact@v4
+        with:
+          path: reports
+          pattern: reports-*
+          merge-multiple: true
+      - name: Merge coverage reports
+        run: |
+          go tool covdata textfmt -i=./reports -o ./coverage.txt
+      - name: Upload coverage to Codecov
+        uses: codecov/codecov-action@v3
+        with:
+          files: ./coverage.txt
+          flags: integration
+          fail_ci_if_error: false
+          token: ${{ secrets.CODECOV_TOKEN }}

.github/workflows/codecov.yml

@@ -21,5 +21,6 @@ jobs:
         uses: codecov/codecov-action@v4
         with:
           file: cover.out
+          flags: unittests
           fail_ci_if_error: false
           token: ${{ secrets.CODECOV_TOKEN }}

.github/workflows/reviewdog.yml

@@ -16,6 +16,7 @@ jobs:
         uses: codecov/codecov-action@v4
         with:
           file: cover.out
+          flags: unittests
           fail_ci_if_error: false
           token: ${{ secrets.CODECOV_TOKEN }}
 

cmd/pbm-agent/commands_test.go

@@ -0,0 +1,87 @@
+package main
+
+import (
+	"bytes"
+	"os"
+	"strings"
+	"testing"
+
+	"github.com/spf13/cobra"
+
+	"github.com/percona/percona-backup-mongodb/pbm/version"
+)
+
+func TestRootCmd_NoArgs(t *testing.T) {
+	rootCmd, _ := setupTestCmd()
+	err := rootCmd.Execute()
+	if err == nil || !strings.Contains(err.Error(), "required flag mongodb-uri") {
+		t.Fatal(err)
+	}
+}
+
+func TestRootCmd_Config(t *testing.T) {
+	tmpConfig, cleanup := createTempConfigFile(`
+log:
+  path: "/dev/stderr"
+  level: "D"
+  json: false
+`)
+	defer cleanup()
+
+	rootCmd, _ := setupTestCmd("--config", tmpConfig)
+	err := rootCmd.Execute()
+	if err == nil || !strings.Contains(err.Error(), "required flag mongodb-uri") {
+		t.Fatal(err)
+	}
+}
+
+func createTempConfigFile(content string) (string, func()) {
+	temp, _ := os.CreateTemp("", "test-config-*.yaml")
+
+	_, _ = temp.WriteString(content)
+	_ = temp.Close()
+
+	return temp.Name(), func() {
+		_ = os.Remove(temp.Name())
+	}
+}
+
+func TestVersionCommand_Default(t *testing.T) {
+	rootCmd, buf := setupTestCmd("version")
+	err := rootCmd.Execute()
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	output := buf.String()
+
+	if !strings.Contains(output, "Version:") {
+		t.Errorf("expected full version info in output, got: %s", output)
+	}
+}
+
+func TestVersionCommand_Short(t *testing.T) {
+	rootCmd, buf := setupTestCmd("version", "--short")
+	err := rootCmd.Execute()
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	output := buf.String()
+
+	if !strings.Contains(output, version.Current().Short()) {
+		t.Errorf("expected short version info in output, got: %s", output)
+	}
+}
+
+func setupTestCmd(args ...string) (*cobra.Command, *bytes.Buffer) {
+	cmd := rootCommand()
+	cmd.AddCommand(versionCommand())
+
+	var buf bytes.Buffer
+	cmd.SetOut(&buf)
+	cmd.SetErr(&buf)
+	cmd.SetArgs(args)
+
+	return cmd, &buf
+}