Integration Test Error about auth0-cli HOT 5 CLOSED

Hey @m3talsmith 👋🏻

Thanks for taking the time to raise this! We can definitely improve the contributing guidelines and the test setup. One important thing we're missing there is the fact that we require a machine to machine application already created on the test tenant that has been authorized to request access tokens for the Management API and has all the required permissions. Only then we can leverage the client's credentials to authenticate through the Auth0 CLI.

We could also take this opportunity to uniform the env var naming to AUTH0_DOMAIN, AUTH0_CLIENT_ID and AUTH0_CLIENT_SECRET so it's aligned with some of our other repos, for example: https://github.com/auth0/terraform-provider-auth0/blob/main/CONTRIBUTING.md#running-the-tests. This would make switching projects and running tests easier.

A further improvement would be the addition of another make command that would only run a single/subset of the integration tests, for example: commander test --verbose --filter "016" test/integration/users-test-cases.yaml, commander being the utility we use for our integration tests.

These combined would make for a much more improved CONTRIBUTING guidelines. Happy to know your thoughts on this and discuss further.

from auth0-cli.

The error is spawned from the suite run script.

This may be able to be fixed by exporting those variables in the makefile before running the script.

from auth0-cli.

I appreciate the sensitivity of these env variables. Perhaps we could change the CONTRIBUTING doc to reflect steps on setting them personally, avoiding the need to generate them.

from auth0-cli.

Thanks @sergiught ! I think we are both on the same page here. There were some minor Makefile changes made to pass the auth0 variables to the script. I'm looking for a better way to do that: definitely open to suggestions.

I'll keep PR #902 open to have code specific comments there.

I especially like the idea of having a filter on commander as that would allow us to have a minimum set of integration tests for a quick pass, and still allow us to run the full suite at a future time.

from auth0-cli.

Closing this down now that we have #902 merged in 💪🏻

from auth0-cli.