Thread suite_version and source through save_result; filter home page to legacy

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: mattip

codespeed/results.py

@@ -69,6 +69,8 @@ def save_result(data, update_repo=True):
             b.units_title = data["units_title"]
         if "lessisbetter" in data:
             b.lessisbetter = data["lessisbetter"]
+        if "source" in data:
+            b.source = data["source"]
         b.full_clean()
         b.save()
 
@@ -127,6 +129,7 @@ def save_result(data, update_repo=True):
     r.val_max = data.get('max')
     r.q1 = data.get('q1')
     r.q3 = data.get('q3')
+    r.suite_version = data.get('suite_version', '')
 
     r.full_clean()
     r.save()

codespeed/tests/test_views.py

@@ -161,6 +161,44 @@ def test_add_result_with_no_project(self):
         self.assertEqual(
             response.content.decode(), "Result data saved successfully")
 
+    def test_suite_version_is_saved(self):
+        """suite_version in the payload should be stored on the Result"""
+        modified_data = copy.deepcopy(self.data)
+        modified_data['suite_version'] = '2.3.1'
+        self.client.post(self.path, modified_data)
+        res = Result.objects.get(
+            revision__commitid='23',
+            benchmark__name='float',
+        )
+        self.assertEqual(res.suite_version, '2.3.1')
+
+    def test_suite_version_defaults_to_empty(self):
+        """Omitting suite_version should store an empty string"""
+        self.client.post(self.path, self.data)
+        res = Result.objects.get(
+            revision__commitid='23',
+            benchmark__name='float',
+        )
+        self.assertEqual(res.suite_version, '')
+
+    def test_source_set_on_new_benchmark(self):
+        """source in the payload should be set on an auto-created Benchmark"""
+        modified_data = copy.deepcopy(self.data)
+        modified_data['benchmark'] = 'newbench'
+        modified_data['source'] = 'pyperformance'
+        self.client.post(self.path, modified_data)
+        b = Benchmark.objects.get(name='newbench')
+        self.assertEqual(b.source, 'pyperformance')
+
+    def test_source_not_changed_on_existing_benchmark(self):
+        """source in the payload should not overwrite an existing Benchmark"""
+        self.client.post(self.path, self.data)
+        modified_data = copy.deepcopy(self.data)
+        modified_data['source'] = 'pyperformance'
+        self.client.post(self.path, modified_data)
+        b = Benchmark.objects.get(name='float')
+        self.assertEqual(b.source, 'legacy')
+
 
 @override_settings(ALLOW_ANONYMOUS_POST=True)
 class TestAddJSONResults(TestCase):

codespeed/views.py

@@ -114,7 +114,8 @@ def gethistoricaldata(request):
         rev0 = rev[0]
         resname = '{} {}'.format(b['executable'], rev0.tag)
         baseline_results.append((resname, Result.objects.filter(
-            executable=baseline_exe, revision=rev0, environment=env)))
+            executable=baseline_exe, revision=rev0, environment=env,
+            benchmark__source='legacy')))
         if not baseline_results[-1][1]:
             logger.error('Could not find results for {} rev="{}" env="{}"'.format(
                     baseline_exe, rev0, env))
@@ -135,8 +136,10 @@ def gethistoricaldata(request):
             ).exclude(tag="").order_by('date')
         all_taggedrevs += default_taggedrevs
         for rev in default_taggedrevs:
+            # Filter to legacy only; pyperformance history can get its own panel later
             res = Result.objects.filter(
-                executable=_default_exe, revision=rev, environment=env)
+                executable=_default_exe, revision=rev, environment=env,
+                benchmark__source='legacy')
             if not res:
                 logger.info("no results for '%s' '%s' '%s'" % (str(_default_exe), str(rev), str(env)))
                 continue
@@ -160,7 +163,8 @@ def gethistoricaldata(request):
         default_lastrev = None
     if default_lastrev is not None:
         default_results['latest'] = Result.objects.filter(
-            executable=default_exe, revision=default_lastrev, environment=env)
+            executable=default_exe, revision=default_lastrev, environment=env,
+            benchmark__source='legacy')
 
     # Collect data
     benchmarks = []