sweep: can you write tests for internal/api/api.go

[harperreed]

---

Add unit tests for HelpHandler in internal/api/api.go

Description:

Create unit tests for the HelpHandler function in internal/api/api.go to ensure proper functionality of the help page rendering.

Tasks:

• Create a new file internal/api/api_test.go for the unit tests
• Implement test cases for the HelpHandler function

Test:

• Test successful rendering of help page
  • Verify correct HTTP status code (200 OK)
  • Check Content-Type header is set to "text/html"
  • Validate presence of expected HTML structure and content
• Test error handling when README.md is not found
  • Verify correct HTTP status code (500 Internal Server Error)
• Test error handling when template parsing fails
  • Mock template.Parse to return an error
  • Verify correct HTTP status code (500 Internal Server Error)

Additional Tasks:

• Update internal/api/api.go to improve testability
  • Extract file reading and template parsing into separate functions
  • Use dependency injection for os.ReadFile and template.Parse to allow mocking in tests

[sweep-ai[bot]]

❌ Error

Traceback (most recent call last):
  File "/app/sweepai/handlers/on_ticket.py", line 167, in on_ticket
    await fix_ci_failures(pull_request, tracking_id)
  File "/app/sweepai/handlers/fix_ci.py", line 266, in fix_ci_failures
    new_pull_request_number = results["pull_request"].get("number")
AttributeError: 'NoneType' object has no attribute 'get'

'NoneType' object has no attribute 'get'

Sweep has encountered a runtime error unrelated to your request. Please let us know via this link or at support@sweep.dev directly.

:book: For more information on how to use Sweep, please read our documentation.

^{Tracking ID: d115851dd9}