[smoke-test] Test coverage review: 411 source files, 13 test files (3.2%) #24

New issue

Closed

opened 2026-05-15 04:46:37 +00:00 by Mark · 2 comments

Mark commented

2026-05-15 04:46:37 +00:00

Owner

Smoke Test Review: Mark/nodedesk

Type: node | Runtime: docker
Source files: 411 | Test files: 13 | Coverage: 3.2%

Observations

Frontend: 72 component files in src/
Test directory: tests, test-results

Risks

⚠️ Low test coverage: 3.2% of source files have tests

Recommendations

Increase test coverage to at least 50%
Install a test framework (vitest recommended for Vite projects)

Auto-filed by project-smoke-test.py on 2026-05-15 04:46:37 UTC

## Smoke Test Review: Mark/nodedesk **Type:** node | **Runtime:** docker **Source files:** 411 | **Test files:** 13 | **Coverage:** 3.2% ### Observations - Frontend: 72 component files in src/ - Test directory: tests, test-results ### Risks - ⚠️ Low test coverage: 3.2% of source files have tests ### Recommendations - Increase test coverage to at least 50% - Install a test framework (vitest recommended for Vite projects) --- _Auto-filed by project-smoke-test.py on 2026-05-15 04:46:37 UTC_

Mark added the

blocked:needs-decision

label

2026-05-15 04:46:37 +00:00

Mark commented

2026-05-15 04:51:40 +00:00

Author

Owner

GSD RESEARCH could not determine a clear fix.

STATUS: UNCLEAR

EVIDENCE: The issue reports 3.2% test coverage (13 Playwright E2E test files vs 411 source files). The codebase hasn't changed since the issue was filed — same test files, no new test infrastructure added. However, the issue's specific recommendation ("Install a test framework — vitest recommended for Vite projects") is wrong for this codebase: nodedesk is a Next.js 14 project with Webpack bundling, not a Vite project. Vitest would require custom configuration for Next.js pa

GSD RESEARCH could not determine a clear fix. STATUS: UNCLEAR EVIDENCE: The issue reports 3.2% test coverage (13 Playwright E2E test files vs 411 source files). The codebase hasn't changed since the issue was filed — same test files, no new test infrastructure added. However, the issue's specific recommendation ("Install a test framework — vitest recommended for Vite projects") is wrong for this codebase: nodedesk is a **Next.js 14** project with Webpack bundling, not a Vite project. Vitest would require custom configuration for Next.js pa

Mark added

bug

and removed

blocked:needs-decision

labels

2026-05-15 13:43:06 +00:00

Mark added the

triage-approved

label

2026-05-17 22:52:50 +00:00

Mark commented

2026-05-19 08:28:00 +00:00