Add accessibility coverage for PackageSizeIncreaseAnalysis
メンテナーはふだん 1 日以内に返信
評価
この issue はまだ評価されていません。
説明
Summary
Add automated Vitest accessibility coverage for PackageSizeIncreaseAnalysis and remove its temporary exclusion from the component-coverage guard.
The component currently depends on the worker-backed useAnalyzeCauseWorker composable, so the test must isolate the UI from real Web Worker and IndexedDB behaviour.
Required changes
- Add an accessibility/component test for
app/components/Package/SizeIncreaseAnalysis.vue. - Mock
useAnalyzeCauseWorker(and the worker boundary if the established test setup requires it) with reactive, type-safe state and no-op actions. - Cover the initial available state and assert that axe reports no accessibility violations.
- Remove the
Package/SizeIncreaseAnalysis.vueTODO/entry fromSKIPPED_COMPONENTSintest/unit/a11y-component-coverage.spec.ts.
Rationale
The current skip leaves the new package-size analysis UI outside the repository’s required component accessibility coverage. A deterministic mock prevents browser-worker and IndexedDB dependencies from making the test brittle.
Affected areas
app/components/Package/SizeIncreaseAnalysis.vueapp/composables/pkg-size/useAnalyzeCauseWorker.tstest/nuxt/a11y.spec.ts(or the established component-test location)test/unit/a11y-component-coverage.spec.ts
Acceptance criteria
- A Vitest test mounts
PackageSizeIncreaseAnalysiswith a mocked analysis composable. - The test performs an axe accessibility assertion without starting a real worker.
- The temporary coverage skip and its TODO are removed.
- The relevant test suite passes.
Requested by @userquin.
Backlinks: https://github.com/npmx-dev/npmx.dev/pull/3077 and https://github.com/npmx-dev/npmx.dev/pull/3077#discussion_r3649237902
- 主要言語
- TypeScript
- スター
- 3.6k
- フォーク
- 533
- 平均マージ
- 4日 13時間
- マージ済み PR（30日）
- 37
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
npmx-dev/npmx.dev のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
a11y front
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
npmx-dev/npmx.dev#2914 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
-
front i18n
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
npmx-dev/npmx.dev#2831 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
back stale stale-to-close
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
npmx-dev/npmx.dev#2511 · コメント 4 件 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
front needs discussion ux
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
npmx-dev/npmx.dev#2275 · コメント 5 件 · リアクション 2 件 ·
メンテナーはふだん 1 日以内に返信
npmx-dev/npmx.dev の issue をすべて見る
似ている issue
-
createAI: the ai peer range allows ai@6, which the provider cannot work with対応中かも @akhil9tiet が今日担当しました。 オープンbug
難易度 1/5 1時間未満 初心者へのやさしさ 83/100
cloudflare/agents#2690 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
-
[fullsend] E2E: theme — global-header ignores app.branding.fullLogo (RHDHBUGS-3670)対応中かも @fullsend-ai-coder が今日担当しました。 オープンe2e-failure ready-to-code
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
redhat-developer/rhdh-plugin-export-overlays#4503 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
needs triage 📥
難易度 1/5 1時間未満 初心者へのやさしさ 86/100
ethereum/ethereum-org-website#19601 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
cloudscape-design/components#5184 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
agilepathway/label-checker#805 ·
メンテナーはふだん 1 日以内に返信