simplify further, and prevent NPE in String constructor on failure

Author: rade

test/src/com/rabbitmq/client/test/functional/PerQueueTTL.java

@@ -127,8 +127,8 @@ public void testPublishAndGetWithExpiry() throws Exception {
 
         publish(MSG[2]);
 
-        assertEquals(MSG[1], new String(get()));
-        assertEquals(MSG[2], new String(get()));
+        assertEquals(MSG[1], get());
+        assertEquals(MSG[2], get());
 
     }
 
@@ -147,7 +147,7 @@ public void testTransactionalPublishWithGet() throws Exception {
         this.channel.txCommit();
         Thread.sleep(50);
 
-        assertEquals(MSG[0], new String(get()));
+        assertEquals(MSG[0], get());
         Thread.sleep(80);
 
         assertNull(get());
@@ -176,12 +176,9 @@ public void testExpiryWithRequeue() throws Exception {
     }
 
 
-    private byte[] get() throws IOException {
+    private String get() throws IOException {
         GetResponse response = basicGet(TTL_QUEUE_NAME);
-        if(response == null) {
-            return null;
-        }
-        return response.getBody();
+        return response == null ? null : new String(response.getBody());
     }
 
     private void publish(String msg) throws IOException {