apache/openwhisk

WskRestOperation implements action create with copy but it should not

Aperta

#3846 aperta il 6 lug 2018

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

Metriche repository

Star
 (6777 stelle)
Metriche merge PR
 (Merge medio 17h 31m) (3 PR mergiate 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
Rimuovere l'operazione di copia da WskRestOperation's action create e spostare i test correlati nei test CLI.
Tech stack
scala
Dominio
apibackend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ScalaGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65