Support for timezone conversion

Author: Val Brodsky

labelbox/utils.py

@@ -86,7 +86,7 @@ def format_iso_datetime(dt: datetime.datetime) -> str:
     Formats a datetime object into the format: 2011-11-04T00:05:23Z
     Note that datetime.isoformat() outputs 2011-11-04T00:05:23+00:00
     """
-    return dt.strftime(ISO_DATETIME_FORMAT)
+    return dt.astimezone(datetime.timezone.utc).strftime(ISO_DATETIME_FORMAT)
 
 
 def format_iso_from_string(date_string: str) -> datetime.datetime:

tests/unit/test_utils.py

@@ -2,15 +2,11 @@
 from labelbox.utils import format_iso_datetime, format_iso_from_string
 
 
-@pytest.mark.parametrize(
-    'datetime_str, expected_datetime_str',
-    [
-        ('2011-11-04T00:05:23Z', '2011-11-04T00:05:23Z'),
-        ('2011-11-04T00:05:23+00:00', '2011-11-04T00:05:23Z'),
-        ('2011-11-04T00:05:23+05:00', '2011-11-04T00:05:23Z'
-        ),  # NOTE: The current implementation is not converting from timezone other then UTC. This how it has been working prior to my change.
-        ('2011-11-04T00:05:23', '2011-11-04T00:05:23Z')
-    ])
+@pytest.mark.parametrize('datetime_str, expected_datetime_str',
+                         [('2011-11-04T00:05:23Z', '2011-11-04T00:05:23Z'),
+                          ('2011-11-04T00:05:23+00:00', '2011-11-04T00:05:23Z'),
+                          ('2011-11-04T00:05:23+05:00', '2011-11-03T19:05:23Z'),
+                          ('2011-11-04T00:05:23', '2011-11-04T00:05:23Z')])
 def test_datetime_parsing(datetime_str, expected_datetime_str):
     # NOTE I would normally not take 'expected' using another function from sdk code, but in this case this is exactly the usage in _validate_parse_datetime
     assert format_iso_datetime(