refactor(tests): replace hardcoded address with constant

Author: mrz1836

cache_test.go

@@ -220,7 +220,7 @@ func TestSavePeerCacheInvalidPath(_ *testing.T) {
 		{
 			ID:       "QmYyQSo1c1Ym7orWxLYvCrM2EmxFTANf8wXmmE7DWjhx5N",
 			Name:     "peer1",
-			Addrs:    []string{"/ip4/127.0.0.1/tcp/4001"},
+			Addrs:    []string{testAddrLocalhost},
 			LastSeen: time.Now(),
 		},
 	}
@@ -478,7 +478,7 @@ func TestSavePeerCacheFilePermissions(t *testing.T) {
 		{
 			ID:       "QmYyQSo1c1Ym7orWxLYvCrM2EmxFTANf8wXmmE7DWjhx5N",
 			Name:     "peer1",
-			Addrs:    []string{"/ip4/127.0.0.1/tcp/4001"},
+			Addrs:    []string{testAddrLocalhost},
 			LastSeen: time.Now(),
 		},
 	}
@@ -503,7 +503,7 @@ func TestLoadAndSavePeerCacheRoundTrip(t *testing.T) {
 		{
 			ID:       "QmYyQSo1c1Ym7orWxLYvCrM2EmxFTANf8wXmmE7DWjhx5N",
 			Name:     "alice",
-			Addrs:    []string{"/ip4/127.0.0.1/tcp/4001"},
+			Addrs:    []string{testAddrLocalhost},
 			LastSeen: now,
 		},
 		{

config_test.go

@@ -9,6 +9,13 @@ import (
 	"github.com/stretchr/testify/assert"
 )
 
+const (
+	logPrefixDebug = "[DEBUG]"
+	logPrefixInfo  = "[INFO]"
+	logPrefixWarn  = "[WARN]"
+	logPrefixError = "[ERROR]"
+)
+
 func TestDefaultLoggerDebugf(t *testing.T) {
 	tests := []struct {
 		name           string
@@ -21,21 +28,21 @@ func TestDefaultLoggerDebugf(t *testing.T) {
 			name:           "simple debug message",
 			format:         "test message",
 			args:           nil,
-			expectedPrefix: "[DEBUG]",
+			expectedPrefix: logPrefixDebug,
 			expectedMsg:    "test message",
 		},
 		{
 			name:           "debug message with formatting",
 			format:         "user %s logged in with ID %d",
 			args:           []any{"alice", 42},
-			expectedPrefix: "[DEBUG]",
+			expectedPrefix: logPrefixDebug,
 			expectedMsg:    "user alice logged in with ID 42",
 		},
 		{
 			name:           "debug message with multiple args",
 			format:         "processing %d items for %s in %v seconds",
 			args:           []any{100, "test", 3.14},
-			expectedPrefix: "[DEBUG]",
+			expectedPrefix: logPrefixDebug,
 			expectedMsg:    "processing 100 items for test in 3.14 seconds",
 		},
 	}
@@ -70,21 +77,21 @@ func TestDefaultLoggerInfof(t *testing.T) {
 			name:           "simple info message",
 			format:         "server started",
 			args:           nil,
-			expectedPrefix: "[INFO]",
+			expectedPrefix: logPrefixInfo,
 			expectedMsg:    "server started",
 		},
 		{
 			name:           "info message with formatting",
 			format:         "listening on port %d",
 			args:           []any{8080},
-			expectedPrefix: "[INFO]",
+			expectedPrefix: logPrefixInfo,
 			expectedMsg:    "listening on port 8080",
 		},
 		{
 			name:           "info message with string formatting",
 			format:         "connected to %s at %s",
 			args:           []any{"database", "localhost:5432"},
-			expectedPrefix: "[INFO]",
+			expectedPrefix: logPrefixInfo,
 			expectedMsg:    "connected to database at localhost:5432",
 		},
 	}
@@ -118,21 +125,21 @@ func TestDefaultLoggerWarnf(t *testing.T) {
 			name:           "simple warning message",
 			format:         "deprecated API used",
 			args:           nil,
-			expectedPrefix: "[WARN]",
+			expectedPrefix: logPrefixWarn,
 			expectedMsg:    "deprecated API used",
 		},
 		{
 			name:           "warning with error details",
 			format:         "retry attempt %d failed: %s",
 			args:           []any{3, "connection timeout"},
-			expectedPrefix: "[WARN]",
+			expectedPrefix: logPrefixWarn,
 			expectedMsg:    "retry attempt 3 failed: connection timeout",
 		},
 		{
 			name:           "warning with multiple values",
 			format:         "cache miss for key %s, loading from %s",
 			args:           []any{"user:123", "database"},
-			expectedPrefix: "[WARN]",
+			expectedPrefix: logPrefixWarn,
 			expectedMsg:    "cache miss for key user:123, loading from database",
 		},
 	}
@@ -166,21 +173,21 @@ func TestDefaultLoggerErrorf(t *testing.T) {
 			name:           "simple error message",
 			format:         "connection failed",
 			args:           nil,
-			expectedPrefix: "[ERROR]",
+			expectedPrefix: logPrefixError,
 			expectedMsg:    "connection failed",
 		},
 		{
 			name:           "error with details",
 			format:         "failed to process request: %s",
 			args:           []any{"invalid input"},
-			expectedPrefix: "[ERROR]",
+			expectedPrefix: logPrefixError,
 			expectedMsg:    "failed to process request: invalid input",
 		},
 		{
 			name:           "error with multiple fields",
 			format:         "database error on table %s: %v (code: %d)",
 			args:           []any{"users", "constraint violation", 1062},
-			expectedPrefix: "[ERROR]",
+			expectedPrefix: logPrefixError,
 			expectedMsg:    "database error on table users: constraint violation (code: 1062)",
 		},
 	}
@@ -223,10 +230,10 @@ func TestDefaultLoggerAllLevels(t *testing.T) {
 	assert.Len(t, lines, 4)
 
 	// Verify each level appears once
-	assert.Contains(t, output, "[DEBUG]")
-	assert.Contains(t, output, "[INFO]")
-	assert.Contains(t, output, "[WARN]")
-	assert.Contains(t, output, "[ERROR]")
+	assert.Contains(t, output, logPrefixDebug)
+	assert.Contains(t, output, logPrefixInfo)
+	assert.Contains(t, output, logPrefixWarn)
+	assert.Contains(t, output, logPrefixError)
 
 	// Verify messages are correct
 	assert.Contains(t, output, "debug message")