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

[aoshimash]

Overview

Enable all Playwright tests in CI environment and implement comprehensive error handling and debugging capabilities.

Background

After Phase 1 (#62) validates basic Playwright functionality and Phase 2 (#63) optimizes performance with caching, we need to enable all Playwright tests and ensure robust error handling.

Implementation Plan

Phase 3 (This Issue)

1. Remove all CI skip conditions from Playwright tests:
   • TestBrowserPool_RenderPage (already enabled in Phase 1)
   • TestJSClient_RenderPage
   • TestJSClient_Get
2. Implement enhanced error handling:
   • Detailed error logs for Playwright failures
   • Browser console logs on test failure
   • Network request/response logs for debugging
3. (Optional) Add screenshot capture on test failure
4. Ensure all tests pass consistently in CI

Technical Details

• Remove all if os.Getenv("CI") == "true" conditions from Playwright tests
• Add error context with browser logs:

  if err \!= nil {
      logs, _ := page.Context().Browser().NewBrowserCDPSession()
      // Capture and log browser state
  }

• Consider implementing retry logic for flaky tests

Success Criteria

• All Playwright tests run and pass in CI environment
• No test flakiness introduced
• Clear error messages when tests fail
• Debugging information readily available in CI logs
• CI remains stable over multiple runs

Dependencies

• Requires Phase 1 (feat: Enable Playwright tests in CI environment - Phase 1 (Basic Setup) #62) and Phase 2 (feat: Implement Playwright browser binary caching in CI - Phase 2 #63) to be completed first

Related Issues

• Phase 1: feat: Enable Playwright tests in CI environment - Phase 1 (Basic Setup) #62 (Basic Playwright setup)
• Phase 2: feat: Implement Playwright browser binary caching in CI - Phase 2 #63 (Browser binary caching)

Future Considerations

• Multi-browser testing (Firefox, WebKit)
• Parallel test execution optimization
• Visual regression testing capabilities

[aoshimash]

✅ Phase 3 完了！

PR #67がマージされ、すべてのPlaywrightテストがCI環境で正常に実行されるようになりました。

実装内容

1. すべてのCIスキップ条件を削除 ✅
2. エラーハンドリングの強化 ✅
   • ブラウザログとネットワークログの自動収集
   • テスト失敗時のデバッグ情報出力
   • スクリーンショット撮影機能
3. テストの安定性向上 ✅
   • タイムアウトを60秒に延長（CI環境での安定性向上）

結果

• すべてのPlaywrightテストがCI環境で成功
• Go 1.23とGo 1.24の両方で動作確認済み
• エラー発生時の詳細なデバッグ情報が利用可能

関連PR

• Phase 1: feat: Enable Playwright tests in CI environment - Phase 1 (Basic Setup) #62 ✅
• Phase 2: fix: Install missing system dependencies for Playwright in CI - Phase 2 Fix #68 (PR fix: Use Playwright Docker image for CI to resolve dependency issues #69) ✅
• Phase 3: feat: Enable all Playwright tests and improve error handling - Phase 3 #64 (PR feat: Enable all Playwright tests and improve error handling - Phase 3 #67) ✅

Playwright統合の全フェーズが完了しました！🎉