Merge pull request #130 from dchrzascik/issue-125-exception-message

Issue #125 Passing exception message to ArangoDBException

Author: mpv1989

src/main/java/com/arangodb/ArangoDBException.java

@@ -49,6 +49,10 @@ public String getErrorMessage() {
 		return entity != null ? entity.getErrorMessage() : null;
 	}
 
+	public String getException() {
+		return entity != null ? entity.getException() : null;
+	}
+
 	public int getResponseCode() {
 		return entity != null ? entity.getCode() : null;
 	}

src/main/java/com/arangodb/entity/ErrorEntity.java

@@ -27,6 +27,7 @@
 public class ErrorEntity {
 
 	private String errorMessage;
+	private String exception;
 	private int code;
 	private int errorNum;
 
@@ -41,6 +42,14 @@ public String getErrorMessage() {
 		return errorMessage;
 	}
 
+	/**
+	 * @return the exception message, passed when transaction fails
+	 * @return
+	 */
+	public String getException() {
+		return exception;
+	}
+
 	/**
 	 * @return the status code
 	 */

src/test/java/com/arangodb/ArangoDatabaseTest.java

@@ -28,6 +28,7 @@
 import static org.hamcrest.Matchers.is;
 import static org.hamcrest.Matchers.not;
 import static org.junit.Assert.assertThat;
+import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.fail;
 
 import java.io.IOException;
@@ -1060,6 +1061,20 @@ public void getDocument() {
 		}
 	}
 
+	@Test
+	public void shouldIncludeExceptionMessage() {
+		final String exceptionMessage = "My error context";
+		final String action = "function (params) {"
+				+ "throw '" + exceptionMessage + "';"
+				+ "}";
+		try {
+			db.transaction(action, VPackSlice.class, null);
+			fail();
+		} catch (final ArangoDBException e) {
+			assertTrue(e.getException().contains(exceptionMessage));
+		}
+	}
+
 	@Test(expected = ArangoDBException.class)
 	public void getDocumentWrongId() {
 		db.getDocument("123", BaseDocument.class);