Do not build linux-tests for fake platform with clang. Disable such t…

…ests in general in cloudbuild. (#20373) (#20424)

* Do not build fake-tests-clang

The compiler platform for fake-tests is //build/toolchain/fake:fake_x64_gcc
which is NOT a clang toolchain.

* Add a comment about why the board check is done

* Fix unit tests

* Do not build tests (not even the clang version) for all build in cloudbuild

* Do not build tests (not even the clang version) for all build in cloudbuild

* Restyle

* Revert "Restyle"

This reverts commit 8bd2f67.

integrations/cloudbuild/build-all.yaml

@@ -19,7 +19,7 @@ steps:
           - >-
               ./scripts/build/build_examples.py --enable-flashbundle
               --target-glob '*' --skip-target-glob
-              '{tizen-*,*-tests,*-chip-test}' build --create-archives
+              '{tizen-*,*-tests*,*-chip-test}' build --create-archives
               /workspace/artifacts/
       id: CompileAll
       waitFor:

integrations/cloudbuild/smoke-test.yaml

@@ -69,7 +69,7 @@ steps:
       args:
           - >-
               ./scripts/build/build_examples.py --enable-flashbundle
-              --target-glob 'linux-*' --skip-target-glob '*-tests' build
+              --target-glob 'linux-*' --skip-target-glob '*-tests*' build
               --create-archives /workspace/artifacts/
       waitFor:
           - Bootstrap

scripts/build/build/targets.py

@@ -305,9 +305,9 @@ def HostTargets():
     yield target_native.Extend('all-clusters-app-nodeps', app=HostApp.ALL_CLUSTERS, **nodeps_args)
 
     test_target = Target(HostBoard.NATIVE.PlatformName(), HostBuilder)
-    for board in [HostBoard.NATIVE, HostBoard.FAKE]:
-        yield test_target.Extend(board.BoardName() + '-tests', board=board, app=HostApp.TESTS)
-        yield test_target.Extend(board.BoardName() + '-tests-clang', board=board, app=HostApp.TESTS, use_clang=True)
+    yield test_target.Extend(HostBoard.NATIVE.BoardName() + '-tests', board=HostBoard.NATIVE, app=HostApp.TESTS)
+    yield test_target.Extend(HostBoard.NATIVE.BoardName() + '-tests-clang', board=HostBoard.NATIVE, app=HostApp.TESTS, use_clang=True)
+    yield test_target.Extend(HostBoard.FAKE.BoardName() + '-tests', board=HostBoard.FAKE, app=HostApp.TESTS)
 
 
 def Esp32Targets():

scripts/build/builders/host.py

@@ -239,6 +239,11 @@ def __init__(self, root, runner, app: HostApp, board=HostBoard.NATIVE,
         if use_clang:
             self.extra_gn_options.append('is_clang=true')
 
+            if self.board == HostBoard.FAKE:
+                # Fake uses "//build/toolchain/fake:fake_x64_gcc"
+                # so setting clang is not correct
+                raise Exception('Fake host board is always gcc (not clang)')
+
         if use_platform_mdns:
             self.extra_gn_options.append('chip_mdns="platform"')
 

scripts/build/testdata/build_linux_on_x64.txt

@@ -149,9 +149,6 @@ PKG_CONFIG_PATH="SYSROOT_AARCH64/lib/aarch64-linux-gnu/pkgconfig" \
 # Generating linux-fake-tests
 gn gen --check --fail-on-unused-args --export-compile-commands --root={root} '--args=chip_build_tests=true custom_toolchain="//build/toolchain/fake:fake_x64_gcc" chip_link_tests=true chip_device_platform="fake" chip_fake_platform=true' {out}/linux-fake-tests
 
-# Generating linux-fake-tests-clang
-gn gen --check --fail-on-unused-args --export-compile-commands --root={root} '--args=is_clang=true chip_build_tests=true custom_toolchain="//build/toolchain/fake:fake_x64_gcc" chip_link_tests=true chip_device_platform="fake" chip_fake_platform=true' {out}/linux-fake-tests-clang
-
 # Generating linux-x64-address-resolve-tool
 gn gen --check --fail-on-unused-args --export-compile-commands --root={root} {out}/linux-x64-address-resolve-tool
 
@@ -353,9 +350,6 @@ ninja -C {out}/linux-arm64-tv-casting-app-ipv6only
 # Building linux-fake-tests
 ninja -C {out}/linux-fake-tests check
 
-# Building linux-fake-tests-clang
-ninja -C {out}/linux-fake-tests-clang check
-
 # Building linux-x64-address-resolve-tool
 ninja -C {out}/linux-x64-address-resolve-tool src/lib/address_resolve:address-resolve-tool