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

Commit 724565d8 authored by yangbill's avatar yangbill
Browse files

Export test data dependencies in module-info 

Add the list of targets that are handled as data inputs and
required by tests at runtime.

Bug: 205726862
Test: m out/target/product/vsoc_x86_64/module-info.json
Change-Id: Id1468ebaf8f72cf37787f3cc514a29724c115db5
parent 8beddf2e

core/base_rules.mk

+12 −0
Original line number Diff line number Diff line
@@ -981,6 +981,18 @@ ALL_MODULES.$(my_register_name).SHARED_LIBS := \ 
ALL_MODULES.$(my_register_name).SYSTEM_SHARED_LIBS := \

    $(ALL_MODULES.$(my_register_name).SYSTEM_SHARED_LIBS) $(LOCAL_SYSTEM_SHARED_LIBRARIES)



ifdef LOCAL_TEST_DATA

  # Export the list of targets that are handled as data inputs and required

  # by tests at runtime. The LOCAL_TEST_DATA format is generated from below

  # https://cs.android.com/android/platform/superproject/+/master:build/soong/android/androidmk.go;l=925-944;drc=master

  # which format is like $(path):$(relative_file) but for module-info, only

  # the string after ":" is needed.

  ALL_MODULES.$(my_register_name).TEST_DATA := \

    $(strip $(ALL_MODULES.$(my_register_name).TEST_DATA) \

      $(foreach f, $(LOCAL_TEST_DATA),\

        $(call word-colon,2,$(f))))

endif



##########################################################################

## When compiling against the VNDK, add the .vendor or .product suffix to

## required modules.

core/tasks/module-info.mk

+1 −0
Original line number Diff line number Diff line
@@ -23,6 +23,7 @@ $(MODULE_INFO_JSON): 
			'"classes_jar": [$(foreach w,$(sort $(ALL_MODULES.$(m).CLASSES_JAR)),"$(w)", )], ' \

			'"test_mainline_modules": [$(foreach w,$(sort $(ALL_MODULES.$(m).TEST_MAINLINE_MODULES)),"$(w)", )], ' \

			'"is_unit_test": "$(ALL_MODULES.$(m).IS_UNIT_TEST)", ' \

			'"data": [$(foreach w,$(sort $(ALL_MODULES.$(m).TEST_DATA)),"$(w)", )], ' \

			'},\n' \

	 ) | sed -e 's/, *\]/]/g' -e 's/, *\}/ }/g' -e '$$s/,$$//' >> $@

	$(hide) echo '}' >> $@