Merge pull request #126 from romuald/gtiset-str

Fix GtidSet __str__ representation

Author: julien-duponchelle

pymysqlreplication/gtid.py

@@ -88,7 +88,7 @@ def __init__(self, gtid_set):
             self.gtids = [Gtid(x) for x in gtid_set.split(',')]
 
     def __str__(self):
-        return ','.join(repr(x) for x in self.gtids)
+        return ','.join(str(x) for x in self.gtids)
 
     def __repr__(self):
         return '<GtidSet "%s"' % ','.join(repr(x) for x in self.gtids)

pymysqlreplication/tests/test_basic.py

@@ -8,6 +8,7 @@
 
 from pymysqlreplication.tests import base
 from pymysqlreplication import BinLogStreamReader
+from pymysqlreplication.gtid import GtidSet
 from pymysqlreplication.event import *
 from pymysqlreplication.constants.BINLOG import *
 from pymysqlreplication.row_event import *
@@ -696,6 +697,15 @@ def test_position_gtid(self):
         self.assertEqual(event.query, 'CREATE TABLE test2 (id INT NOT NULL, data VARCHAR (50) NOT NULL, PRIMARY KEY (id))');
 
 
+class TestGtidRepresentation(unittest.TestCase):
+    def test_gtidset_representation(self):
+        set_repr = '57b70f4e-20d3-11e5-a393-4a63946f7eac:1-56,' \
+                   '4350f323-7565-4e59-8763-4b1b83a0ce0e:1-20'
+
+        myset = GtidSet(set_repr)
+        self.assertEqual(str(myset), set_repr)
+
+
 if __name__ == "__main__":
     import unittest
     unittest.main()