Flaky Tests Invalid server state: initialized

[dplewis]

New Issue Checklist

• Report security issues confidentially.
• Any contribution is under this license.
• Before posting search existing issues.

Issue Description

The test suite in this repo and in the JS SDK both have flaky tests. I believe this is due to the asyncronous nature of starting the server instance.

await ParseServer.startApp(newConfiguration);

I think the issue is after the promise resolves the server is still initializing. https://github.com/parse-community/parse-server/pull/8232

Parse.Query testing dontSelect query without conditions
  - Unhandled promise rejection: ParseError: 111 Invalid server state: initialized,ParseError: 111 Invalid server state: initialized,ParseError: 111 Invalid server state: initialized,ParseError: 111 Invalid server state: initialized,ParseError: 111 Invalid server state: initialized  

Steps to reproduce

I can't reproduce this locally

Actual Outcome

Tests fail

Expected Outcome

Test Pass

[parse-github-assistant[bot]]

Thanks for opening this issue!

• 🚀 You can help us to fix this issue faster by opening a pull request with a failing test. See our Contribution Guide for how to make a pull request, or read our New Contributor's Guide if this is your first time contributing.

[mtrezza]

So initialized is the state after calling the ParseServer constructor. As soon as you call start() on the instance, the state changes to starting. From what you describe it looks like start() has not been called at all, because starting is set synchronously after calling start(). So it may be related to the reconfigureServer logic; that promise may return early, rather than the server init itself. Which would be good, because that means it's only a testing issue, not a server issue.

[mtrezza]

Closing via https://github.com/parse-community/Parse-SDK-JS/pull/2183

[mtrezza]

Re-opened, I think we first need to upgrade the Parse JS SDK in Parse Server to see whether the Parse Server CI shows less flakiness.

[dplewis]

I agree, I still don’t know why this is happening. I would like to think I solved it on the JS SDK

[mtrezza]

Issue still occurs in recent PRs; I have never seen this before; is this a recently introduced issue?

[dplewis]

@mtrezza Looks like the server shutsdown and restarts randomly after a few hundred tests. If we turn off the randomized tests we should find a pattern.

[mtrezza]

Okay, if you want to open a PR to temporarily disable random tests I'll go ahead and merge

[parseplatformorg]

🎉 This change has been released in version 7.2.0-beta.1

[parseplatformorg]

🎉 This change has been released in version 7.2.0