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 <sven.lange-l...@de.ibm.com> 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)