Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add test for WAIC-TEST-0029-01 #479

Open
wants to merge 1 commit into
base: betajp
Choose a base branch
from
Open

Add test for WAIC-TEST-0029-01 #479

wants to merge 1 commit into from

Conversation

nishimotz
Copy link
Member

@nishimotz nishimotz commented Sep 22, 2024

問題番号へのリンク:

問題の概要:

NVDA 日本語版をリリースするたびに、WAIC は AS テスト を行い、特に AS 情報(例: aria-describedby 属性の正しい動作)を整備する必要があります。

この作業は手動で行う必要があり、不定期なボランティア活動に依存しています。自動化により、特定の支援技術とブラウザの組み合わせに限られますが、AS 情報の整備を効率化できます。

さらに、可能であれば NVDA 日本語版のビルドやリリースにおいて、AS テストのリグレッションを防ぐべきです。これにより、以前のバージョンで正常に動作していた機能が新しいリリースで壊れることを防ぎ、ユーザーに対して一貫した品質を提供することができます。

ユーザーに対する変更の説明

このプルリクエストは、test_waic_as_0029_01 テストケースを追加し、aria-describedby 属性による説明ラベルの提供を検証します。このテストは、ボタン要素と aria-label 属性を併用した場合の動作を確認します。

https://github.com/waic/as_test/blob/master/WAIC-TEST/HTML/WAIC-TEST-0029-01.md

なお、システムテストではNVDAの読み上げを文字列として比較しています。これは「期待される結果」の一例に過ぎないため、文字列が一致しなかった場合は、テストを修正するか、ASでないと判断するか、詳細な検討が必要です。

開発アプローチの説明

テストは、特定の HTML 構造を持つページを読み込み、aria-describedby 属性が正しく機能するかどうかを確認します。テストは、NVDA が正しいスピーチを出力することを確認するために、スクリプトを使用して実行されます。

テスト戦略:

  • 既存のユニットテストとシステムテストに加えて、新しいシステムテストを追加します。
  • ブラウザは Google Chrome に限られています。
  • 本家版のテストに合わせて、NVDA の言語は en のまま出力を検証します。
  • https://github.com/nvdajp/nvdajp/blob/betajp/readme-nvdajp.md

プルリクエストに関する既知の問題:

特に既知の問題はありませんが、テストが他のテストに影響を与えないことを確認する必要があります。

コードレビュー チェックリスト:

  • ドキュメント:
    • 変更ログのエントリ
    • ユーザードキュメント
    • 開発者 / 技術ドキュメント
    • GUI 変更に対するコンテキストセンスのヘルプ
  • テスト:
    • ユニットテスト
    • システム(エンドツーエンド)テスト
    • 手動テスト
  • すべてのユーザーの UX を考慮:
    • スピーチ
    • 点字
    • 低視力
    • 異なるウェブブラウザ
    • 英語以外の言語 / 文化におけるローカリゼーション
  • API は既存のアドオンと互換性があります。
  • セキュリティ対策が講じられています。

@coderabbitai summary

@nishimotz nishimotz changed the title test_waic_as_0029_01: Add test for WAIC-CODE-0029-01.html Add test for WAIC-CODE-0029-01.html Sep 22, 2024
@nishimotz nishimotz changed the title Add test for WAIC-CODE-0029-01.html Add test for WAIC-TEST-0029-01 Sep 22, 2024
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.

1 participant