Skip to content

fix: 「色」ボタン化で壊れたiOSのUIテストを直す - #126

Merged
K0mork merged 1 commit into
mainfrom
fix/ui-test-after-color-dialog
Oct 7, 2026
Merged

fix: 「色」ボタン化で壊れたiOSのUIテストを直す#126
K0mork merged 1 commit into
mainfrom
fix/ui-test-after-color-dialog

Conversation

@K0mork

@K0mork K0mork commented Oct 7, 2026

Copy link
Copy Markdown
Owner

原因

mainでtestEditorChromeIgnoresPageZoomAndTextSelection（iOSのUIテスト）が失敗する。#123で足したこのテストは、ツール列の「色」の静的テキストを「ラベル」として長押ししていた。#115で「色」は「記号の色」ダイアログを開くボタン（aria-label付き）になり、中の「色」の文字が読み上げに出なくなったため、要素が見つからない（143行目・166行目）。2つのPRは別々にはCIが通っていたが、両方がマージされたmainで壊れた（PR #119のCIのios (iPhone 17)・ios (iPad (A16))で発生）。

変更

  • 長押しする「ラベル」を、見出しの下に出る編み図名の文字に替えた。起動時に開く編み図は先に実行したテストで変わるので、名前では探さず、読み上げの順で見出しの次の静的テキストを使う（documentNameText(below:in:)）。
  • テストの意図（見出し・ラベル・リンク・ボタンの長押しでメニューが出ないこと、ピンチ・ダブルタップで画面全体が拡大されないこと）は変えていない。
  • ほかのUIテストでfeat: 編み図で使っている色の一覧から色を選べるようにする #115・#124の画面変更に依存するものをgrepで確かめ、ほかに壊れたものは無かった。
  • docs/dev-log/2026-10-07-ui-test-after-color-dialog.mdを足した。

確認

  • 修正前のmainで、iPhone 16（iOS 18.2）のSimulatorで同じ箇所の失敗を再現した。
  • 修正後、同じSimulatorでtestEditorChromeIgnoresPageZoomAndTextSelection、testDocumentSwitchAutosavesEachDocument、testCoreEditorControlsExposeAccessibleNamesAndState、testTwoFingerGestureDoesNotDrawOnBoardがすべて成功した（別の編み図が開いた状態でも名前を見つけられた）。
  • iOS版のCSSの文字選択の抑止を一時的に外したビルドで、編み図名の長押しで文字選択のメニューが出てテストが失敗することを確かめてから戻した（コミットしない）。
  • iOSのSimulatorのテスト一式などはこのPRのCIに任せる。Web版と共通コードは変えていない。

🤖 Generated with Claude Code

@K0mork @claude
fix: update the long-press UI test for the color dialog button
96f1200 
The color tool became a dialog button in #115, so its "色" text is no
longer exposed as a static text and the long-press check added in #123
could not find it. Long-press the document name under the heading
instead, found by its position after the heading.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@K0mork K0mork added the bug Something isn't working label Oct 7, 2026
Copilot AI balanced review requested due to automatic review settings October 7, 2026 00:18
Copilot AI reviewed Oct 7, 2026

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@K0mork
K0mork merged commit 2bde0a3 into main Oct 7, 2026
15 checks passed
@K0mork
K0mork deleted the fix/ui-test-after-color-dialog branch October 7, 2026 00:38
This was referenced Oct 7, 2026
Merged
Merged
Merged
Merged
@K0mork

K0mork commented Oct 7, 2026

Copy link
Copy Markdown
Owner Author

デプロイ確認

main の e1a2001（#120 のマージ。#118・#123・#115・#124・#126・#120 をすべて含む）の CI and deploy Pages（run 37560031792）が成功しました。

  • ci-gate: 成功
  • deploy: 成功（github-pages 環境の最新のデプロイが e1a2001）
  • smoke: 成功（本番の HTML が配信物と一致し、参照する /assets/ などが取れること）
  • そのほかのジョブ（web、ios_web、ios（iPhone 17・iPad (A16)）、app_update（両端末）、release_archive）もすべて成功

途中の 736dd6d（#124）と 2bde0a3（#126）の run は、iOS の UI テストの失敗で ci-gate が通らず、配信されていません。前者は #115 と #123 の組み合わせで壊れたテスト（#126 で修正）、後者は Simulator の一時的な失敗です。

このPRについて

このPRは iOS だけの変更なので、Pages の配信には影響しません。この run で iOS のジョブ（ios（iPhone 17・iPad (A16)）、app_update（両端末）、release_archive）がすべて成功しました。実機（TestFlight）での確認は、まだ残っています。
#126 は iOS の UI テストだけの変更です。736dd6d の run で失敗した testEditorChromeIgnoresPageZoomAndTextSelection が、この run では iPhone と iPad の両方で成功しました。

@K0mork K0mork mentioned this pull request Oct 7, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees

No one assigned

Labels

bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@K0mork