Merge pull request #277 from compas-dev/fix-rosmsg-repr

Fix ROSmsg representation

Author: gonzalocasas

CHANGELOG.rst

@@ -16,6 +16,8 @@ Unreleased
 
 **Fixed**
 
+* Fix ``repr()`` of ``ROSmsg`` class
+
 **Deprecated**
 
 **Removed**

src/compas_fab/backends/ros/messages/std_msgs.py

@@ -32,7 +32,11 @@ def __str__(self):
         return str(self.msg)
 
     def __repr__(self):
-        return self.__str__
+        args = []
+        for key, value in self.__dict__.items():
+            args.append('{}={!r}'.format(key, value))
+
+        return '{}({})'.format(self.__class__.__name__, ', '.join(args))
 
 
 class Time(ROSmsg):

tests/backends/ros/messages/test_std_msgs.py

@@ -0,0 +1,20 @@
+from compas_fab.backends.ros.messages import ROSmsg
+from compas_fab.backends.ros.messages import Header
+from compas_fab.backends.ros.messages import Time
+
+
+def test_rosmsg_attributes():
+    r = ROSmsg(a=1, b=2)
+    assert r.a == 1
+    assert r.b == 2
+
+
+def test_rosmsg_str():
+    r = ROSmsg(a=1, b='2')
+    assert str(r) == str(dict(a=1, b='2'))
+
+
+def test_nested_repr():
+    t = Time(80, 20)
+    h = Header(seq=10, stamp=t, frame_id='/wow')
+    assert repr(h) == "Header(seq=10, stamp=Time(secs=80, nsecs=20), frame_id='/wow')"