X Tutup
Skip to content

Fixing test p5.js web#3986

Open
Jatin24062005 wants to merge 2 commits intoprocessing:developfrom
Jatin24062005:fixing_Test_P5.js_web
Open

Fixing test p5.js web#3986
Jatin24062005 wants to merge 2 commits intoprocessing:developfrom
Jatin24062005:fixing_Test_P5.js_web

Conversation

@Jatin24062005
Copy link
Contributor

@Jatin24062005 Jatin24062005 commented Mar 9, 2026

Issue:

Fixes #3978

Demo:

image

Changes

  1. Added cross-env to scripts for consistent environment variable handling across platforms.
  2. Removed the explicit MongoDB version from MongoMemoryServer.create() to avoid timeout errors and repeated binary downloads during tests.

Why

  • cross-env ensures scripts run correctly on Windows, macOS, and Linux.
  • Dropping the pinned MongoDB version prevents Jest hooks from stalling while fetching unsupported or slow binaries.

I have verified that this pull request:

  • has no linting errors (npm run lint)
  • has no test errors (npm run test)
  • has no typecheck errors (npm run typecheck)
  • is from a uniquely-named feature branch and is up to date with the develop branch.
  • is descriptively named and links to an issue number, i.e. Fixes #123
  • meets the standards outlined in the accessibility guidelines

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tests not runnable on running npm run test ( NODE_ENV not recognized)

1 participant

X Tutup