apache/openwhisk

WskRestOperation implements action create with copy but it should not

Open

#3846 aperta il 6 lug 2018

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Scala (1177 fork)batch import
help wantedtesting

Metriche repository

Star
 (6777 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Il problema fa riferimento a un file specifico: tests/src/test/scala/common/rest/WskRestOperations.scala. Il compito è rimuovere l'operazione di copia dall'implementazione `wsk.action.create` in WskRestOperations e assicurarsi che tutti i test REST che utilizzano questa funzionalità vengano spostati nei test CLI. Innanzitutto, esaminare il codice collegato per comprendere l'implementazione corrente. Quindi, rimuovere il codice pertinente e verificare la presenza di eventuali test dipendenti. Infine, verificare che tutti i test passino.
Tech stack
scala
Dominio
apicli
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Scala basicsUnderstanding of OpenWhisk CLI vs API
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85