WIP

[ci.skip]

Author: Hyask

src/errors/cassie.py

@@ -421,21 +421,19 @@ def get_metadata_for_buckets(bucketids, release=None):
 def get_user_crashes(user_token: str, limit: int = 50, start=None):
     results = {}
     try:
-        query = UserOOPS.objects.filter(key=user_token.encode()).limit(limit)
+        query = UserOOPS.objects.filter(key=user_token.encode()).limit(limit).order_by("-column1")
 
         if start:
-            # Filter to get items greater than start
-            query = query.filter(column1__gt=start)
+            # Filter to get items lower than start (reverse order)
+            query = query.filter(column1__lt=start)
 
         for row in query:
             # Since we don't have timestamp directly, we'll use the column1 to compute it
             results[row.column1] = datetime_from_uuid1(UUID(row.column1))
     except DoesNotExist:
         return []
 
-    return [
-        (k, results[k]) for k in sorted(results.keys(), key=lambda x: results[x], reverse=True)
-    ]
+    return [(k, results[k]) for k in results.keys()]
 
 
 def get_average_crashes(field, release, days=7):

src/errors/manage.py

@@ -1,4 +1,4 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
 # from django.core.management import execute_from_command_line
 # import imp
 # try:

src/tests/test_cassie.py

@@ -334,7 +334,9 @@ def test_get_metadata_for_buckets(self, cassandra_data):
         assert isinstance(metadata, dict)
         assert len(metadata) == 2
         assert metadata["/usr/bin/already-bucketed:11:func1:main"]["Source"] == "already-bucketed"
-        assert metadata["/usr/bin/failed-retrace:11:failed_func:main"]["Source"] == "failed-retrace"
+        assert (
+            metadata["/usr/bin/failed-retrace:11:failed_func:main"]["Source"] == "failed-retrace"
+        )
 
     def test_get_metadata_for_buckets_empty(self, cassandra_data):
         """Test get_metadata_for_buckets returns empty dict for empty list"""
@@ -347,12 +349,18 @@ def test_get_user_crashes(self, cassandra_data):
         user_token = "imatestsystem"
         crashes = cassie.get_user_crashes(user_token, limit=5)
         assert isinstance(crashes, list)
-        # Should have some crashes
-        assert len(crashes) > 0
-        # Each item should be a tuple of (uuid_str, datetime)
+        assert len(crashes) == 5
         for uuid_str, crash_time in crashes:
             assert isinstance(uuid_str, str)
             assert isinstance(crash_time, datetime)
+        first_crash = crashes[0]
+        more_crashes = cassie.get_user_crashes(user_token, limit=5, start=first_crash[0])
+        print(crashes)
+        print(more_crashes)
+        assert len(crashes) == 5
+        assert crashes[1] == more_crashes[0]
+        assert crashes[2] == more_crashes[1]
+        assert more_crashes[-1] not in crashes
 
     def test_get_user_crashes_nonexistent(self, cassandra_data):
         """Test get_user_crashes returns empty list for non-existent user"""