testcontainers / testcontainers/testcontainers-java
Log the exception before swallowing it
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
Here the exception is swallowed without any logs. That makes debugging confusing. In my case, the test failed with the error "Docker must be present blah blah", but the actual issue was disk space problem.
Another solution could be to change the level of log in DockerClientFactory#checkDiskSpace to warning.
Let me know if you need the PR.
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 in core/src/main/java/org/testcontainers/DockerClientFactory.java beim verlinkten Code um Zeile 133 und untersuche anschließend DockerClientFactory#checkDiskSpace sowie die Logging-Konventionen in der Nähe. Bestätige, wie die Exception verschluckt wird und welcher Log-Level derzeit verwendet wird. Als erledigt gilt die Änderung, wenn der Fehler bei der Überprüfung des Speicherplatzes in den Logs sichtbar ist, statt nur die irreführende Meldung zu erzeugen, dass Docker vorhanden ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- docker, java
- Bereich
- infrastructure, testing
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 38/100