Shorten field name

mozilla · Aug 29, 2023 · c153295 · c153295
1 parent b76faec
commit c153295
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 17 deletions.
diff --git a/tests/webapp/api/test_perfcompare_api.py b/tests/webapp/api/test_perfcompare_api.py
@@ -112,8 +112,8 @@ def test_perfcompare_results_against_no_base(
             'header_name': response['header_name'],
             'base_repository_name': base_sig.repository.name,
             'new_repository_name': new_sig.repository.name,
-            'base_application': 'firefox',
-            'new_application': 'geckoview',
+            'base_app': 'firefox',
+            'new_app': 'geckoview',
             'is_complete': response['is_complete'],
             'base_measurement_unit': base_sig.measurement_unit,
             'new_measurement_unit': new_sig.measurement_unit,
@@ -267,8 +267,8 @@ def test_perfcompare_results_with_only_one_run_and_diff_repo(
             'header_name': response['header_name'],
             'base_repository_name': base_sig.repository.name,
             'new_repository_name': new_sig.repository.name,
-            'base_application': 'firefox',
-            'new_application': 'geckoview',
+            'base_app': 'firefox',
+            'new_app': 'geckoview',
             'is_complete': response['is_complete'],
             'base_measurement_unit': base_sig.measurement_unit,
             'new_measurement_unit': new_sig.measurement_unit,
@@ -425,8 +425,8 @@ def test_perfcompare_results_multiple_runs(
             'header_name': first_row['header_name'],
             'base_repository_name': sig1.repository.name,
             'new_repository_name': sig2.repository.name,
-            'base_application': '',
-            'new_application': '',
+            'base_app': '',
+            'new_app': '',
             'is_complete': first_row['is_complete'],
             'base_measurement_unit': sig1.measurement_unit,
             'new_measurement_unit': sig2.measurement_unit,
@@ -471,8 +471,8 @@ def test_perfcompare_results_multiple_runs(
             'header_name': second_row['header_name'],
             'base_repository_name': sig3.repository.name,
             'new_repository_name': sig4.repository.name,
-            'base_application': '',
-            'new_application': '',
+            'base_app': '',
+            'new_app': '',
             'is_complete': second_row['is_complete'],
             'base_measurement_unit': sig3.measurement_unit,
             'new_measurement_unit': sig4.measurement_unit,

diff --git a/treeherder/webapp/api/performance_data.py b/treeherder/webapp/api/performance_data.py
@@ -919,9 +919,6 @@ def list(self, request):
                     new_is_better, base_avg_value, new_avg_value, confidence
                 )
 
-                base_application = base_sig.get('application', '')
-                new_application = new_sig.get('application', '')
-
                 is_improvement = class_name == 'success'
                 is_regression = class_name == 'danger'
                 is_meaningful = class_name == ''
@@ -931,8 +928,8 @@ def list(self, request):
                     'new_rev': new_rev,
                     'header_name': header,
                     'platform': platform,
-                    'base_application': base_application,
-                    'new_application': new_application,
+                    'base_app': base_sig.get('application', ''),
+                    'new_app': new_sig.get('application', ''),
                     'suite': base_sig.get('suite', ''),  # same suite for base_result and new_result
                     'test': base_sig.get('test', ''),  # same test for base_result and new_result
                     'is_complete': is_complete,

diff --git a/treeherder/webapp/api/performance_serializers.py b/treeherder/webapp/api/performance_serializers.py
@@ -443,11 +443,11 @@ def validate(self, data):
 class PerfCompareResultsSerializer(serializers.ModelSerializer):
     base_rev = serializers.CharField()
     new_rev = serializers.CharField()
-    base_application = serializers.CharField(
+    base_app = serializers.CharField(
         max_length=10,
         default='',
     )
-    new_application = serializers.CharField(
+    new_app = serializers.CharField(
         max_length=10,
         default='',
     )
@@ -497,8 +497,8 @@ class Meta:
         fields = [
             'base_rev',
             'new_rev',
-            'base_application',
-            'new_application',
+            'base_app',
+            'new_app',
             'framework_id',
             'platform',
             'suite',