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

Reply via email to