dubee commented on a change in pull request #3799: Display proper error when sequence invocation fails due to missing component URL: https://github.com/apache/incubator-openwhisk/pull/3799#discussion_r206212295
########## File path: tests/src/test/scala/whisk/core/controller/test/SequenceApiTests.scala ########## @@ -53,6 +49,22 @@ class SequenceApiTests extends ControllerTestCommon with WhiskActionsApi { val allowedActionDuration = 120 seconds + it should "produce proper error when sequence component is missing during sequence invocation" in { + implicit val tid = transid() + val seqName = s"${aname()}_seq" + val compName = s"${aname()}_comp" Review comment: Hmm, I think we need to create a integration test if we want to rely on the activation to show the sequence partially activated. Using a unit test would block for 60 seconds while the first action activated as the action doesn't actually get invoked in a unit test, but instead times out. ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services