test: test:e2e fails, and the website, files, manifest and watch code have no tests

Offen
  • Stimmt etwas nicht?
#139 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
52/100
Issue-Typ
Feature
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
node.js, typescript
Bereich
testing, tooling

Rechercherichtung

Start with the root package.json and turbo.json to trace why pnpm test:e2e has no task, then inspect the existing tests for safe-path and editor-ipc. The issue proposes several possible scopes: define or remove the e2e task, add loader tests for /manifest, /env and /game/*, and add website cache tests. Agree on a scope first; done means the selected tests or script run successfully.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

enhancement tests
Which app is this feature request for?

other

Feature

pnpm test:e2e (root package.json) fails because no package defines the task, and turbo.json has no test:e2e entry (CI passes e2e-script: ""):

$ pnpm test:e2e
x Could not find task `test:e2e` in project
[ELIFECYCLE] Command failed with exit code 1.
$ pnpm test:unit   ->  Tasks: 2 successful, 2 total  (the website has no test:unit)

Unit tests only cover safe-path (client) and editor-ipc (server). The riskiest code has no test: the env filter (env.ts), files.ts in both apps, manifest hashing, worker spawn and restart, the watchers, and the website cache (cache.ts: hash, size and eviction).

Found during a read-only audit of main @ 044c766 (2026-11-11).

Ideal solution or implementation
  • Either remove test:e2e from the root scripts, or add a minimal e2e: start the client and server loaders on a fixture build, then request /manifest, /env and /game/*.
  • Add unit tests for getGameEnv (client and server), getFiles and updateFiles.
  • Add a test:unit to apps/website for the cache logic.
Alternative solutions or implementations

No response

Other context

Several bugs found in the same audit sit in this untested code (env filtering, worker restart, watchers).

Vorherrschende Sprache
TypeScript
Sterne
1
Forks
0
Ø Merge
17 Std. 25 Min.
Gemergte PRs (30 T.)
11

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus NanoForge-dev/Loader

Alle Issues in NanoForge-dev/Loader

Ähnliche Issues

Weitere Issues zu TypeScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.