[Proposal] Adding supporting docs for E2E test development under development guide #1313
Labels
good first issue
Good for newcomers
kind/documentation
Improvements or additions to documentation
size/XS
2 days work
triaged/resolved
The issue has been triaged
Describe the proposal
cli/development.md
we do not have a supporting guide to run the E2E test. This includes maybe the command used to run the individual test because to run E2E tests one needs to add --tags=e2e while running the command and it looks something likego test --tags=e2e ./test/e2e/... -run NameOfTest
which is different from how one runs unit tests. Additionally one has to build the project usingmake build
command before running the E2E test as it requires the binary to be indist/darwin_arch/release
folder. Now this might be non-intuitive to a new contributor.tests/e2e/standalone/uninstall_test.go
which requires the addition of two environment variablesDAPR_DASHBOARD_PINNED_VERSION
andDAPR_RUNTIME_PINNED_VERSION
. Now this was discovered while dry-running the code to navigate to a function where on the comment it is written that we need to add these variables as environment variables, otherwise, the test run fails. Although I do understand that the error message while tests failed was accurate in describing where the issue is and it clearly mentioned thatDAPR_RUNTIME_PINNED_VERSION
variable was not set. However, for someone new to the code base it would not make sense as to what the value should be of the given variableDAPR_RUNTIME_PINNED_VERSION
or why it needs to be added.The proposal is to add some supporting docs at least with basic details under
deployment.md
or at some other appropriate location.My Setup
I was having docker installed on Mac machine. I built the code using
make build
and used the binary in thedist/darwin_arch/release
folder to rundapr init
. This did not automatically set the environment variablesDAPR_DASHBOARD_PINNED_VERSION
andDAPR_RUNTIME_PINNED_VERSION
required to run the E2E test.The text was updated successfully, but these errors were encountered: