Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ajax Test missing on the error case behavior in the request chain #1800

Open
werpu opened this issue Mar 10, 2023 · 1 comment
Open

Ajax Test missing on the error case behavior in the request chain #1800

werpu opened this issue Mar 10, 2023 · 1 comment

Comments

@werpu
Copy link
Contributor

werpu commented Mar 10, 2023

Hi, I noticed following, the spec is rather vague on how to treat the error chain in the ajax error case.

The spec states in the jsdocs:

Regardless of the outcome of the request (success or error) every request in the queue must be handled. Examine the status of each request in the queue starting from the request that has been in the queue the longest. If the status of the request is complete (readyState 4), dequeue the request (remove it from the queue). If the request has not been sent (readyState 0), send the request. Requests that are taken off the queue and sent should not be put back on the queue.

I am not aware of a test testing this behavior! (I might have missed one)

A test would look like this:

  1. Offer a series of xhr request
  2. One of the tests in the middle causes an error on the server (exception suffices in an action or action listener)!
  3. The result must be that all 5 requests must have been processed one way or the other and onerror must have been called!
@werpu
Copy link
Contributor Author

werpu commented Mar 10, 2023

I have the test working, but it needs a proper name.
Question, how should I name the test? I probably cannot name it Issue1800IT because it refers to a github issue!
Once the name issue is cleared i can offer a pull request! (very likely monday)
To test exactly this behavior!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant