feat(nimbus): Add Sorting to the QA Runs table on the feature health page. (#13742)

Because

- We need to add sorting to the QA Runs table on the feature health page

This commit

- Adds sorting to the QA Runs Table
- Adds 2 new model fields to the experiment model to keep track of QA
runs and dates.

Fixes #13708

---------

Co-authored-by: Yashika Khurana <[email protected]>

Author: b4handjr

experimenter/experimenter/experiments/constants.py

@@ -453,6 +453,12 @@ def get_icon_info(status):
                 status, QA_STATUS_ICON_MAP[NimbusConstants.QAStatus.NOT_SET]
             )
 
+    class QATestType(models.TextChoices):
+        FULL = "FULL", "Full"
+        SMOKE = "SMOKE", "Smoke"
+        SELF = "SELF", "Self"
+        REGRESSION = "REGRESSION", "Regression"
+
     APPLICATION_CONFIGS = {
         Application.DESKTOP: APPLICATION_CONFIG_DESKTOP,
         Application.FENIX: APPLICATION_CONFIG_FENIX,

experimenter/experimenter/experiments/migrations/0296_nimbusexperiment_qa_run_date_and_more.py

@@ -0,0 +1,33 @@
+# Generated by Django 5.2.7 on 2025-10-23 19:11
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('experiments', '0295_nimbusexperiment_exclude_countries_and_more'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='nimbusexperiment',
+            name='qa_run_date',
+            field=models.DateField(blank=True, default=None, null=True, verbose_name='QA Run Date'),
+        ),
+        migrations.AddField(
+            model_name='nimbusexperiment',
+            name='qa_run_test_plan',
+            field=models.URLField(blank=True, default=None, max_length=500, null=True, verbose_name='QA Run Test Plan Link'),
+        ),
+        migrations.AddField(
+            model_name='nimbusexperiment',
+            name='qa_run_testrail_link',
+            field=models.URLField(blank=True, default=None, max_length=500, null=True, verbose_name='QA Run TestRail Link'),
+        ),
+        migrations.AddField(
+            model_name='nimbusexperiment',
+            name='qa_run_type',
+            field=models.CharField(blank=True, choices=[('FULL', 'Full'), ('SMOKE', 'Smoke'), ('SELF', 'Self'), ('REGRESSION', 'Regression')], default=None, max_length=255, null=True, verbose_name='QA Run Type'),
+        ),
+    ]

experimenter/experimenter/experiments/models.py

@@ -454,6 +454,29 @@ class NimbusExperiment(NimbusConstants, TargetingConstants, FilterMixin, models.
         default=list,
     )
     tags = models.ManyToManyField(Tag, blank=True, related_name="experiments")
+    qa_run_date = models.DateField("QA Run Date", blank=True, null=True, default=None)
+    qa_run_type = models.CharField(
+        "QA Run Type",
+        max_length=255,
+        blank=True,
+        null=True,
+        default=None,
+        choices=NimbusConstants.QATestType.choices,
+    )
+    qa_run_test_plan = models.URLField(
+        "QA Run Test Plan Link",
+        max_length=500,
+        blank=True,
+        null=True,
+        default=None,
+    )
+    qa_run_testrail_link = models.URLField(
+        "QA Run TestRail Link",
+        max_length=500,
+        blank=True,
+        null=True,
+        default=None,
+    )
 
     class Meta:
         verbose_name = "Nimbus Experiment"

experimenter/experimenter/experiments/tests/test_changelog_utils.py

@@ -98,8 +98,12 @@ def test_outputs_expected_schema_for_empty_experiment(self):
                 "publish_status": NimbusExperiment.PublishStatus.IDLE,
                 "published_dto": None,
                 "qa_comment": None,
+                "qa_run_date": None,
+                "qa_run_type": None,
                 "qa_signoff": False,
                 "qa_status": NimbusExperiment.QAStatus.NOT_SET,
+                "qa_run_test_plan": None,
+                "qa_run_testrail_link": None,
                 "reference_branch": None,
                 "required_experiments": [],
                 "requires_restart": False,
@@ -220,8 +224,12 @@ def test_outputs_expected_schema_for_complete_experiment(self):
                 "public_description": experiment.public_description,
                 "publish_status": experiment.publish_status,
                 "qa_comment": experiment.qa_comment,
+                "qa_run_date": experiment.qa_run_date,
+                "qa_run_type": experiment.qa_run_type,
                 "qa_signoff": False,
                 "qa_status": experiment.qa_status,
+                "qa_run_test_plan": experiment.qa_run_test_plan,
+                "qa_run_testrail_link": experiment.qa_run_testrail_link,
                 "required_experiments": [],
                 "requires_restart": False,
                 "results_data": experiment.results_data,

experimenter/experimenter/experiments/tests/test_models.py

@@ -3715,6 +3715,26 @@ def test_qa_status_badge_class(self, qa_status, expected_badge_class):
         )
         self.assertEqual(experiment.qa_status_badge_class, expected_badge_class)
 
+    @parameterized.expand(NimbusExperiment.QATestType.choices)
+    def test_qa_run_type_display(self, qa_run_type, expected_display):
+        experiment = NimbusExperimentFactory.create(qa_run_type=qa_run_type)
+        self.assertEqual(experiment.get_qa_run_type_display(), expected_display)
+
+    def test_qa_run_date_accepts_valid_date(self):
+        test_date = datetime.date(2024, 1, 15)
+        experiment = NimbusExperimentFactory.create(qa_run_date=test_date)
+        self.assertEqual(experiment.qa_run_date, test_date)
+
+    def test_qa_run_test_plan_accepts_valid_url(self):
+        test_url = "https://example.com/test-plan"
+        experiment = NimbusExperimentFactory.create(qa_run_test_plan=test_url)
+        self.assertEqual(experiment.qa_run_test_plan, test_url)
+
+    def test_qa_run_testrail_link_accepts_valid_url(self):
+        test_url = "https://testrail.example.com/index.php?/runs/view/12345"
+        experiment = NimbusExperimentFactory.create(qa_run_testrail_link=test_url)
+        self.assertEqual(experiment.qa_run_testrail_link, test_url)
+
     def test_clone_created_experiment(self):
         owner = UserFactory.create()
         required_experiment = NimbusExperimentFactory.create()
@@ -3864,6 +3884,10 @@ def _clone_experiment_and_assert_common_expectations(
         self.assertEqual(child.conclusion_recommendations, [])
         self.assertEqual(child.qa_status, NimbusExperiment.QAStatus.NOT_SET)
         self.assertEqual(child.qa_comment, None)
+        self.assertEqual(child.qa_run_date, None)
+        self.assertEqual(child.qa_run_type, None)
+        self.assertEqual(child.qa_run_test_plan, None)
+        self.assertEqual(child.qa_run_testrail_link, None)
         self.assertEqual(child._start_date, None)
         self.assertEqual(child._end_date, None)
         self.assertEqual(child._enrollment_end_date, None)

experimenter/experimenter/nimbus_ui/filtersets.py

@@ -616,6 +616,12 @@ class Deliveries(models.TextChoices):
         SIZE_UP = "total_enrolled_clients", "Pop. Size"
         SIZE_DOWN = "-total_enrolled_clients", "Pop. Size"
 
+    class QARuns(models.TextChoices):
+        DATE_UP = "qa_run_date", "Date"
+        DATE_DOWN = "-qa_run_date", "Date"
+        TYPE_UP = "qa_run_type", "Testing Type"
+        TYPE_DOWN = "-qa_run_type", "Testing Type"
+
     @staticmethod
     def sortable_headers(table_name):
         seen = set()