Merge pull request quiltdata#79 from quiltdata/log2

quilt log prints reverse chrono, handles timestamps instead of datetime

quilt/test/test_command.py

@@ -2,12 +2,12 @@
 Tests for commands.
 """
 
-from datetime import datetime
 import json
 import os
 import pytest
 import requests
 import responses
+import time
 
 from six import assertRaisesRegex
 
@@ -149,7 +149,7 @@ def _mock_logs_list(self, owner, package, pkg_hash):
         logs_url = "%s/api/log/%s/%s/" % (command.QUILT_PKG_URL, owner, package)
         resp = dict(logs=[dict(
             hash=pkg_hash,
-            created=str(datetime.now()),
+            created=time.time(),
             author=owner)])
         print("MOCKING URL=%s" % logs_url)
         self.requests_mock.add(responses.GET, logs_url, json.dumps(resp))

quilt/tools/command.py

@@ -6,6 +6,7 @@
 from __future__ import print_function
 from builtins import input
 import argparse
+from datetime import datetime
 import json
 import os
 import stat
@@ -203,10 +204,11 @@ def log(session, package):
 
     format_str = "%-64s %-19s %s"
 
-    print(format_str % ("Hash", "Created", "Author"))
-    for entry in response.json()['logs']:
-        # TODO: convert "created" to local time.
-        print(format_str % (entry['hash'], entry['created'], entry['author']))
+    print(format_str % ("Hash", "Pushed", "Author"))
+    for entry in reversed(response.json()['logs']):
+        ugly = datetime.fromtimestamp(entry['created'])
+        nice = ugly.strftime("%Y-%m-%d %H:%M:%S")
+        print(format_str % (entry['hash'], nice, entry['author']))
 
 def push(session, package):
     """