chetanmeh commented on issue #3583: RuncClientTests heisenbug URL: https://github.com/apache/incubator-openwhisk/issues/3583#issuecomment-384579256 > futureValue doesn't throw exceptions in a useful way (That's used for shouldBe thrownBy) I tried and exception testing was fine > is there much value of using futureValue over Await I prefer the `ScalaFutures` as it expresses the intention clearly and keeps timeout handling consistent across usages with decent defaults (15 secs). Further it opens up the possibility of [scaling][1] the timeouts as per environment in a global way i.e. increasing the scaling factor for slow CI setups. Currently various test have there own specific timeouts and which need to be tweaked as and when we see such failures. Much of scalatest is anyway magical! [1]: http://doc.scalatest.org/1.8/org/scalatest/concurrent/ScaledTimeSpans.html
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
