test

statsig/statsig_user.py

@@ -29,7 +29,7 @@ class StatsigUser:
 
     def __post_init__(self):
         # ensure there is a user id or at least a custom ID, empty dict evaluates to false in python so we can use "not" operator to check
-        if (self.user_id is None or self.user_id == "") and not self.custom_ids:
+        if (self.user_id is None or self.user_id == "") and (self.custom_ids is None or not self.custom_ids):
             raise ValueError(
                 'user_id or at least a custom ID is required: learn more https://docs.statsig.com/messages/serverRequiredUserID')
 

tests/test_bad_input.py

@@ -20,19 +20,17 @@ def test_bad_key(self):
     def test_no_user_id(self):
         with self.assertRaises(ValueError) as context:
             StatsigUser("")
-
         self.assertTrue('user_id' in str(context.exception))
 
         with self.assertRaises(ValueError) as context:
             StatsigUser(None)
-
         self.assertTrue('user_id' in str(context.exception))
 
         with self.assertRaises(ValueError) as context:
             StatsigUser(None, custom_ids=dict())
-
         self.assertTrue('user_id' in str(context.exception))
 
+        u = StatsigUser(None, custom_ids=dict())
         user = StatsigUser(None, custom_ids=dict(stableID='123'))
         self.assertFalse(user.user_id)
         self.assertTrue(user.custom_ids)