Merge pull request #63 from deploymenttheory/dev

Dev

Author: ShocOne

httpclient/httpclient_rate_handler_test.go

@@ -22,7 +22,8 @@ func TestCalculateBackoff(t *testing.T) {
 	}
 
 	for _, tt := range tests {
-		t.Run("RetryCount"+string(tt.retry), func(t *testing.T) {
+		t.Run("RetryCount"+strconv.Itoa(tt.retry), func(t *testing.T) {
+
 			delay := calculateBackoff(tt.retry)
 
 			// The delay should never exceed maxDelay

logger/zaplogger_logger_test.go

@@ -117,7 +117,7 @@ func TestDefaultLogger_Error(t *testing.T) {
 	dLogger := &defaultLogger{logger: mockLogger.Logger, logLevel: LogLevelError}
 
 	expectedErrorMsg := "error message"
-	mockLogger.On("Error", expectedErrorMsg, mock.Anything).Once()
+	mockLogger.On("Error", expectedErrorMsg, mock.Anything).Once().Return(nil) // Ensure Error method is called exactly once and returns nil
 
 	err := dLogger.Error(expectedErrorMsg)