testcontainers/testcontainers-java

Add getMappedPort(Integer originalPort, InternetProtocol protocol)

Aperta

#554 aperta il 23 gen 2018

Vedi su GitHubGuida contributor
 (12 commenti) (0 reazioni) (0 assegnatari)Java (1588 fork)batch import
good first issueresolution/acknowledgedtype/feature

Metriche repository

Star
 (7535 stelle)
Metriche merge PR
 (Merge medio 2g 17h) (2 PR mergiate in 30 g)

Descrizione

It will just look so much nicer in my test than

genericContainer.getContainerInfo().getNetworkSettings().getPorts().getBindings().get(new ExposedPort(originalPort, protocol))[0].getHostPortSpec()

I think it has something to do with the law of demeter.

But in all fairness root cause is https://github.com/moby/moby/issues/4635

Guida contributor

Direzione di ricerca
Esamina la classe GenericContainer esistente e il suo metodo getContainerInfo() per capire come aggiungere un metodo getMappedPort che incapsula la catena di chiamate per ottenere la porta dell'host. Osserva come vengono utilizzati ExposedPort e InternetProtocol. Implementa un semplice metodo di delega.
Tech stack
java
Dominio
backendtesting qaapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaDocker
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75