apache/openwhisk

WskRestOperation implements action create with copy but it should not

Open

#3.846 aberto em 6 de jul. de 2018

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)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.

Guia do colaborador

Pilha de tecnologia
scala
Domain
apicli
Tipo Issue
refactor
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Scala basicsUnderstanding of OpenWhisk CLI vs API
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85
Direção de pesquisa
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.