Fix backend test suite failures

Offen
  • Stimmt etwas nicht?
#2,136 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 4 Tagen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
45/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Ruhig
Tech-Stack
javascript
Bereich
backend, testing-qa

Rechercherichtung

Beginne mit dem Backend-Vitest-Runner und den genannten Router-Tests, models/event.model.test.js, models/projectTeamMember.model.test.js und setup-test.js. Prüfe die in den Aktionspunkten beschriebenen Fehler und das Test-Setup und führe anschließend die Backend-Suite aus. Erledigt ist die Aufgabe, wenn alle aufgeführten Tests zuverlässig bestehen, einschließlich der teardown- und empty-describe-Fälle, ohne Race-Condition-Fehler.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Bug complexity: medium feature: Package Update housekeeping role: Back End size: 2pt
Overview

The backend test suite has several pre-existing failures that predate the ESM migration. These need to be resolved so the test suite can serve as a reliable quality gate.

Action Items
  • Router tests (auth, events, projects, users) fail because process.env.CUSTOM_REQUEST_HEADER is undefined — tests should set required env vars themselves
  • models/event.model.test.js and models/projectTeamMember.model.test.js have updatedDate assertions that fail because Mongoose pre-update hooks aren't firing on findOneAndUpdate and updateMany
  • setup-test.js has un-awaited deleteMany() calls causing race conditions between tests
  • MongoMemoryServer teardown produces MongoNetworkError: connection closed errors from parallel test execution
  • Empty describe blocks cause Vitest to fail (Jest silently skipped them)
Resources/Instructions
  • Related: #2133 (ESM migration)
  • Backend test runner: Vitest (migrated from Jest as part of #2133)
Vorherrschende Sprache
JavaScript
Sterne
63
Forks
106
Ø Merge
3 T. 19 Std.
Gemergte PRs (30 T.)
5

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 hackforla/VRMS

Alle Issues in hackforla/VRMS

Ähnliche Issues

Weitere Issues zu JavaScript

Neue Issues direkt in Ihr Postfach

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