akka/akka-http

Bump h2spec external binary to newer versions

Open

#3534 aperta il 15 ott 2020

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Scala (598 fork)batch import
help wantednice-to-have (low-prio)t:http2

Metriche repository

Star
 (1311 star)
Metriche merge PR
 (Merge medio 1h 17m) (1 PR mergiata in 30 g)

Descrizione

In #1578 there was some initial work to update the build to use a newer version of https://github.com/summerwind/h2spec/. That work focused on the build.

I've just replaced the binary in ./akka-http2-support/target/h2spec_darwin_amd64/h2spec with the latest and ran H2SpecIntegrationSpec. Tests, as they are now, fail because the binary API has changed from:

https://github.com/summerwind/h2spec/tree/9b3d1a5078cb4003506bd8004eb1b01e3be2a3e7

to

https://github.com/summerwind/h2spec/#usage

The parameter -s: Section number on which to run the test. (Example: -s 6.1 -s 6.2) is not available in h2spec since version 2.0.0. As a consequence, the execution of H2SpecIntegrationSpec goes over tests we don't need taking (a lot) longer and reporting failures we would ignore anyway.

Guida contributor

Direzione di ricerca
Indaga le modifiche attuali della CLI h2spec tra le versioni 1.x e 2.x. La vecchia API utilizzava s per specificare le sezioni, che non è più disponibile. Aggiorna la configurazione del test H2SpecIntegrationSpec per utilizzare i nuovi argomenti CLI, probabilmente coinvolgendo flag diversi o un file di configurazione separato. Controlla la documentazione di utilizzo del repository h2spec per la sintassi corretta e adatta il test runner di conseguenza.
Tech stack
Nessuno
Dominio
testingbuild system
Tipo issue
Manutenzione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Basic understanding of HTTP/2Familiarity with h2spec tool
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
30