fix: logic change: don't return from WAITING to NEW when the expectat…

…ion isn't ready to be worked on.

Reason: We should stay in waiting, since there should be no need to return to NEW (ie to do a new query for available workers), since at this poing we do have a worker assigned. Better then to simply stay at waiting to reduce the amount of unnecessary queries.

shared/packages/expectationManager/src/evaluationRunner/evaluateExpectationStates/waiting.ts

@@ -74,8 +74,8 @@ export async function evaluateExpectationStateWaiting({
 						})
 					} else {
 						// Not ready to start because of some other reason (e.g. the source doesn't exist)
+						// Stay here in WAITING state:
 						tracker.trackedExpectationAPI.updateTrackedExpectationStatus(trackedExp, {
-							state: ExpectedPackageStatusAPI.WorkStatusState.NEW,
 							reason: readyToStart.reason,
 							status: newStatus,
 						})