[Proposal] Adding supporting docs for E2E test development under development guide
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 45/100
- Tipo di issue
- Documentazione
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- docker, go
- Ambito
- cli, documentation, testing
Direzione di ricerca
Inizia da cli/development.md e deployment.md, quindi esamina tests/e2e/standalone/uninstall_test.go e i commenti esistenti sulla configurazione E2E. Verifica il prerequisito make build, il comando go test --tags=e2e, il percorso richiesto del binario e le variabili DAPR_DASHBOARD_PINNED_VERSION e DAPR_RUNTIME_PINNED_VERSION. Il lavoro è completato quando la guida allo sviluppo spiega come compilare ed eseguire un singolo test E2E e perché sono necessarie queste variabili.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Describe the proposal
- Right now under the
cli/development.mdwe 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 NameOfTestwhich is different from how one runs unit tests. Additionally one has to build the project usingmake buildcommand before running the E2E test as it requires the binary to be indist/darwin_arch/releasefolder. Now this might be non-intuitive to a new contributor. - Also while contributing or making changes to the E2E test, I had to add some environment variables for example in my case I was working on
tests/e2e/standalone/uninstall_test.gowhich requires the addition of two environment variablesDAPR_DASHBOARD_PINNED_VERSIONandDAPR_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_VERSIONvariable 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_VERSIONor 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 the dist/darwin_arch/release folder to run dapr init. This did not automatically set the environment variables DAPR_DASHBOARD_PINNED_VERSION and DAPR_RUNTIME_PINNED_VERSION required to run the E2E test.
- Lingua principale
- Go
- Stelle
- 353
- Fork
- 220
- Merge medio
- 6g 16h
- PR unite (30g)
- 1
Preparare l'ambiente
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di dapr/cli
-
kind/bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
dapr/cli#1700 · 1 commento · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
good first issue kind/proposal
Difficoltà 5/5 Più di una settimana Idoneità per principianti 42/100
I maintainer di solito rispondono entro 1 giorno
-
good first issue kind/proposal
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
I maintainer di solito rispondono entro 1 giorno
-
Docs (Automation) - Dapr CLIForse di nuovo libera @mikeee l’ha presa 558 giorni fa e non c’è nessuna pull request aperta. ApertaP1 pinned
dapr/cli#1507 · 3 commenti · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
-
Update the CLI install script release sourceForse di nuovo libera @mikeee l’ha presa 732 giorni fa e non c’è nessuna pull request aperta. Apertakind/bug pinned
dapr/cli#1454 · 8 commenti · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
Issue simili
-
bug needs triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
netdata/netdata#24062 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
meshery/meshery#22119 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
automation documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno