Conversation
There was a problem hiding this comment.
this test got disabled because windows has no lsof command, so the childprocess stream doesn't work
|
Common themes that we could consider moving to some utility functions are to create |
|
Totally didn't notice #1547 but that's a better start to this. It does need a bit more, but I think that should get merged first and then this pull request updated. |
|
Does #1547 change any of these modifications? |
|
Yes, it'll make it easy to clean things up if that one gets merged first. — On Thu, Apr 3, 2014 at 11:03 PM, Eran Hammer [email protected]
|
|
Now that #1547 was merged I'll get this updated |
|
Ok, this is now down to the above mentioned timing issues.. All of which are inconsistent. One pass they run, next pass they're off by a fairly large margin. |
2 participants
This is where I'm at so far. The remaining problems are inconsistent, and all timeout related
test/server.jsid 788 'Server Timeouts return server error message when server taking too long'test/server.jsid 791 'Server Timeouts does not return an error response when server is slow but faster than timeout'test/clientTimeout.jsid 44 'Client Timeout does not return a client error message when response is taking a long time to send'test/proxy.jsid 333 'Proxy times out when proxy timeout is less than server'I have no idea at all how to fix these ones, so any hints would be appreciated.