fix: E2Eテストが失敗する問題を修正

[bougan1160]

Fixes #464

Summary

• 存在しないファイルを参照していた問題を修正
• 実装されていないログをチェックしていた問題を修正

Checklist

• Tests added/updated
• Unitテスト、他のE2Eテストをパスすることを確認

Summary by CodeRabbit

リリースノート

• テスト改善
  • 外部スタイルURLリクエストの検証テストを追加しました。
  • APIキー警告関連のテストを削除しました。
  • E2Eテスト構造が最適化されました。

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e9c5e670-a394-4e4a-85c1-f4b2171be9c5

📥 Commits

Reviewing files that changed from the base of the PR and between d9abfa7 and bb7d274.

📒 Files selected for processing (2)

• docs/e2e/external-style.html
• e2e/external-style.spec.ts

---

📝 Walkthrough

ウォークスルー

e2eテスト external-style.spec.ts が、相対パスから外部URLへのナビゲーション、ヘルパー関数を使用したマップ読み込み待機、テスト説明の日本語化に変更され、外部スタイルURL要求の検証テストを追加する修正です。

変更内容

コホート / ファイル(s)	概要
E2Eテスト外部スタイル e2e/external-style.spec.ts	相対パスナビゲーションを TEST_URL 外部パスに置き換え、マップ準備状態の直接ポーリングをヘルパー関数 waitForMapLoad に切り替え。テスト説明を日本語化。APIキー警告と外部スタイル読み込みログに関するテストを削除し、新たに外部スタイルURLリクエストの検証テストを追加。

推定コードレビュー工数

🎯 2 (Simple) | ⏱️ ~10分

ポエム

🐰 相対パスの迷宮を出て
外部URLへと進む道
ヘルパーの手を借りて
テストは確かに走る
日本語で美しく
バグは今や消えたり ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	プルリクエストのタイトルは、修正対象であるE2Eテストの失敗問題を適切に説明しており、変更内容の主要なポイントを明確に示しています。
Linked Issues check	✅ Passed	プルリクエストの変更内容（テストファイルパスの修正とログチェックの削除）がIssue #464で指摘された問題を解決しており、E2Eテストの失敗問題に対応しています。
Out of Scope Changes check	✅ Passed	すべての変更（テストのヘルパー関数化、ファイルパスの修正、ログチェックの削除）がIssue #464のE2Eテスト失敗問題の解決に直接関連しており、スコープ外の変更はありません。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-fail-e2e-test

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[smellman]

LGTM!

[yuiseki]

LGTM! Thank you for the fix.

I've confirmed that the two issues addressed here are valid:

1. external-style.html was in e2e/ but needed to be in docs/e2e/ to be served by the dev server
2. The removed console message checks ("API key not found", "Loading external style from") were testing for logs that don't exist in the codebase

The replacement test using page.waitForRequest to verify the external style URL is a better approach.

Note: PR #463 also includes these same E2E fixes as part of a larger change. Once this PR is merged, it would be good to coordinate with @niryuu to rebase #463 on master so the duplicate changes are resolved cleanly.