Skip to content

ビルドせずにE2Eテストを実行した場合にExternal Style Support関連のテストが失敗する問題を修正#471

Merged
sanak merged 2 commits into
masterfrom
fix-external-style-e2e-test
Mar 16, 2026
Merged

ビルドせずにE2Eテストを実行した場合にExternal Style Support関連のテストが失敗する問題を修正#471
sanak merged 2 commits into
masterfrom
fix-external-style-e2e-test

Conversation

@sanak

@sanak sanak commented Mar 13, 2026

Copy link
Copy Markdown
Member

Fixes #468

Summary

#468 について、対応案3で記載していた内容となります。

Checklist (optional)

Summary by CodeRabbit

リリースノート

  • ドキュメント
    • 外部スクリプト参照のパス表記を更新しました(読み込み先の参照を変更)。
    • READMEにエンドツーエンド(E2E)テスト実行コマンドの記載を追加しました(ローカルでの実行手順を明示)。

@sanak
sanak requested a review from yuiseki March 13, 2026 08:51
@sanak sanak self-assigned this Mar 13, 2026
@coderabbitai

coderabbitai Bot commented Mar 13, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

docs/e2e/external-style.html の script リファレンスを embed.js から拡張子なしの embed に変更し、README に e2e 実行コマンド (npm run e2e) の記載を追加しました。動作や公開 API の宣言には変更はありません。

Changes

Cohort / File(s) Summary
External Style E2E Test
docs/e2e/external-style.html
<script src="../embed.js"><script src="../embed"> に変更。webpack 開発サーバーでホストされるファイル名に合わせた修正。
Documentation: E2E command
README.md
開発手順とテスト節に + $ npm run e2e # run e2e tests を追記して、E2E 実行コマンドを明示。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐇 embed の道しるべを直したよ
拡張子をそっと脱がせて、
テストの風がやさしく通る。
ビルドの手間が一つ消え、
にんじん一つで乾杯だよ!

🚥 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 プルリクエストのタイトルは、#468で報告されたExternal Style Support関連のテスト失敗問題を修正することを明確に説明している。
Linked Issues check ✅ Passed PR#471の変更内容は、#468で提案された対応案3(external-style.htmlを拡張子なしで読み込む変更)とREADMEのE2Eテスト実行手順の追加要件を完全に満たしている。
Out of Scope Changes check ✅ Passed すべての変更(external-style.htmlのscript参照修正とREADMEへのe2e実行手順追加)は、#468で指定された対応案の範囲内である。
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 unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-external-style-e2e-test
📝 Coding Plan
  • Generate coding plan for human review comments

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

@sanak
sanak requested review from bougan1160 and niryuu March 13, 2026 08:51

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@README.md`:
- Line 106: Remove the shell prompt prefix from the example command in
README.md: replace the line containing "$ npm run e2e" with the same command
without the leading "$" so it reads as the plain command (e.g., npm run e2e) to
satisfy markdownlint MD014 and avoid CI lint noise; just edit the README.md
entry that currently shows "$ npm run e2e".

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0b366609-be12-49c4-96d2-f7aa5bd1de25

📥 Commits

Reviewing files that changed from the base of the PR and between f96fa43 and ff7991e.

📒 Files selected for processing (1)
  • README.md

Comment thread README.md

@yuiseki yuiseki left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@sanak
sanak merged commit 3668dfc into master Mar 16, 2026
10 checks passed
@sanak
sanak deleted the fix-external-style-e2e-test branch March 16, 2026 03:32
@yuiseki yuiseki mentioned this pull request Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

E2Eテスト実行時に、 npm run build を実行していないとExternal Style Support関連のテストが失敗する

2 participants