ci: fix workflows, remove redundant steps

Author: alenon

.github/workflows/test.yml

@@ -1,4 +1,4 @@
-name: "Build and Test"
+name: "Run Tests"
 
 on:
   push:
@@ -10,46 +10,13 @@ on:
   workflow_dispatch:
 
 concurrency:
-  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }}
+  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
   cancel-in-progress: true
 
 jobs:
-  build:
-    name: Build ${{ matrix.os }}
-    strategy:
-      fail-fast: false
-      matrix:
-        os: [ ubuntu, windows, macos ]
-    runs-on: ${{ matrix.os }}-latest
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v5
-        with:
-          ref: ${{ github.event.pull_request.head.sha }}
-
-      - name: Setup Go with cache
-        uses: jfrog/.github/actions/install-go-with-cache@main
-
-      - name: Install dependencies
-        run: make deps
-
-      - name: Build binary
-        run: make build
-
-      - name: Upload build artifacts
-        uses: actions/upload-artifact@v4
-        with:
-          name: jfrog-evidence-${{ matrix.os }}
-          path: build/jfrog-evidence*
-          retention-days: 7
-
-  test:
-    name: Test ${{ matrix.os }}
-    strategy:
-      fail-fast: false
-      matrix:
-        os: [ ubuntu, windows, macos ]
-    runs-on: ${{ matrix.os }}-latest
+  unit-tests:
+    name: Unit Tests
+    runs-on: ubuntu-latest
     steps:
       - name: Checkout code
         uses: actions/checkout@v5
@@ -64,196 +31,3 @@ jobs:
 
       - name: Run unit tests
         run: make test-unit
