Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1ef901c1 authored by Simran Basi's avatar Simran Basi
Browse files

build/make: Add default suite for all native tests/benchmarks 

Not all native tests have the "tests" tag so adjust the logic to be:
NATIVE_TESTS||NATIVE_BENCHMARKS||(APPS&&tags==tests)

Bug: 74339384
Test: Rebuilt module-info.json, verified native tests now have
      their suite set to null-suite and have auto_test_config=True
      Double checked a frameworks App unittest as well.

Change-Id: Idea00adcebfc1a78b0c6d8ade40feddb1e90ed83
Merged-In: Idea00adcebfc1a78b0c6d8ade40feddb1e90ed83
parent 17444d16

core/base_rules.mk

+6 −3
Original line number Diff line number Diff line
@@ -494,12 +494,15 @@ endif 
endif

endif



# For modules tagged as tests but lacking a suite tag, set null-suite as the default.

# For test modules that lack a suite tag, set null-suite as the default.

# We only support adding a default suite to native tests, native benchmarks, and instrumentation tests.

# This is because they are the only tests we currently auto-generate test configs for.

ifneq ($(filter $(my_module_tags),tests),)

ifndef LOCAL_COMPATIBILITY_SUITE

ifneq ($(filter NATIVE_TESTS NATIVE_BENCHMARK APPS, $(LOCAL_MODULE_CLASS)),)

ifneq ($(filter NATIVE_TESTS NATIVE_BENCHMARK, $(LOCAL_MODULE_CLASS)),)

LOCAL_COMPATIBILITY_SUITE := null-suite

endif

ifneq ($(filter APPS, $(LOCAL_MODULE_CLASS)),)

ifneq ($(filter $(my_module_tags),tests),)

LOCAL_COMPATIBILITY_SUITE := null-suite

endif

endif