[incubator-openwhisk] branch master updated: Add additional debug information for intermittently failing ContainerProxyTests (#4278)

[rabbah]

This is an automated email from the ASF dual-hosted git repository.

rabbah pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk.git


The following commit(s) were added to refs/heads/master by this push:
     new a205f8b  Add additional debug information for intermittently failing 
ContainerProxyTests (#4278)
a205f8b is described below

commit a205f8b50311181e14e8af6d8eb91eba84045ea6
Author: Christian Bickel <git...@cbickel.de>
AuthorDate: Wed Feb 13 15:38:38 2019 +0100

    Add additional debug information for intermittently failing 
ContainerProxyTests (#4278)
    
    The `ContainerProxyTests` are failing intermittently in the pipeline. All 
of these intermittent test failures have either a wrong initialisation-time or 
it is missing completely.
    Our theory currently is, that the Buffer in `acker` is called in the wrong 
order, as these calls are made asynchronously. If that theory is correct, this 
can be proven with this additional output.
---
 .../core/containerpool/test/ContainerProxyTests.scala    | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git 
a/tests/src/test/scala/org/apache/openwhisk/core/containerpool/test/ContainerProxyTests.scala
 
b/tests/src/test/scala/org/apache/openwhisk/core/containerpool/test/ContainerProxyTests.scala
index b51832b..1c4bb51 100644
--- 
a/tests/src/test/scala/org/apache/openwhisk/core/containerpool/test/ContainerProxyTests.scala
+++ 
b/tests/src/test/scala/org/apache/openwhisk/core/containerpool/test/ContainerProxyTests.scala
@@ -303,6 +303,10 @@ class ContainerProxyTests
       collector.calls should have size 2
       container.suspendCount shouldBe 0
       acker.calls should have size 2
+
+      // Add debugging information for the case of failure.
+      println(acker.calls.map(_._2.toJson.compactPrint))
+
       store.calls should have size 2
 
       val initRunActivation = acker.calls(0)._2
@@ -360,6 +364,10 @@ class ContainerProxyTests
       container.suspendCount shouldBe 1
       container.resumeCount shouldBe 1
       acker.calls should have size 2
+
+      // Add debugging information for the case of failure.
+      println(acker.calls.map(_._2.toJson.compactPrint))
+
       store.calls should have size 2
       acker
         .calls(0)
@@ -548,6 +556,10 @@ class ContainerProxyTests
       container.suspendCount shouldBe 1
       container.resumeCount shouldBe 0
       acker.calls should have size 6
+
+      // Add debugging information for the case of failure.
+      println(acker.calls.map(_._2.toJson.compactPrint))
+
       store.calls should have size 6
       acker
         .calls(0)
@@ -611,6 +623,10 @@ class ContainerProxyTests
       container.suspendCount shouldBe 0
       container.destroyCount shouldBe 0
       acker.calls should have size 2
+
+      // Add debugging information for the case of failure.
+      println(acker.calls.map(_._2.toJson.compactPrint))
+
       store.calls should have size 2
 
       val initErrorActivation = acker.calls(0)._2