fix(qwik-city): correct validator failed type

[wtlin1228]

Overview

• Wrap the type of rest DataValidators with FailReturn to correct the type if we're using action with validator$s.
• Update the tests for routeAction$ with validators.

What is it?

• Feature / enhancement
• Bug
• Docs / tests / types / typos

Description

Before:

const action1 = useAction1();
if (action1.value) {
  if (action1.value.failed) {
    action1.value satisfies { failed: true } & (
      | TypedDataValidatorError
      | ActionFailedObject
    );
  } else {
    action1.value satisfies ActionSuccessObject | DataValidatorError;  // ❌ DataValidatorError shouldn't be here
  }
}

After:

if (action1.value) {
  if (action1.value.failed) {
    action1.value satisfies { failed: true } & (
      | TypedDataValidatorError
      | DataValidatorError // ✅ DataValidatorError should be here
      | ActionFailedObject
    );
  } else {
    action1.value satisfies ActionSuccessObject;
  }
}

Use cases and why

• 1. One use case
• 2. Another use case

Checklist:

• My code follows the developer guidelines of this project
• I have performed a self-review of my own code
• I have made corresponding changes to the documentation
• Added new tests to cover the fix / functionality

[netlify]

👷 Deploy request for qwik-insights pending review.

Visit the deploys page to approve it

Name	Link
🔨 Latest commit	8468d5c

[mhevery]

You rock!