fix(#152): match issued_at default to naive column schema

sqlalchemy_history/transaction.py

@@ -16,7 +16,9 @@ def compile_big_integer(element, compiler, **kw):
 
 
 class TransactionBase:
-    issued_at = sa.Column(sa.DateTime, default=lambda: datetime.datetime.now(datetime.timezone.utc))
+    issued_at = sa.Column(
+        sa.DateTime, default=lambda: datetime.datetime.now(datetime.timezone.utc).replace(tzinfo=None)
+    )
 
     @property
     def entity_names(self):

tests/relationships/test_many_to_many_relations.py

@@ -34,7 +34,7 @@ class Article(self.Model):
                 sa.DateTime,
                 nullable=False,
                 server_default=sa.func.current_timestamp(),
-                default=lambda: datetime.datetime.now(datetime.timezone.utc),
+                default=lambda: datetime.datetime.now(datetime.timezone.utc).replace(tzinfo=None),
             ),
         )
 

tests/reported_bugs/test_bug_27_datetime_insertion_issue.py

@@ -19,7 +19,7 @@ def create_models(self):
                 sa.DateTime,
                 nullable=False,
                 server_default=sa.func.current_timestamp(),
-                default=lambda: datetime.datetime.now(datetime.timezone.utc),
+                default=lambda: datetime.datetime.now(datetime.timezone.utc).replace(tzinfo=None),
             ),
         )
 

tests/schema/test_update_end_transaction_id.py

@@ -24,7 +24,7 @@ def create_models(self):
                 sa.DateTime,
                 nullable=False,
                 server_default=sa.func.current_timestamp(),
-                default=lambda: datetime.datetime.now(datetime.timezone.utc),
+                default=lambda: datetime.datetime.now(datetime.timezone.utc).replace(tzinfo=None),
             ),
         )
 

tests/test_hybrid_property.py

@@ -18,7 +18,9 @@ class Article(self.Model):
             name = sa.Column(sa.Unicode(255), nullable=False)
             content = sa.Column(sa.UnicodeText)
             description = sa.Column(sa.UnicodeText)
-            publish = sa.Column(sa.DateTime, default=lambda: datetime.datetime.now(datetime.timezone.utc))
+            publish = sa.Column(
+                sa.DateTime, default=lambda: datetime.datetime.now(datetime.timezone.utc).replace(tzinfo=None)
+            )
 
             author_id = sa.Column(sa.Integer, sa.ForeignKey("article_author.id"), nullable=False)
 

tests/test_transaction.py

@@ -1,3 +1,4 @@
+import datetime
 import os
 import time
 
@@ -51,6 +52,13 @@ def test_transaction_issued_at(self):
         self.session.commit()
         assert self.article.versions[0].transaction.issued_at != self.article.versions[1].transaction.issued_at
 
+    def test_transaction_issued_at_is_naive(self):
+        uow = versioning_manager.unit_of_work(self.session)
+        tx = uow.create_transaction(self.session)
+
+        assert isinstance(tx.issued_at, datetime.datetime)
+        assert tx.issued_at.tzinfo is None
+
 
 # Check that the tests pass without TransactionChangesPlugin
 class TestTransactionWithoutChangesPlugin(TestTransaction):

tests/utils/test_parent_table.py

@@ -21,7 +21,7 @@ def create_models(self):
                 sa.DateTime,
                 nullable=False,
                 server_default=sa.func.current_timestamp(),
-                default=lambda: datetime.datetime.now(datetime.timezone.utc),
+                default=lambda: datetime.datetime.now(datetime.timezone.utc).replace(tzinfo=None),
             ),
         )
 

tests/utils/test_version_table.py

@@ -22,7 +22,7 @@ def create_models(self):
                 sa.DateTime,
                 nullable=False,
                 server_default=sa.func.current_timestamp(),
-                default=lambda: datetime.datetime.now(datetime.timezone.utc),
+                default=lambda: datetime.datetime.now(datetime.timezone.utc).replace(tzinfo=None),
             ),
         )