plotly / plotly/plotly.py

test_utils.py uninstalls numpy

Aperta
#4,852 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

P2 testing
Lingua principale
Python
Stelle
18.8k
Fork
2.8k
Merge medio
16h 26m
PR unite (30g)
21

Descrizione

In #4622, I encountered a very odd recursion error, and the solution was to move the test that was failing to the top of the file. This was because the test above it was uninstalling numpy. We should investigate why this test is even doing that and potentially change the approach to prevent similar issues.

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Leggi packages/python/plotly/plotly/tests/test_optional/test_utils/test_utils.py intorno alle righe 453–465 e confrontalo con il test spostato in PR #4622. Determina perché il test disinstalla numpy, quindi verifica che l’approccio rivisto prevenga l’errore di ricorsione e non interferisca con gli altri test opzionali.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
numpy, python
Ambito
testing-qa
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
38/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.