[FIX] Split regression_testid_passed query to optimize (#834)

* Split query for get_info_for_pr_comment

Signed-off-by: Tarun Arora <tarun.arora.030402@gmail.com>

* Fix pycodestyle issues

Signed-off-by: Tarun Arora <tarun.arora.030402@gmail.com>

---------

Signed-off-by: Tarun Arora <tarun.arora.030402@gmail.com>

Author: Tarun-Arora

mod_ci/controllers.py

@@ -1548,6 +1548,15 @@ def set_avg_time(platform, process_type: str, time_taken: int) -> None:
 
 def get_info_for_pr_comment(test_id: int) -> PrCommentInfo:
     """Return info about the given test id for use in a PR comment."""
+    regression_testid_passed = g.db.query(TestResult.regression_test_id).outerjoin(
+        TestResultFile, TestResult.test_id == TestResultFile.test_id).filter(
+        TestResult.test_id == test_id,
+        TestResult.expected_rc == TestResult.exit_code,
+        and_(
+            RegressionTestOutput.regression_id == TestResult.regression_test_id,
+            RegressionTestOutput.ignore.is_(True)
+        )).distinct().subquery()
+
     regression_testid_passed = g.db.query(TestResult.regression_test_id).outerjoin(
         TestResultFile, TestResult.test_id == TestResultFile.test_id).filter(
         TestResult.test_id == test_id,
@@ -1560,11 +1569,9 @@ def get_info_for_pr_comment(test_id: int) -> PrCommentInfo:
                      and_(
                      RegressionTestOutputFiles.regression_test_output_id == TestResultFile.regression_test_output_id,
                      TestResultFile.got == RegressionTestOutputFiles.file_hashes
-                 ))),
-            and_(
-                RegressionTestOutput.regression_id == TestResult.regression_test_id,
-                RegressionTestOutput.ignore.is_(True),
-            ))).distinct().subquery()
+                 )))
+        )).distinct().union(g.db.query(regression_testid_passed.c.regression_test_id))
+
     passed = g.db.query(label('category_id', Category.id), label(
         'success', count(regressionTestLinkTable.c.regression_id))).filter(
             regressionTestLinkTable.c.regression_id.in_(regression_testid_passed),