rabbah commented on a change in pull request #2326: Alphabetize listings in CLI
URL: 
https://github.com/apache/incubator-openwhisk/pull/2326#discussion_r132253606
 
 

 ##########
 File path: tests/src/test/scala/system/basic/WskBasicTests.scala
 ##########
 @@ -185,6 +185,27 @@ class WskBasicTests
         stderr should include regex (s"""Unable to get package '$name': The 
requested resource does not exist. \\(code \\d+\\)""")
     }
 
+    it should "return a list of alphabetized packages" in 
withAssetCleaner(wskprops) {
+        (wp, assetHelper) =>
+            // Declare 3 packages, create them out of alphabetical order
+            val packageName = "activationBasicTesting"
+            for (i <- 1 to 3) {
+                val name = s"$packageName$i"
+                assetHelper.withCleaner(wsk.pkg, name) {
+                    (pkg, name) =>
+                        pkg.create(name)
+                }
+            }
+            val original = wsk.pkg.list(nameSort = Some(true)).stdout
 
 Review comment:
   it may be prudent to wrap these checks with a `retry` (here and elsewhere) 
since a list operation may take some time to converge.
 
----------------------------------------------------------------
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