integration tests require manual setup and cleanup about openwhisk-client-js HOT 7 CLOSED

I was not able to run the integrations tests running npm run test-integration

from openwhisk-client-js.

@csantanapr Can you provide more details about what exactly failed?

I have run them recently and didn't have any issues but that's using my setup :)

from openwhisk-client-js.

There are no instructions what environment variables I need to set before running the tests, namespace should not be required, user should be able to just use _ but the tests query the api and use the namespace value [email protected]_dev to check

Also I had a package demo/ with action hello in my namespace, one of the tests failed to list actions, since doing a get on the action returns namespace:[email protected]_dev/demo NS plus package and the test always assume that there is no package present in namespace field.

This is all I recall, if we setup travis integration, meaning deploying openwhisk, and then running the tests using the guest ow key, it should reveal how to run the tests and what instructions we need to document for a contributor trying to run test locally before sending a PR.

from openwhisk-client-js.

Did you see these instructions?
https://github.com/openwhisk/openwhisk-client-js/tree/master/test/integration

I've been using a "test" namespace with the assets in to isolate the results.

Having the integration tests run using Travis is a good idea. If someone has time to look at this, it would be great.

from openwhisk-client-js.

No I didn't saw them 🤦‍♂️

But they are manual steps we should not have any manual steps
And user should not set namespace value we should use _ and then use the ok to get the namespace value

from openwhisk-client-js.

We should put a note in the README.md with a link to that document.

from openwhisk-client-js.

Resolved by #59

from openwhisk-client-js.