-
-  test-coverage:
-    name: Test Coverage Report
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v5
-        with:
-          ref: ${{ github.event.pull_request.head.sha }}
-
-      - name: Setup Go with cache
-        uses: jfrog/.github/actions/install-go-with-cache@main
-
-      - name: Install dependencies
-        run: make deps
-
-      - name: Run tests with coverage
-        run: make test
-
-      - name: Generate coverage report
-        run: make coverage
-
-      - name: Upload coverage report
-        uses: actions/upload-artifact@v4
-        with:
-          name: coverage-report
-          path: |
-            coverage/coverage.out
-            coverage/coverage.html
-          retention-days: 30
-
-      - name: Display coverage summary
-        run: |
-          echo "### Test Coverage Summary" >> $GITHUB_STEP_SUMMARY
-          echo "" >> $GITHUB_STEP_SUMMARY
-          echo "```" >> $GITHUB_STEP_SUMMARY
-          go tool cover -func=coverage/coverage.out | tail -n 1 >> $GITHUB_STEP_SUMMARY
-          echo "```" >> $GITHUB_STEP_SUMMARY
-          echo "" >> $GITHUB_STEP_SUMMARY
-          echo "📊 Full coverage report available in artifacts" >> $GITHUB_STEP_SUMMARY
-
-      - name: Check coverage threshold
-        run: |
-          COVERAGE=$(go tool cover -func=coverage/coverage.out | grep total | awk '{print $3}' | sed 's/%//')
-          echo "Total coverage: ${COVERAGE}%"
-          THRESHOLD=70
-          if (( $(echo "$COVERAGE < $THRESHOLD" | bc -l) )); then
-            echo "⚠️ Coverage ${COVERAGE}% is below threshold ${THRESHOLD}%"
-            echo "::warning::Code coverage ${COVERAGE}% is below the recommended threshold of ${THRESHOLD}%"
-          else
-            echo "✅ Coverage ${COVERAGE}% meets threshold ${THRESHOLD}%"
-          fi
-
-  integration-test:
-    name: Integration Tests
-    runs-on: ubuntu-latest
-    if: github.event_name == 'push' && github.ref == 'refs/heads/main'
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v5
-
-      - name: Setup Go with cache
-        uses: jfrog/.github/actions/install-go-with-cache@main
-
-      - name: Install dependencies
-        run: make deps
-
-      - name: Run integration tests
-        run: make test-integration
-        continue-on-error: true
-        env:
-          INTEGRATION_TEST: true
-
-  benchmark:
-    name: Benchmarks
-    runs-on: ubuntu-latest
-    if: github.event_name == 'push' && github.ref == 'refs/heads/main'
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v5
-
-      - name: Setup Go with cache
-        uses: jfrog/.github/actions/install-go-with-cache@main
-
-      - name: Install dependencies
-        run: make deps
-
-      - name: Run benchmarks
-        run: make benchmark
-
-      - name: Upload benchmark results
-        uses: actions/upload-artifact@v4
-        with:
-          name: benchmark-results
-          path: benchmark-results.txt
-          retention-days: 30
-        if: always()
-
-  multi-platform-build:
-    name: Multi-Platform Build
-    runs-on: ubuntu-latest
-    if: github.event_name == 'push' && github.ref == 'refs/heads/main'
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v5
-
-      - name: Setup Go with cache
-        uses: jfrog/.github/actions/install-go-with-cache@main
-
-      - name: Install dependencies
-        run: make deps
-
-      - name: Build for all platforms
-        run: make build-all
-
-      - name: Upload multi-platform artifacts
-        uses: actions/upload-artifact@v4
-        with:
-          name: jfrog-evidence-all-platforms
-          path: build/
-          retention-days: 30
-
-  code-quality:
-    name: Code Quality Check
-    runs-on: ubuntu-latest
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v5
-        with:
-          ref: ${{ github.event.pull_request.head.sha }}
-
-      - name: Setup Go with cache
-        uses: jfrog/.github/actions/install-go-with-cache@main
-
-      - name: Install dependencies
-        run: make deps
-
-      - name: Install tools
-        run: make install-tools
-
-      - name: Format check
-        run: |
-          make fmt
-          if [ -n "$(git status --porcelain)" ]; then
-            echo "::error::Code is not properly formatted. Please run 'make fmt'"
-            git diff
-            exit 1
-          fi
-
-      - name: Run vet
-        run: make vet
-
-      - name: Run linter
-        run: make lint
-        continue-on-error: true
-
-  summary:
-    name: Test Summary
-    needs: [build, test, test-coverage, code-quality]
-    runs-on: ubuntu-latest
-    if: always()
-    steps:
-      - name: Summary
-        run: |
-          echo "## Workflow Summary" >> $GITHUB_STEP_SUMMARY
-          echo "" >> $GITHUB_STEP_SUMMARY
-          
-          if [[ "${{ needs.build.result }}" == "success" ]]; then
-            echo "✅ **Build:** Passed" >> $GITHUB_STEP_SUMMARY
-          else
-            echo "❌ **Build:** Failed" >> $GITHUB_STEP_SUMMARY
-          fi
-          
-          if [[ "${{ needs.test.result }}" == "success" ]]; then
-            echo "✅ **Tests:** Passed" >> $GITHUB_STEP_SUMMARY
-          else
-            echo "❌ **Tests:** Failed" >> $GITHUB_STEP_SUMMARY
-          fi
-          
-          if [[ "${{ needs.test-coverage.result }}" == "success" ]]; then
-            echo "✅ **Coverage:** Generated" >> $GITHUB_STEP_SUMMARY
-          else
-            echo "❌ **Coverage:** Failed" >> $GITHUB_STEP_SUMMARY
-          fi
-          
-          if [[ "${{ needs.code-quality.result }}" == "success" ]]; then
-            echo "✅ **Code Quality:** Passed" >> $GITHUB_STEP_SUMMARY
-          else
-            echo "⚠️ **Code Quality:** Issues found" >> $GITHUB_STEP_SUMMARY
-          fi
-          
-          echo "" >> $GITHUB_STEP_SUMMARY
-          echo "📦 Artifacts available for download in the workflow run" >> $GITHUB_STEP_SUMMARY

Makefile

@@ -105,47 +105,6 @@ coverage: test ## Generate test coverage report
 	$(GOCMD) tool cover -html=$(COVERAGE_DIR)/coverage.out -o $(COVERAGE_DIR)/coverage.html
 	@echo "$(GREEN)Coverage report generated: $(COVERAGE_DIR)/coverage.html$(NC)"
 
