#57 - Added tests for the exception message.

Original pull request: #97.

Author: schauder

src/main/java/org/springframework/data/r2dbc/support/AbstractFallbackR2dbcExceptionTranslator.java

@@ -111,6 +111,11 @@ public DataAccessException translate(String task, @Nullable String sql, R2dbcExc
 	 * @return the message {@code String} to use.
 	 */
 	protected String buildMessage(String task, @Nullable String sql, R2dbcException ex) {
-		return task + "; " + (sql != null ? "SQL [" + sql + "]; " : "") + ex.getMessage();
+
+		return task + "; " + //
+				(sql != null //
+						? "SQL [" + sql + "]; " //
+						: "" //
+				) + ex.getMessage();
 	}
 }

src/test/java/org/springframework/data/r2dbc/support/R2dbcExceptionSubclassTranslatorUnitTests.java

@@ -27,7 +27,6 @@
 import io.r2dbc.spi.R2dbcTransientResourceException;
 
 import org.junit.Test;
-
 import org.springframework.dao.ConcurrencyFailureException;
 import org.springframework.dao.DataAccessResourceFailureException;
 import org.springframework.dao.DataIntegrityViolationException;
@@ -49,9 +48,10 @@ public class R2dbcExceptionSubclassTranslatorUnitTests {
 	@Test // gh-57
 	public void shouldTranslateTransientResourceException() {
 
-		Exception exception = translator.translate("", "", new R2dbcTransientResourceException());
+		Exception exception = translator.translate("", "", new R2dbcTransientResourceException(""));
 
-		assertThat(exception).isInstanceOf(TransientDataAccessResourceException.class);
+		assertThat(exception)
+				.isInstanceOf(TransientDataAccessResourceException.class);
 	}
 
 	@Test // gh-57
@@ -110,5 +110,35 @@ public void shouldTranslateBadSqlGrammarException() {
 		assertThat(exception).isInstanceOf(BadSqlGrammarException.class);
 	}
 
+	@Test // gh-57
+	public void messageGeneration() {
+
+		Exception exception = translator.translate("TASK", "SOME-SQL", new R2dbcTransientResourceException("MESSAGE"));
+
+		assertThat(exception) //
+				.isInstanceOf(TransientDataAccessResourceException.class) //
+				.hasMessage("TASK; SQL [SOME-SQL]; MESSAGE; nested exception is io.r2dbc.spi.R2dbcTransientResourceException: MESSAGE");
+	}
+
+	@Test // gh-57
+	public void messageGenerationNullSQL() {
+
+		Exception exception = translator.translate("TASK", null, new R2dbcTransientResourceException("MESSAGE"));
+
+		assertThat(exception) //
+				.isInstanceOf(TransientDataAccessResourceException.class) //
+				.hasMessage("TASK; MESSAGE; nested exception is io.r2dbc.spi.R2dbcTransientResourceException: MESSAGE");
+	}
+
+	@Test // gh-57
+	public void messageGenerationNullMessage() {
+
+		Exception exception = translator.translate("TASK", "SOME-SQL", new R2dbcTransientResourceException());
+
+		assertThat(exception) //
+				.isInstanceOf(TransientDataAccessResourceException.class) //
+				.hasMessage("TASK; SQL [SOME-SQL]; null; nested exception is io.r2dbc.spi.R2dbcTransientResourceException");
+	}
+
 	private static class MyTransientExceptions extends R2dbcException {}
 }