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 天 1 小时
- 30 天内合并 PR
- 39
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南，环境需要你自己搭建：先看它的 README，通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
npmx-dev/npmx.dev 的其他 Issue
-
pending triage
难度 2/5 1-3 小时 新手友好度 76/100
维护者通常 1 天内回复
-
难度 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 个 reaction ·
维护者通常 1 天内回复
-
back stale stale-to-close
难度 2/5 1-3 小时 新手友好度 62/100
npmx-dev/npmx.dev#2511 · 4 条评论 · 1 个 reaction ·
维护者通常 1 天内回复
查看 npmx-dev/npmx.dev 的全部 Issue
相似的 Issue
-
area:desktop bug documentation
难度 2/5 1-3 小时 新手友好度 76/100
anthropics/claude-code#104801 ·
维护者通常 1 天内回复
-
check:passed streams:add
难度 2/5 1-3 小时 新手友好度 67/100
维护者通常 2 天内回复
-
难度 2/5 1-3 小时 新手友好度 68/100
spudly/design-system#110 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 85/100
维护者通常 1 天内回复
-
[Bug]: REST API URL starting with "http" but no scheme (e.g. httpbin.org/get) is not given an http:// prefix可能已有人在做 @ItsDeadlyProgrammer 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 68/100
appsmithorg/appsmith#42471 · 1 条评论 ·
维护者通常 2 天内回复