tests: start server on integration test

[straker]

Closes issue: #2658

Reviewer checks

Required fields, to be filled out by PR reviewer(s)

• Follows the commit message policy, appropriate for next version
• Code is reviewed for security

[stephenmathieson]

Looks like we're intentionally adding a race condition, but I'm not sure how else to handle this. LGTM but we should keep an eye on this.

[straker]

Nope, this is still timing out after 47 tests... I have no idea what is going on but I don't know how else to do this in a single command so we don't have to start the server beforehand

[stephenmathieson]

If you like, we can pair on it next week. Should I set something up?

[straker]

IT WORKS! So this is the "best" way I've found to do this but it adds a lot of npm scripts.

start-server-and-test does what we want, but it has a problem in that it can't pass command line arguments to the test command. That resulted in creating individual test scripts for each of the browsers. Unfortunately, it also has a bug in Windows that doesn't allow it to work with npm commands, so I had to create yet another set of individual test scripts so we can run the script name instead of a command.

[stephenmathieson]

Well done! Glad you figured it out!