[GitHub] mdeuser commented on a change in pull request #3774: Add retry in WskRestBasicTests to avoid intermittent test failures

[GitBox <gi...@apache.org>]

mdeuser commented on a change in pull request #3774: Add retry in WskRestBasicTests to avoid intermittent test failures
URL: https://github.com/apache/incubator-openwhisk/pull/3774#discussion_r195868127
 
 

 ##########
 File path: tests/src/test/scala/system/basic/WskRestBasicTests.scala
 ##########
 @@ -120,7 +120,11 @@ class WskRestBasicTests extends TestHelpers with WskTestHelpers {
     }
 
     wsk.action.create(packageName + "/" + actionName, defaultAction, annotations = actionAnnots)
-    val result = wsk.pkg.get(packageName)
+    val result = cacheRetry({
+      val p = wsk.pkg.get(packageName)
+      p.getFieldListJsObject("actions") should have size 1
 
 Review comment:
   kind of.. the pkg.get having 1 action.  in some cross-controller cache race conditions, the package is obtained but does not yet have the freshly created action.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services