Added test for comma subsecond separator and moved setting of tz_form…

…at down in file

arrow/parser.py

@@ -155,13 +155,6 @@ def parse_iso(self, datetime_string):
 
             time_parts = re.split(r"[\+\-Z]", time_string, 1, re.IGNORECASE)
 
-            is_basic_time_format = ":" not in time_parts[0]
-            tz_format = "Z"
-
-            # use 'ZZ' token instead since tz offset is present in non-basic format
-            if len(time_parts) == 2 and ":" in time_parts[1]:
-                tz_format = "ZZ"
-
             time_components = self._TIME_RE.match(time_parts[0])
 
             if time_components is None:
@@ -178,6 +171,13 @@ def parse_iso(self, datetime_string):
             has_seconds = seconds is not None
             has_subseconds = subseconds is not None
 
+            is_basic_time_format = ":" not in time_parts[0]
+            tz_format = "Z"
+
+            # use 'ZZ' token instead since tz offset is present in non-basic format
+            if has_tz and ":" in time_parts[1]:
+                tz_format = "ZZ"
+
             time_sep = "" if is_basic_time_format else ":"
 
             if has_subseconds:

tests/parser_tests.py

@@ -976,6 +976,12 @@ def test_parse_subsecond(self):
             self.parser.parse_iso("2013-01-01 12:30:45.987654"), self.expected
         )
 
+        # use comma as subsecond separator
+        self.expected = datetime(2013, 1, 1, 12, 30, 45, 987654)
+        self.assertEqual(
+            self.parser.parse_iso("2013-01-01 12:30:45,987654"), self.expected
+        )
+
     def test_gnu_date(self):
         """
         regression tests for parsing output from GNU date(1)