csantanapr commented on issue #2911: Proposal to to reorganize core/ and root(openwhisk/) folders to reflect existing architecture better URL: https://github.com/apache/incubator-openwhisk/issues/2911#issuecomment-346614387 Hi @ServoKvd There should not be a "tests" repo, each component in whisk has a corresponding repo and that repo contains the source code for the component and the associated tests for that component. For the tests: 1. **tests/src/test/scala/common/rest/WskRest.scala** Should remain on this repo as is the base framework to run tests using REST API instead of using cli 2. **tests/src/test/scala/whisk/core/apigw/actions/test** Should be remove once the apiwg source for the actions are moved the CLI repo They are already duplicated in CLI repo https://github.com/apache/incubator-openwhisk-cli/tree/master/tests/src/test/scala/whisk/core/apigw/actions/test/ 3. **tests/src/test/scala/whisk/core/cli/test/WskApiGwTests.scala** This one is using **common.Wsk**, and is testing the CLI apigw feature, this tests is already in the CLI repo here https://github.com/apache/incubator-openwhisk-cli/blob/master/tests/src/test/scala/whisk/core/cli/test/WskApiGwTests.scala @houshengbo is this correct? That it should be removed, and also any tests using **common.Wsk** instead of **common.WskRest**
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
