testcontainers / testcontainers/testcontainers-java
Add getMappedPort(Integer originalPort, InternetProtocol protocol)
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Java
- Sterne
- 8.7k
- Forks
- 1.9k
- Ø Merge
- 2 T. 17 Std.
- Gemergte PRs (30 T.)
- 9
Beschreibung
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
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die GenericContainer API rund um getContainerInfo(), NetworkSettings, bindings und ExposedPort nachzuverfolgen. Füge den angeforderten Einstiegspunkt getMappedPort(Integer originalPort, InternetProtocol protocol) hinzu, damit Aufrufer den zugeordneten Host-Port abrufen können, ohne durch die verschachtelten Netzwerkobjekte navigieren zu müssen, und überprüfe anschließend das Verhalten mit den relevanten Tests für die Container-Netzwerkfunktionalität.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- docker, java
- Bereich
- testing
- Issue-Typ
- Feature
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Aktiv
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 74/100