Fix operator parsing errors (#169)

Author: keelerm84

ldclient/operators.py

@@ -54,7 +54,7 @@ def _parse_time(input):
             log.warning("Couldn't parse timestamp:" + str(input) + " with message: " + str(e))
             return None
 
-    log.warning("Got unexpected type: " + type(input) + " with value: " + str(input) + " when attempting to parse time")
+    log.warning("Got unexpected type: " + str(type(input)) + " with value: " + str(input) + " when attempting to parse time")
     return None
 
 def _time_operator(u, c, fn):
@@ -69,6 +69,8 @@ def _parse_semver(input):
     try:
         VersionInfo.parse(input)
         return input
+    except TypeError:
+        return None
     except ValueError as e:
         try:
             input = _add_zero_version_component(input)

testing/test_operators.py

@@ -59,7 +59,11 @@
     [ "before", True, 1000,                        False ],  # wrong type
     [ "after",  "1970-01-01T00:00:02.500Z", 1000,  True ],
     [ "after",  "1970-01-01 00:00:02.500Z", 1000,  False ],  # malformed timestamp
+    [ "after", "1970-01-01T00:00:02+01:00", None, False ],
+    [ "after", None, "1970-01-01T00:00:02+01:00", False ],
     [ "before", "1970-01-01T00:00:02+01:00", 1000, True ],
+    [ "before", "1970-01-01T00:00:02+01:00", None, False ],
+    [ "before", None, "1970-01-01T00:00:02+01:00", False ],
     [ "before", -1000, 1000,                       True ],
     [ "after",  "1970-01-01T00:00:01.001Z", 1000,  True ],
     [ "after",  "1970-01-01T00:00:00-01:00", 1000, True ],
@@ -68,6 +72,8 @@
     [ "semVerEqual",       "2.0.1", "2.0.1",    True ],
     [ "semVerEqual",       "2.0",   "2.0.0",    True ],
     [ "semVerEqual",       "2",     "2.0.0",    True ],
+    [ "semVerEqual",       2,     "2.0.0",    False ],
+    [ "semVerEqual",       "2.0.0",     2,    False ],
     [ "semVerEqual",       "2.0-rc1", "2.0.0-rc1", True ],
     [ "semVerLessThan",    "2.0.0", "2.0.1",    True ],
     [ "semVerLessThan",    "2.0",   "2.0.1",    True ],