Fix parsing of lists with relativedelta objects.

The relativedelta objects were rejected because we cannot compare them to
strings. Both `==` and `!=` results to `False`.

Instead we have to first test the type and then do the comparison.

Author: olii

pyhocon/config_parser.py

@@ -751,8 +751,16 @@ def postParse(self, instring, loc, token_list):
         :return:
         """
         cleaned_token_list = []
+        # Note that a token can be a duration value object:
+        # >>> relativedelta(hours = 1) == ''
+        # False
+        # >>> relativedelta(hours = 1) != ''
+        # False
+        # relativedelta.__eq__() raises NotImplemented if it is compared with
+        # a different object type so Python falls back to identity comparison.
+        # We cannot compare this object to a string object.
         for token in token_list:
-            if token == '':
+            if isinstance(token, str) and token == '':
                 # This is the case when there was a trailing comma in the list.
                 # The last token is just an empty string so we can safely ignore
                 # it.

tests/test_config_parser.py

@@ -148,6 +148,16 @@ def test_parse_string_with_duration_with_long_unit_name(self):
         )
         assert config['b'] == period(weeks=10)
 
+    def test_parse_with_list_mixed_types_with_durations_and_trailing_comma(self):
+        config = ConfigFactory.parse_string(
+            """
+            a: foo
+            b: [a, 1, 10 weeks, 5 minutes,]
+            c: bar
+            """
+        )
+        assert config['b'] == ['a', 1, period(weeks=10), period(minutes=5)]
+
     def test_parse_with_enclosing_square_bracket(self):
         config = ConfigFactory.parse_string("[1, 2, 3]")
         assert config == [1, 2, 3]