Improved test coverage for new functions

Author: agilira

.github/workflows/ci.yml

@@ -35,6 +35,13 @@ jobs:
         go test -v -race -coverprofile=coverage.out -covermode=atomic ./...
         go tool cover -func=coverage.out
 
+    - name: Upload coverage to Codecov
+      uses: codecov/codecov-action@v4
+      with:
+        file: ./coverage.out
+        fail_ci_if_error: false
+        verbose: true
+      continue-on-error: true
 
 
   quality:

README.md

@@ -7,7 +7,7 @@ Built for Styx, it includes error codes, stack traces, user messages, and JSON s
 [![CI](https://github.com/agilira/go-errors/actions/workflows/ci.yml/badge.svg)](https://github.com/agilira/go-errors/actions/workflows/ci.yml)
 [![Security](https://img.shields.io/badge/Security-gosec-brightgreen)](https://github.com/agilira/go-errors/actions/workflows/ci.yml)
 [![Go Report Card](https://goreportcard.com/badge/github.com/agilira/go-errors)](https://goreportcard.com/report/github.com/agilira/go-errors)
-[![Coverage](https://img.shields.io/badge/coverage-86%25-brightgreen)](https://github.com/agilira/go-errors/actions/workflows/ci.yml)
+[![Coverage](https://codecov.io/gh/agilira/go-errors/branch/main/graph/badge.svg)](https://codecov.io/gh/agilira/go-errors)
 
 ## Features
 - Structured error type: code, message, context, cause, severity

coverage

@@ -0,0 +1,44 @@
+mode: set
+github.com/agilira/go-errors/errors.go:35.49,43.2 1 1
+github.com/agilira/go-errors/errors.go:47.72,57.2 1 1
+github.com/agilira/go-errors/errors.go:61.92,69.2 1 1
+github.com/agilira/go-errors/helpers.go:17.61,27.2 1 1
+github.com/agilira/go-errors/helpers.go:31.32,33.2 1 1
+github.com/agilira/go-errors/helpers.go:36.32,38.2 1 1
+github.com/agilira/go-errors/helpers.go:42.33,43.6 1 1
+github.com/agilira/go-errors/helpers.go:43.6,45.23 2 1
+github.com/agilira/go-errors/helpers.go:45.23,47.4 1 1
+github.com/agilira/go-errors/helpers.go:48.3,48.18 1 1
+github.com/agilira/go-errors/helpers.go:54.46,55.17 1 1
+github.com/agilira/go-errors/helpers.go:55.17,56.52 1 1
+github.com/agilira/go-errors/helpers.go:56.52,58.4 1 1
+github.com/agilira/go-errors/helpers.go:59.3,59.27 1 1
+github.com/agilira/go-errors/helpers.go:61.2,61.14 1 1
+github.com/agilira/go-errors/helpers.go:66.39,67.19 1 1
+github.com/agilira/go-errors/helpers.go:67.19,69.3 1 1
+github.com/agilira/go-errors/helpers.go:70.2,70.35 1 1
+github.com/agilira/go-errors/helpers.go:70.35,72.3 1 1
+github.com/agilira/go-errors/helpers.go:73.2,73.14 1 1
+github.com/agilira/go-errors/helpers.go:79.45,81.2 1 1
+github.com/agilira/go-errors/json.go:15.47,22.24 2 1
+github.com/agilira/go-errors/json.go:22.24,23.22 1 1
+github.com/agilira/go-errors/json.go:23.22,25.5 1 1
+github.com/agilira/go-errors/json.go:26.4,26.13 1 1
+github.com/agilira/go-errors/stacktrace.go:23.46,28.2 4 1
+github.com/agilira/go-errors/stacktrace.go:32.38,33.36 1 1
+github.com/agilira/go-errors/stacktrace.go:33.36,35.3 1 1
+github.com/agilira/go-errors/stacktrace.go:36.2,38.6 3 1
+github.com/agilira/go-errors/stacktrace.go:38.6,46.12 8 1
+github.com/agilira/go-errors/stacktrace.go:46.12,47.9 1 1
+github.com/agilira/go-errors/stacktrace.go:50.2,50.19 1 1
+github.com/agilira/go-errors/usermsg.go:11.52,14.2 2 1
+github.com/agilira/go-errors/usermsg.go:18.67,19.22 1 1
+github.com/agilira/go-errors/usermsg.go:19.22,21.3 1 1
+github.com/agilira/go-errors/usermsg.go:22.2,23.10 2 1
+github.com/agilira/go-errors/usermsg.go:28.38,31.2 2 1
+github.com/agilira/go-errors/usermsg.go:35.54,38.2 2 1
+github.com/agilira/go-errors/usermsg.go:42.38,43.21 1 1
+github.com/agilira/go-errors/usermsg.go:43.21,45.3 1 1
+github.com/agilira/go-errors/usermsg.go:46.2,46.18 1 1
+github.com/agilira/go-errors/usermsg.go:51.39,53.2 1 1
+github.com/agilira/go-errors/usermsg.go:57.36,59.2 1 1

errors_test.go

@@ -265,3 +265,87 @@ func TestUserMessageWithEmptyBoth(t *testing.T) {
 		t.Errorf("Expected empty string when both messages are empty, got: %s", result)
 	}
 }
+
+func TestWithContext(t *testing.T) {
+	err := New(TestCodeValidation, "Test error")
+
+	// Test adding context to error with nil context
+	err.Context = nil
+	err = err.WithContext("key1", "value1")
+	if err.Context == nil {
+		t.Error("Expected context to be initialized")
+	}
+	if err.Context["key1"] != "value1" {
+		t.Errorf("Expected context key1 to be 'value1', got '%v'", err.Context["key1"])
+	}
+
+	// Test adding more context
+	err = err.WithContext("key2", "value2")
+	if err.Context["key2"] != "value2" {
+		t.Errorf("Expected context key2 to be 'value2', got '%v'", err.Context["key2"])
+	}
+
+	// Test updating existing context
+	err = err.WithContext("key1", "updated_value")
+	if err.Context["key1"] != "updated_value" {
+		t.Errorf("Expected context key1 to be 'updated_value', got '%v'", err.Context["key1"])
+	}
+}
+
+func TestAsRetryable(t *testing.T) {
+	err := New(TestCodeValidation, "Test error")
+
+	// Test marking error as retryable
+	err = err.AsRetryable()
+	if !err.Retryable {
+		t.Error("Expected error to be marked as retryable")
+	}
+
+	// Test that it returns the same error instance for chaining
+	if err.AsRetryable() != err {
+		t.Error("Expected AsRetryable to return the same error instance")
+	}
+}
+
+func TestWithSeverity(t *testing.T) {
+	err := New(TestCodeValidation, "Test error")
+
+	// Test setting severity
+	err = err.WithSeverity("warning")
+	if err.Severity != "warning" {
+		t.Errorf("Expected severity to be 'warning', got '%s'", err.Severity)
+	}
+
+	// Test setting different severity
+	err = err.WithSeverity("critical")
+	if err.Severity != "critical" {
+		t.Errorf("Expected severity to be 'critical', got '%s'", err.Severity)
+	}
+
+	// Test that it returns the same error instance for chaining
+	if err.WithSeverity("info") != err {
+		t.Error("Expected WithSeverity to return the same error instance")
+	}
+}
+
+func TestMethodChaining(t *testing.T) {
+	err := New(TestCodeValidation, "Test error").
+		WithUserMessage("User message").
+		WithContext("key", "value").
+		WithSeverity("warning").
+		AsRetryable()
+
+	// Verify all methods were applied
+	if err.UserMsg != "User message" {
+		t.Error("Expected user message to be set")
+	}
+	if err.Context["key"] != "value" {
+		t.Error("Expected context to be set")
+	}
+	if err.Severity != "warning" {
+		t.Error("Expected severity to be set")
+	}
+	if !err.Retryable {
+		t.Error("Expected error to be retryable")
+	}
+}