chetanmeh commented on a change in pull request #3890: Use ArtifactStore to 
manipulate data in NamespaceBlacklistTests
URL: 
https://github.com/apache/incubator-openwhisk/pull/3890#discussion_r206079624
 
 

 ##########
 File path: tests/src/test/scala/whisk/core/database/test/DbUtils.scala
 ##########
 @@ -128,6 +128,28 @@ trait DbUtils extends Assertions {
     assert(success.isSuccess, "wait aborted")
   }
 
+  /**
+   * Wait on a view to update with documents added(don't specify the 
namespace). This uses retry above,
+   * where the step performs a direct db query to retrieve the view and check 
the count
+   * matches the given value.
+   */
+  def waitOnView[Au](db: ArtifactStore[Au], count: Int, view: View)(implicit 
context: ExecutionContext,
 
 Review comment:
   Given this we can then refactor `ArtifactStoreSubjectQueryBehaviors` to drop 
`waitOnBlacklistView` and `blacklistCount` methods

----------------------------------------------------------------
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

Reply via email to