No test coverage for ipaddr in HTTP.start
Nessuno ha ancora preso questa issue.
- Lingua principale
- Ruby
- Stelle
- 148
- Fork
- 95
- Merge medio
- 10h 54m
- PR unite (30g)
- 4
Descrizione
We use a code like this:
request = Net::HTTP::Get.new(uri.request_uri)
Net::HTTP.start(uri.hostname, uri.port, ipaddr: ip, use_ssl: true) do |http|
http.request(request)
end
What I have noticed is that it is difficult to see if ipaddr is actually used for the request... If there is a change in the future the ipaddr param may be just ignored and nobody will notice.
Furthermore I have noticed that in the code of this gem there is no test coverage for the ipaddr parameter passed to HTTP.start.
- Is there a more "solid" and "future-proof" solution to set the ipaddr in the above code?
- Can you add test coverage for this parameter so that it won't break in future versions?
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
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.
Direzione di ricerca
Inizia dal punto di ingresso HTTP.start e traccia il percorso con cui il suo argomento ipaddr raggiunge la connessione della richiesta. Aggiungi una copertura di test che dimostri che viene utilizzato l’ipaddr fornito, quindi esegui i test HTTP.start pertinenti e conferma che il comportamento rimanga coperto nelle modifiche future.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- ruby
- Ambito
- backend, testing-qa
- Tipo di issue
- Bug
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100