Fix flaky CI test by replacing external dependency with local test server

The TestExecuteHook_CurlWebhook test was failing intermittently in CI
with 'signal: killed' errors due to external dependency on httpbin.org.
This fix replaces the external HTTP request with a local httptest.Server
to eliminate network-related flakiness and make the test deterministic.

Changes:
- Replace httpbin.org with httptest.NewServer for reliable testing
- Add request verification to ensure the webhook functionality works
- Maintain the same test coverage while eliminating external dependency

Co-authored-by: Maksym Pavlenko <[email protected]>

Author: claude[bot]

pkg/feed/hooks_test.go

@@ -1,6 +1,10 @@
 package feed
 
 import (
+	"fmt"
+	"io"
+	"net/http"
+	"net/http/httptest"
 	"os"
 	"path/filepath"
 	"testing"
@@ -93,8 +97,28 @@ func TestExecuteHook_CornerCases(t *testing.T) {
 }
 
 func TestExecuteHook_CurlWebhook(t *testing.T) {
+	// Create a local test server to avoid external dependencies
+	receivedData := ""
+	receivedHeaders := make(map[string]string)
+
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		// Capture the request data for verification
+		body, err := io.ReadAll(r.Body)
+		if err == nil {
+			receivedData = string(body)
+		}
+		receivedHeaders["Content-Type"] = r.Header.Get("Content-Type")
+		receivedHeaders["User-Agent"] = r.Header.Get("User-Agent")
+
+		// Return a simple response
+		w.WriteHeader(http.StatusOK)
+		fmt.Fprintln(w, `{"status": "ok"}`)
+	}))
+	defer server.Close()
+
+	// Use the local test server URL instead of external httpbin.org
 	hook := &ExecHook{
-		Command: []string{"curl -s -X POST -d \"$EPISODE_TITLE\" httpbin.org/post"},
+		Command: []string{fmt.Sprintf("curl -s -X POST -d \"$EPISODE_TITLE\" %s", server.URL)},
 		Timeout: 10,
 	}
 
@@ -106,4 +130,8 @@ func TestExecuteHook_CurlWebhook(t *testing.T) {
 
 	err := hook.Invoke(env)
 	assert.NoError(t, err, "Curl webhook should execute successfully")
+
+	// Verify that the request was actually made and data was received
+	assert.Equal(t, "Test Episode for Webhook", receivedData, "Server should receive the episode title")
+	assert.Contains(t, receivedHeaders["User-Agent"], "curl", "Request should be made by curl")
 }