- apply repr() to server default string, #31

diff --git a/CHANGES b/CHANGES
index 5d6e7c22d2c149e02ad1ffc9287f3fd068570b67..421ffc720d648f0945d82ee8a0e000c85ee26fca 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -1,3 +1,9 @@
+0.3.5
+=====
+- [bug] Fixed issue whereby reflected server defaults
+  wouldn't be quoted correctly; uses repr() now.
+  #31
+
 0.3.4
 =====
 - [bug] Fixed command-line bug introduced by the

diff --git a/alembic/__init__.py b/alembic/__init__.py
index 81898ff5abe61da2108dddcfca89f8427cbb22db..e761d540346a2b344a928e2259ff0682cc40dc1d 100644 (file)
--- a/alembic/__init__.py
+++ b/alembic/__init__.py
@@ -1,6 +1,6 @@
 from os import path
 
-__version__ = '0.3.4'
+__version__ = '0.3.5'
 
 package_dir = path.abspath(path.dirname(__file__))
 

diff --git a/alembic/autogenerate.py b/alembic/autogenerate.py
index 1b0360435288cad9515e64b14b9c18618f113928..2b6f2d86320b8f111c893f8061c4333cdcd92141 100644 (file)
--- a/alembic/autogenerate.py
+++ b/alembic/autogenerate.py
@@ -497,7 +497,7 @@ def _render_server_default(default, autogen_context):
         # tests to pass until we figure out
         # WTF sqlite is doing
         default = re.sub(r"^'|'$", "", default)
-        return "'%s'" % default
+        return repr(default)
     else:
         return None
 

diff --git a/tests/test_autogenerate.py b/tests/test_autogenerate.py
index 5d4a06f0db26a8a6ea140d61354640ccabcd59a7..1b9e8fc1c91e9c7213f3b586990e399fe29d251e 100644 (file)
--- a/tests/test_autogenerate.py
+++ b/tests/test_autogenerate.py
@@ -418,6 +418,14 @@ class AutogenRenderTest(TestCase):
             "op.drop_column('foo', 'x')"
         )
 
+    def test_render_quoted_server_default(self):
+        eq_(
+            autogenerate._render_server_default(
+                "nextval('group_to_perm_group_to_perm_id_seq'::regclass)",
+                    self.autogen_context),
+            '"nextval(\'group_to_perm_group_to_perm_id_seq\'::regclass)"'
+        )
+
     def test_render_modify_type(self):
         eq_ignore_whitespace(
             autogenerate._modify_col(