ARTEMIS-3363: add a couple of assertions for completeness (+fix depre…

…cation warning)

(cherry picked from commit ea8fe11)

artemis-core-client/src/test/java/org/apache/activemq/artemis/api/core/TransportConfigurationTest.java

@@ -29,6 +29,7 @@
 
 import static org.hamcrest.CoreMatchers.containsString;
 import static org.hamcrest.CoreMatchers.not;
+import static org.hamcrest.MatcherAssert.assertThat;
 
 public class TransportConfigurationTest {
 
@@ -82,7 +83,9 @@ public void testExtraParamsEquals() {
 
       Assert.assertNotEquals(new TransportConfiguration(className, params, name, null), new TransportConfiguration(className, params, name, extraParams));
       Assert.assertNotEquals(new TransportConfiguration(className, params, name, Collections.emptyMap()), new TransportConfiguration(className, params, name, extraParams));
-      Assert.assertNotEquals(new TransportConfiguration(className, params, name, extraParams), new TransportConfiguration(className, params, name, Collections.singletonMap("key", "too")));
+      Assert.assertNotEquals(new TransportConfiguration(className, params, name, extraParams), new TransportConfiguration(className, params, name, Collections.singletonMap("key", "other")));
+      Assert.assertNotEquals(new TransportConfiguration(className, params, name, extraParams), new TransportConfiguration(className, params, name, null));
+      Assert.assertNotEquals(new TransportConfiguration(className, params, name, extraParams), new TransportConfiguration(className, params, name, Collections.emptyMap()));
    }
 
    @Test
@@ -93,7 +96,7 @@ public void testToStringObfuscatesPasswords() {
 
       TransportConfiguration configuration = new TransportConfiguration("SomeClass", params, null);
 
-      Assert.assertThat(configuration.toString(), not(containsString("secret_password")));
+      assertThat(configuration.toString(), not(containsString("secret_password")));
    }
 
    @Test