This is an automated email from the ASF dual-hosted git repository.
cbickel 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 8851cab Fix broken CLI tests. (#4084)
8851cab is described below
commit 8851cab741ad6cecce40a93275c4a2f3c1b82532
Author: Sven Lange-Last <[email protected]>
AuthorDate: Fri Oct 26 12:58:19 2018 +0200
Fix broken CLI tests. (#4084)
PR #3950 broke CLI tests in repo
https://github.com/apache/incubator-openwhisk-cli because these tests override
`val wsk` with a CLI specific implementation of type `Wsk` instead of
`WskRestOperations`.
This change makes sure that `val wsk` has the common parent class
`WskOperations`.
---
tests/src/test/scala/system/basic/WskActionTests.scala | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/tests/src/test/scala/system/basic/WskActionTests.scala
b/tests/src/test/scala/system/basic/WskActionTests.scala
index 7830da7..bec66b7 100644
--- a/tests/src/test/scala/system/basic/WskActionTests.scala
+++ b/tests/src/test/scala/system/basic/WskActionTests.scala
@@ -32,7 +32,9 @@ import spray.json.DefaultJsonProtocol._
class WskActionTests extends TestHelpers with WskTestHelpers with JsHelpers
with WskActorSystem {
implicit val wskprops = WskProps()
- val wsk = new WskRestOperations
+ // wsk must have type WskOperations so that tests using CLI (class Wsk)
+ // instead of REST (WskRestOperations) still work.
+ val wsk: WskOperations = new WskRestOperations
val testString = "this is a test"
val testResult = JsObject("count" -> testString.split(" ").length.toJson)
