feat: Enable all Playwright tests and improve error handling - Phase 3

[aoshimash]

Summary

• Enable all Playwright tests in CI environment (Reverted due to missing system dependencies)
• Implement comprehensive error handling and debugging capabilities
• Add screenshot capture on test failure (optional feature)

Background

This PR was intended to complete Phase 3 of the Playwright CI integration (#64), following Phase 1 (#62) which validated basic functionality and Phase 2 (#63) which was supposed to optimize performance with caching.

However, we discovered that the CI environment is missing critical system dependencies for Playwright to run properly. This should have been addressed in Phase 2.

Current Status

Due to missing system dependencies in CI (GTK4, GStreamer, multimedia codecs, etc.), we've had to revert the CI enablement while keeping the enhanced error handling features.

Missing Dependencies in CI:

• libgtk-4.so.1
• libgraphene-1.0.so.0
• libwoff2dec.so.1.0.2
• libvpx.so.9
• libevent-2.1.so.7
• libopus.so.0
• Various GStreamer libraries
• And many more...

Changes Made

1. Removed CI Skip Conditions (Reverted)

• Tests still skip in CI due to missing dependencies
• Tests run successfully in local environments

2. Enhanced Error Handling ✅

• Created test_helpers.go with debug capture utilities:
  • SetupPageDebugHandlers() - Captures console logs and network activity
  • CaptureScreenshotOnFailure() - Takes screenshots when tests fail
  • LogTestDebugInfo() - Logs comprehensive debug information
  • RetryTest() - Provides retry logic for potentially flaky tests

3. Browser Pool Integration ✅

• Modified browser_pool.go to automatically capture debug logs when running in test mode
• Enhanced error messages with console logs and network activity details
• Improved debugging output when tests fail

Test Results

All Playwright tests pass locally where dependencies are available:

=== RUN   TestBrowserPool_RenderPage
--- PASS: TestBrowserPool_RenderPage (2.51s)
=== RUN   TestJSClient_RenderPage
--- PASS: TestJSClient_RenderPage (2.15s)
=== RUN   TestJSClient_Get
--- PASS: TestJSClient_Get (2.16s)

Next Steps

1. Phase 2 (feat: Implement Playwright browser binary caching in CI - Phase 2 #63) needs to be properly implemented to install all required system dependencies
2. Once dependencies are installed, we can remove the CI skip conditions
3. Consider using the official Playwright Docker image or GitHub Action for better dependency management

Success Criteria (Partially Met)

• All Playwright tests run and pass in CI environment (blocked by dependencies)
• Clear error messages when tests fail
• Debugging information readily available in logs
• Screenshot capture capability on test failure

Related Issues

• Partially addresses feat: Enable all Playwright tests and improve error handling - Phase 3 #64
• Blocked by incomplete feat: Implement Playwright browser binary caching in CI - Phase 2 #63 (Phase 2)
• Depends on feat: Enable Playwright tests in CI environment - Phase 1 (Basic Setup) #62 (Phase 1)

Test Plan

1. Run all tests locally with go test ./... ✅
2. Verify Playwright tests run without CI skip conditions (blocked)
3. Check that error handling provides useful debug information ✅
4. Confirm CI pipeline remains stable (tests skip in CI)

🤖 Generated with Claude Code

[aoshimash]

現在の状況

このPRは当初の目的（Phase 3: すべてのPlaywrightテストをCI環境で有効化）を完全には達成できませんでした。

実装できたこと ✅

• エラーハンドリングの強化（ブラウザログ、ネットワークログの収集）
• テスト失敗時のスクリーンショット撮影機能
• デバッグ情報の自動収集機能

実装できなかったこと ❌

• CI環境でのPlaywrightテストの有効化（システム依存関係の不足により）

次のステップ

1. issue fix: Install missing system dependencies for Playwright in CI - Phase 2 Fix #68 でPhase 2の実装不備を修正
2. CI環境に必要なシステム依存関係をインストール
3. その後、このPRに戻ってCI環境でのテスト有効化を完了

現時点では、このPRはマージせずに保留とし、#68の解決後に再度作業を進めます。