Flaky CI test

[mxpv]

The following unit tests fails on CI once in a while:

=== RUN   TestExecuteHook_CurlWebhook
    hooks_test.go:108: 
        	Error Trace:	/home/runner/work/podsync/podsync/pkg/feed/hooks_test.go:108
        	Error:      	Received unexpected error:
        	            	hook execution failed: signal: killed, output: {
        	            	  "args": {}, 
        	            	  "data": "", 
        	            	  "files": {}, 
        	            	  "form": {
        	            	    "Test Episode for Webhook": ""
        	            	  }, 
        	            	  "headers": {
        	            	    "Accept": "*/*", 
        	            	    "Content-Length": "24", 
        	            	    "Content-Type": "application/x-www-form-urlencoded", 
        	            	    "Host": "httpbin.org", 
        	            	    "User-Agent": "curl/8.5.0", 
        	            	    "X-Amzn-Trace-Id": "Root=1-68b0b848-31abeffa05e132b03d227118"
        	            	  }, 
        	            	  "json": null, 
        	            	  "origin": "172.182.226.21", 
        	            	  "url": "http://httpbin.org/post"
        	            	}
        	Test:       	TestExecuteHook_CurlWebhook
        	Messages:   	Curl webhook should execute successfully

[mxpv]

@claude can you make a fix?

[claude]

Claude finished @mxpv's task —— View job • claude/issue-768-20250828-2100 • Create PR ➔

---

Todo List ✅

• Analyze the failing test code to understand what it does
• Examine the hook execution implementation
• Identify the root cause of the "signal: killed" error
• Implement a fix for the flaky test
• Run tests to verify the fix works
• Run linting to ensure code quality
• Commit and push the changes
• Provide PR creation link

Summary

Fixed the flaky TestExecuteHook_CurlWebhook test that was failing intermittently in CI with "signal: killed" errors. The issue was caused by the test's dependency on the external httpbin.org service, which could be unreliable or slow in CI environments.

Solution Implemented

• Replaced external dependency with a local httptest.NewServer
• Added proper request verification to ensure webhook functionality is tested correctly
• Maintained full test coverage while eliminating network-related flakiness
• Improved test reliability by making it deterministic and independent of external services

The test now:

1. Creates a local HTTP server to receive webhook requests
2. Captures and verifies the request data and headers
3. Runs ~100x faster (from ~1.45s to ~0.01s)
4. Will never fail due to network connectivity issues

All tests pass and the code follows project formatting standards.