Fixed tests

ros2java-alfred · Jul 14, 2017 · 156253a · 156253a
1 parent 3c27f7a
commit 156253a
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 11 deletions.
diff --git a/rcljava/src/main/java/org/ros2/rcljava/RCLJava.java b/rcljava/src/main/java/org/ros2/rcljava/RCLJava.java
@@ -609,8 +609,7 @@ public static long convertQoSProfileToHandle(final QoSProfile qosProfile) {
       int durability = qosProfile.getDurability().getValue();
       boolean avoidRos = qosProfile.getAvoidRosNamespaceConventions();
 
-      RCLJava.logger.debug("Convert QosProfile...");
-
+      // RCLJava.logger.debug("Convert QosProfile...");
       return RCLJava.nativeConvertQoSProfileToHandle(history, depth, reliability, durability, avoidRos);
     }
 

diff --git a/rcljava/src/main/java/org/ros2/rcljava/node/NativeNode.java b/rcljava/src/main/java/org/ros2/rcljava/node/NativeNode.java
@@ -213,7 +213,7 @@ public NativeNode(final String defaultName) {
      * @param defaultName name of node.
      */
     public NativeNode(final String namespace, final String defaultName, final String[] args) {
-        NativeNode.logger.debug("Create Node stack : " + defaultName);
+        NativeNode.logger.debug("Create Node stack...");
 
         if (!RCLJava.isInitialized()) {
             throw new NotInitializedException();
@@ -256,7 +256,7 @@ public NativeNode(final String namespace, final String defaultName, final String
         if (nodeHandle==0) throw new NullPointerException("Node Handle is not define !");
         this.nodeHandle     = nodeHandle;
 
-        NativeNode.logger.debug("Create Node stack : " + GraphName.getFullName(this.nameSpace, this.name));
+        NativeNode.logger.debug("Created Node stack : " + GraphName.getFullName(this.nameSpace, this.name));
 
         if (args != null) {
             for (String arg : args) {

diff --git a/rcljava/src/test/java/org/ros2/rcljava/MessageGeometryTest.java b/rcljava/src/test/java/org/ros2/rcljava/MessageGeometryTest.java
@@ -132,13 +132,13 @@ public final void testPubPoint() throws Exception {
 
         Point msg = new Point();
         msg.setX(10.0d);
-        msg.setX(20.0d);
-        msg.setX(30.0d);
+        msg.setY(20.0d);
+        msg.setZ(30.0d);
 
         Point value = this.pubSubTest(msg);
         assertEquals(10.0d, value.getX(), 0.1d);
-        assertEquals(10.0d, value.getY(), 0.1d);
-        assertEquals(10.0d, value.getZ(), 0.1d);
+        assertEquals(20.0d, value.getY(), 0.1d);
+        assertEquals(30.0d, value.getZ(), 0.1d);
     }
 
     @Test
@@ -183,9 +183,9 @@ public final void testPubPose() throws Exception {
         msg.getOrientation().setX(20.0d);
         msg.getOrientation().setY(30.0d);
         msg.getOrientation().setZ(40.0d);
-        msg.getPosition().setX(50.0D);
-        msg.getPosition().setX(60.0D);
-        msg.getPosition().setX(70.0D);
+        msg.getPosition().setX(50.0d);
+        msg.getPosition().setY(60.0d);
+        msg.getPosition().setZ(70.0d);
 
         Pose value = this.pubSubTest(msg);
         assertEquals(10.0d, value.getOrientation().getW(), 0.1d);