twister: for ci and in twister.yml, treat built tests as passed

For CI and for the condensed version of the report, consider built tests
as passed and do not mark them as skipped. Tests that are built only
will be reported as skipped in the full and suite reports.

Signed-off-by: Anas Nashif <[email protected]>

Author: nashif

scripts/pylib/twister/twisterlib.py

@@ -3866,7 +3866,7 @@ def process_log(log_file):
 
 
     @staticmethod
-    def xunit_testcase(eleTestsuite, name, classname, status, ts_status, reason, duration, runnable, stats, log):
+    def xunit_testcase(eleTestsuite, name, classname, status, ts_status, reason, duration, runnable, stats, log, build_only_as_skip):
         fails, passes, errors, skips = stats
 
         if status in ['skipped', 'filtered']:
@@ -3880,6 +3880,7 @@ def xunit_testcase(eleTestsuite, name, classname, status, ts_status, reason, dur
 
         if status in ['skipped', 'filtered']:
             skips += 1
+            # temporarily add build_only_as_skip to restore existing CI report behaviour
             if ts_status == "passed" and not runnable:
                 tc_type = "build"
             else:
@@ -3896,7 +3897,7 @@ def xunit_testcase(eleTestsuite, name, classname, status, ts_status, reason, dur
             if log:
                 el.text = log
         elif status == 'passed':
-            if not runnable:
+            if not runnable and build_only_as_skip:
                 ET.SubElement(eleTestcase, 'skipped', type="build", message="built only")
                 skips += 1
             else:
@@ -3960,7 +3961,7 @@ def xunit_report_suites(self, json_file, filename):
                 classname = ".".join(name.split(".")[:2])
                 fails, passes, errors, skips = self.xunit_testcase(eleTestsuite,
                     name, classname, status, ts_status, reason, tc_duration, runnable,
-                    (fails, passes, errors, skips), log)
+                    (fails, passes, errors, skips), log, True)
 
             total = (errors + passes + fails + skips)
 
@@ -4036,15 +4037,15 @@ def xunit_report(self, json_file, filename, selected_platform=None, full_report=
                         classname = ".".join(name.split(".")[:2])
                         fails, passes, errors, skips = self.xunit_testcase(eleTestsuite,
                             name, classname, status, ts_status, reason, tc_duration, runnable,
-                            (fails, passes, errors, skips), log)
+                            (fails, passes, errors, skips), log, True)
                 else:
                     reason = ts.get('reason', 'Unknown')
                     name = ts.get("name")
                     classname = f"{platform}:{name}"
                     log = ts.get("log")
                     fails, passes, errors, skips = self.xunit_testcase(eleTestsuite,
                         name, classname, ts_status, ts_status, reason, duration, runnable,
-                        (fails, passes, errors, skips), log)
+                        (fails, passes, errors, skips), log, False)
 
             total = (errors + passes + fails + skips)