From 9fd564711ec5fd7361d0fc723256bb58569501f5 Mon Sep 17 00:00:00 2001 From: Paul Duffin Date: Thu, 31 Mar 2022 15:42:30 +0100 Subject: bootclasspath_fragment: Require at least one hidden_api package property Previously, the split_packages, single_packages and package_prefixes properties were all optional and the split_packages defaulted to ["*"]. As that value conflicted with the other package properties that meant that split_packages always had to be specified even if it was to just set it to an empty array. This change requires at least one of them to be specified and defaults split_packages to an empty list which means it is not required, although it can be helpful to make that explicit. Bug: 194063708 Test: m nothing Change-Id: I5a4c2d68e72e39f5c4a2441326dfce8685fc8ff2 --- apex/classpath_element_test.go | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'apex/classpath_element_test.go') diff --git a/apex/classpath_element_test.go b/apex/classpath_element_test.go index 60f18bd57..9142eed99 100644 --- a/apex/classpath_element_test.go +++ b/apex/classpath_element_test.go @@ -88,6 +88,9 @@ func TestCreateClasspathElements(t *testing.T) { "baz", "quuz", ], + hidden_api: { + split_packages: ["*"], + }, } java_library { @@ -134,6 +137,9 @@ func TestCreateClasspathElements(t *testing.T) { contents: [ "bar", ], + hidden_api: { + split_packages: ["*"], + }, } java_library { -- cgit v1.2.3-59-g8ed1b