apache/openwhisk

WskRestOperation implements action create with copy but it should not

Open

#3 846 ouverte le 6 juil. 2018

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)Scala (6 777 stars) (1 177 forks)batch import
help wantedtesting

Description

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.

Guide contributeur

Stack technique
scala
Domaine
apicli
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Scala basicsUnderstanding of OpenWhisk CLI vs API
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85
Direction de recherche
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.