Skip to content

Resolve "ee:geo QA specs are failing as of !7210"

Michael Kozono requested to merge resolve-issue-7434 into master

What does this MR do?

Resolves a race condition in Geo Nodes API spec vs GeoNodeStatus creation, by making the spec wait until they are created.

I believe it is reasonable for Geo node statuses to not exist on first startup. For performance reasons, it is not reasonable to update any status synchronously. We attempt to update the status every minute, so this initial condition goes away quickly.

  • Removes most tests in Geo Nodes API QA spec that are redundant with tests in the Geo Nodes request spec.
  • Adds to the Geo Nodes request spec to explicitly define behavior when 1 vs 2 geo node statuses exist
  • Avoids the race condition that caused failing tests

What are the relevant issue numbers?

Does this MR meet the acceptance criteria?

Edited by Michael Kozono

Merge request reports