break up tests

Author: jaredmixpanel

requirements-testing.txt

@@ -1,4 +1,4 @@
-pytest~=7.1.2
+pytest~=4.6.11
 responses~=0.13.3
 more-itertools==8.10.0 ; python_version=='3.5' # more-itertools added some f-strings after this.
-typing; python_version>='3.7' and python_version<'3.10' # To work around CI fail.
+typing; python_version>='3.4' and python_version<'3.5' # To work around CI fail.

test_mixpanel.py

@@ -30,7 +30,7 @@ def clear(self):
         self.log = []
 
 
-class TestMixpanel:
+class TestMixpanelBase:
     TOKEN = '12345'
 
     def setup_method(self, method):
@@ -39,6 +39,9 @@ def setup_method(self, method):
         self.mp._now = lambda: 1000.1
         self.mp._make_insert_id = lambda: "abcdefg"
 
+
+class TestMixpanelTracking(TestMixpanelBase):
+
     def test_track(self):
         self.mp.track('ID', 'button press', {'size': 'big', 'color': 'blue', '$insert_id': 'abc123'})
         assert self.consumer.log == [(
@@ -122,6 +125,9 @@ def test_track_meta(self):
             }
         )]
 
+
+class TestMixpanelPeople(TestMixpanelBase):
+
     def test_people_set(self):
         self.mp.people_set('amq', {'birth month': 'october', 'favorite color': 'purple'})
         assert self.consumer.log == [(
@@ -284,6 +290,26 @@ def test_people_set_created_date_datetime(self):
             }
         )]
 
+    def test_people_meta(self):
+        self.mp.people_set('amq', {'birth month': 'october', 'favorite color': 'purple'},
+                           meta={'$ip': 0, '$ignore_time': True})
+        assert self.consumer.log == [(
+            'people', {
+                '$time': self.mp._now(),
+                '$token': self.TOKEN,
+                '$distinct_id': 'amq',
+                '$set': {
+                    'birth month': 'october',
+                    'favorite color': 'purple',
+                },
+                '$ip': 0,
+                '$ignore_time': True,
+            }
+        )]
+
+
+class TestMixpanelIdentity(TestMixpanelBase):
+
     def test_alias(self):
         # More complicated since alias() forces a synchronous call.
 
@@ -333,22 +359,8 @@ def test_merge(self):
             ('my_good_api_key', 'my_secret'),
         )]
 
-    def test_people_meta(self):
-        self.mp.people_set('amq', {'birth month': 'october', 'favorite color': 'purple'},
-                           meta={'$ip': 0, '$ignore_time': True})
-        assert self.consumer.log == [(
-            'people', {
-                '$time': self.mp._now(),
-                '$token': self.TOKEN,
-                '$distinct_id': 'amq',
-                '$set': {
-                    'birth month': 'october',
-                    'favorite color': 'purple',
-                },
-                '$ip': 0,
-                '$ignore_time': True,
-            }
-        )]
+
+class TestMixpanelGroups(TestMixpanelBase):
 
     def test_group_set(self):
         self.mp.group_set('company', 'amq', {'birth month': 'october', 'favorite color': 'purple'})