opendev/gerrit
Code Issues Proposed changes

ElasticContainer: Include cause in AssumptionViolatedException

Browse Source 
Providing the cause will make failure logs more useful.

Change-Id: I08e0688fd7cb5c8e367aa6cb5c987397b693d207
This commit is contained in:
David Pursehouse
2018-05-25 16:03:52 +09:00
parent ba6fc47f5f
commit 7f87063bff
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-elasticsearch/src/main/java/com/google/gerrit/elasticsearch/testing/ElasticContainer.java
View File

@@ -34,7 +34,7 @@ public class ElasticContainer<SELF extends ElasticContainer<SELF>> extends Gener
container.start(); container.start();
return container; return container;
} catch (Throwable t) { } catch (Throwable t) {
throw new AssumptionViolatedException("Unable to start container[might be docker related]"); throw new AssumptionViolatedException("Unable to start container", t);
} }
} }