-coverage-view: coverage ## View coverage report in browser
-	@echo "$(GREEN)Opening coverage report...$(NC)"
-	open $(COVERAGE_DIR)/coverage.html 2>/dev/null || xdg-open $(COVERAGE_DIR)/coverage.html 2>/dev/null || echo "Please open $(COVERAGE_DIR)/coverage.html manually"
-
-benchmark: ## Run benchmarks
-	@echo "$(GREEN)Running benchmarks...$(NC)"
-	$(GOTEST) -bench=. -benchmem ./$(EVIDENCE_DIR)/...
-
-##@ Mocking
-
-# This target is generic and automatically finds all files with //go:generate mockgen annotations
-# No need to update this when adding new mocks - just add the annotation to your interface
-mocks: install-mockgen ## Generate all mocks
-	@echo "$(GREEN)Generating mocks...$(NC)"
-	@mkdir -p $(MOCK_DIR)
-	@echo "$(YELLOW)Finding and generating all mocks from go:generate annotations...$(NC)"
-	@export PROJECT_DIR=$(shell pwd) && \
-	find $(EVIDENCE_DIR) -name "*.go" -type f -exec grep -l "//go:generate.*mockgen" {} \; | while read file; do \
-		echo "$(YELLOW)Processing $$file...$(NC)"; \
-		cd $(shell pwd) && go generate $$file; \
-	done
-	@echo "$(GREEN)Mock generation complete$(NC)"
-
-mocks-specific: install-mockgen ## Generate mocks for specific interfaces (fallback for manual generation)
-	@echo "$(GREEN)Generating specific mocks manually...$(NC)"
-	@mkdir -p $(MOCK_DIR)
-	@# Find all interfaces in the codebase and generate mocks
-	@for file in $$(find $(EVIDENCE_DIR) -name "*.go" -type f); do \
-		if grep -q "^type.*interface" $$file 2>/dev/null; then \
-			interfaces=$$(grep "^type.*interface" $$file | awk '{print $$2}'); \
-			if [ ! -z "$$interfaces" ]; then \
-				echo "$(YELLOW)Generating mocks for $$file...$(NC)"; \
-				$(MOCKGEN) -source=$$file -destination=$(MOCK_DIR)/mock_$$(basename $$file) -package=mocks 2>/dev/null || true; \
-			fi; \
-		fi; \
-	done
-	@echo "$(GREEN)Specific mock generation complete$(NC)"
-
-clean-mocks: ## Remove generated mocks
-	@echo "$(YELLOW)Cleaning mocks...$(NC)"
-	rm -rf $(MOCK_DIR)
 
 ##@ Code Quality
 
@@ -222,16 +181,6 @@ clean-all: clean clean-mocks ## Clean everything including mocks and vendor
 	rm -rf vendor/
 	@echo "$(GREEN)Full clean complete$(NC)"
 
-##@ Docker
-
-docker-build: ## Build Docker image
-	@echo "$(GREEN)Building Docker image...$(NC)"
-	docker build -t $(BINARY_NAME):$(VERSION) -t $(BINARY_NAME):latest .
-	@echo "$(GREEN)Docker build complete$(NC)"
-
-docker-run: ## Run Docker container
-	@echo "$(GREEN)Running Docker container...$(NC)"
-	docker run --rm -it $(BINARY_NAME):latest
 
 ##@ Git Hooks
 
@@ -266,16 +215,5 @@ fixme: ## Show FIXME items in code
 	@echo "$(YELLOW)FIXME items in code:$(NC)"
 	@grep -r "FIXME" $(EVIDENCE_DIR) --include="*.go" || echo "No FIXME items found"
 
-stats: ## Show code statistics
-	@echo "$(GREEN)Code Statistics:$(NC)"
-	@echo "Lines of Go code:"
-	@find $(EVIDENCE_DIR) -name "*.go" -not -path "*/vendor/*" -not -path "*/mocks/*" | xargs wc -l | tail -1
-	@echo ""
-	@echo "Number of Go files:"
-	@find $(EVIDENCE_DIR) -name "*.go" -not -path "*/vendor/*" -not -path "*/mocks/*" | wc -l
-	@echo ""
-	@echo "Number of test files:"
-	@find $(EVIDENCE_DIR) -name "*_test.go" -not -path "*/vendor/*" | wc -l
-
 # Default target
 .DEFAULT_GOAL := help

evidence/cli/docs/create/help_test.go

@@ -0,0 +1,19 @@
+package create
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestGetDescription(t *testing.T) {
+	description := GetDescription()
+	assert.NotEmpty(t, description)
+	assert.Contains(t, description, "evidence")
+}
+
+func TestGetArguments(t *testing.T) {
+	args := GetArguments()
+	assert.NotNil(t, args)
+	assert.Empty(t, args) // Currently returns empty slice
+}

evidence/cli/docs/get/help_test.go

@@ -0,0 +1,19 @@
+package get
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestGetDescription(t *testing.T) {
+	description := GetDescription()
+	assert.NotEmpty(t, description)
+	assert.Contains(t, description, "evidence")
+}
+
+func TestGetArguments(t *testing.T) {
+	args := GetArguments()
+	assert.NotNil(t, args)
+	assert.Empty(t, args) // Currently returns empty slice
+}

evidence/cli/docs/verify/help_test.go

@@ -0,0 +1,19 @@
+package verify
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestGetDescription(t *testing.T) {
+	description := GetDescription()
+	assert.NotEmpty(t, description)
+	assert.Contains(t, description, "evidence")
+}
+
+func TestGetArguments(t *testing.T) {
+	args := GetArguments()
+	assert.NotNil(t, args)
+	assert.Empty(t, args) // Currently returns empty slice
+}