[Tests] fix posttest script on Windows

[targos]

Use double quotes in cmd argument.
Resolve deprecation warning for --production.

Closes: #331

[socket-security]

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package	New capabilities	Transitives	Size	Publisher

View full report↗︎

[ljharb]

Also, appveyor continues to pass (and fail) just like it does on main. Is there some kind of appveyor test we can add that would have caught this?

[targos]

The other more simple solution is to remove posttest from the citgm config

[ljharb]

while that's also a fine citgm workaround, it'd be nice if windows contributors could run the tests locally :-)

[targos]

Also, appveyor continues to pass (and fail) just like it does on main. Is there some kind of appveyor test we can add that would have caught this?

appveyor doesn't run the posttest script:

resolve/appveyor.yml

Lines 70 to 81 in cf4ee88

	# Post-install test scripts.
	test_script:
	# Output useful info for debugging.
	- node --version
	- npm --version
	# run tests
	- cd
	- npm run tests-only
	- 'subst X: ..'
	- 'cd /d X:\resolve'
	- cd
	- npm run tests-only

it'd be nice if windows contributors could run the tests locally

npm test still fails on Windows because of the prelint script which uses bash-specific syntax (I won't fix it)

[ljharb]

Gotcha, this is fine as-is then. Thanks!