WskRestOperation implements action create with copy but it should not · apache/openwhisk#3846

(1 comment) (0 reactions) (0 assignees)Scala (6,777 stars) (1,177 forks)batch import

help wantedtesting

説明

The copy operation is a function of the cli, not the rest api. So supporting a copy in the WskRestOperation's implementation of wsk.action.create doesn't make sense.

https://github.com/apache/incubator-openwhisk/blob/3ab604c167ac5bec76c756e3e76473863eb0a25d/tests/src/test/scala/common/rest/WskRestOperations.scala#L332

The code should be removed and any rest tests in place should be moved strictly to the cli tests. It's arguable we should preserve this behavior --- if we reify this implementation into a scala SDK for openwhisk. Otherwise, I'm not sure why we should support this.

コントリビューターガイド

技術スタック: scala
領域: apicli
Issue 種別: refactor
難度: 2
推定時間: under 1 hour
活動状況: stale
明確さ: clear
前提条件: Scala basicsUnderstanding of OpenWhisk CLI vs API
初心者向け度: 85
調査方針: The issue points to a specific file: tests/src/test/scala/common/rest/WskRestOperations.scala. The task is to remove the copy operation from the `wsk.action.create` implementation in WskRestOperations and ensure that any REST tests using this feature are moved to CLI tests. First, examine the linked code to understand the current implementation. Then, remove the relevant code and check for any dependent tests. Finally, verify that all tests pass.