Merge pull request #67 from deploymenttheory/dev

Refactor rate limit header parsing and add test cases***

Author: ShocOne

httpclient/httpclient_rate_handler_test.go

@@ -37,7 +37,6 @@ func TestCalculateBackoff(t *testing.T) {
 	}
 }
 
-// TestParseRateLimitHeaders tests parsing of rate limit headers and calculation of wait duration
 func TestParseRateLimitHeaders(t *testing.T) {
 	tests := []struct {
 		name         string
@@ -54,7 +53,7 @@ func TestParseRateLimitHeaders(t *testing.T) {
 		{
 			name: "RetryAfterHTTPDate",
 			headers: map[string]string{
-				"Retry-After": http.TimeFormat, // Use current time for simplicity
+				"Retry-After": time.Now().UTC().Format(time.RFC1123), // Use current time in RFC1123 format
 			},
 			expectedWait: 0, // Immediate retry since the date is current
 		},

logger/zaplogger_logger_test.go

@@ -105,7 +105,7 @@ func TestDefaultLogger_Warn(t *testing.T) {
 	dLogger := &defaultLogger{logger: mockLogger.Logger, logLevel: LogLevelWarn}
 
 	expectedMessage := "warn message"
-	mockLogger.On("Warn", expectedMessage, mock.Anything).Once()
+	mockLogger.On("Warn", expectedMessage, []zapcore.Field(nil)).Once()
 
 	dLogger.Warn(expectedMessage)
 
@@ -136,9 +136,11 @@ func TestDefaultLogger_Panic(t *testing.T) {
 	mockLogger := NewMockLogger()
 	dLogger := &defaultLogger{logger: mockLogger.Logger, logLevel: LogLevelPanic}
 
-	mockLogger.On("Panic", "panic message", mock.Anything).Once()
+	expectedMessage := "panic message"
+	mockLogger.On("Panic", expectedMessage, mock.Anything).Once()
 
-	assert.Panics(t, func() { dLogger.Panic("panic message") }, "The Panic method should trigger a panic")
+	// Assert that calling Panic method triggers a panic
+	assert.Panics(t, func() { dLogger.Panic(expectedMessage) }, "The Panic method should trigger a panic")
 
 	mockLogger.AssertExpectations(t)
 }