Merge pull request #299 from m-aciek/git-authored-date

Use authored date instead of committed date as git file timestamp

tests/test_utils.py

@@ -2,7 +2,7 @@
 import time
 import unittest
 import six
-from mock import patch, MagicMock, mock_open
+from mock import patch, MagicMock, Mock, mock_open
 from urllib3.exceptions import SSLError
 
 from txclib import utils, exceptions
@@ -383,4 +383,13 @@ def test_git_timestamp_is_parsable(self):
             os.path.dirname(os.path.abspath(__file__))
         )
         parsed_ts = time.mktime(time.gmtime(epoch_ts))
-        self.assertIsNotNone(parsed_ts)
+        self.assertIsNotNone(parsed_ts)
+
+    def test_uses_authorized_date(self):
+        commit = Mock()
+        commit.authored_date = 1590969254
+        commit.committed_date = 1590970456
+        with patch('txclib.utils.git.Repo') as repo:
+            repo.return_value.iter_commits.return_value = [commit]
+            epoch_ts = utils.get_git_file_timestamp('any')
+        self.assertEqual(1590969254, epoch_ts)

txclib/utils.py

@@ -747,7 +747,7 @@ def get_git_file_timestamp(file_path):
         )
         if commits_touching_path:
             latest_commit = commits_touching_path[0]
-            latest_commit_ts = latest_commit.committed_date
+            latest_commit_ts = latest_commit.authored_date
             return latest_commit_ts
         else:
             return None