Skip to content
This repository has been archived by the owner on Dec 5, 2019. It is now read-only.

Pipeline:Rest - Add timeout setting for pipes #25

Merged
merged 2 commits into from Feb 19, 2013

Conversation

kborchers
Copy link
Contributor

No description provided.

@@ -611,4 +611,14 @@ $.mockjax({
}
});

$.mockjax({
url: "long",
type: "GET",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would it be same for POST etc ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, they all behave the same so didn't bother with those tests

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hrm - ok..

@qmx
Copy link
Contributor

qmx commented Feb 19, 2013

+1

@matzew
Copy link
Contributor

matzew commented Feb 19, 2013

Is the tests/unit/pipeline/pipeline-rest.js testing also that no success has been invoked, on an actual timeout ?

@matzew
Copy link
Contributor

matzew commented Feb 19, 2013

👍

@kborchers kborchers merged commit cd02eee into aerogear-attic:master Feb 19, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants