Rename to BASE_CLS_TO_INCLUDE, etc. as Android's inclusive language guidance
See https://source.android.com/setup/contribute/respectful-code for reference BUG=162536543 BUG=268488144 Test: atest AAAPlusPlusVerifySysuiRequiredTestPropertiesTest Change-Id: If1a381f93896821356012a1d4652822dcbe928a4
This commit is contained in:
@@ -59,7 +59,7 @@ public class AAAPlusPlusVerifySysuiRequiredTestPropertiesTest extends SysuiTestC
|
||||
|
||||
private static final String TAG = "AAA++VerifyTest";
|
||||
|
||||
private static final Class[] BASE_CLS_WHITELIST = {
|
||||
private static final Class[] BASE_CLS_TO_INCLUDE = {
|
||||
SysuiTestCase.class,
|
||||
SysuiBaseFragmentTest.class,
|
||||
};
|
||||
@@ -81,7 +81,7 @@ public class AAAPlusPlusVerifySysuiRequiredTestPropertiesTest extends SysuiTestC
|
||||
if (!isTestClass(cls)) continue;
|
||||
|
||||
boolean hasParent = false;
|
||||
for (Class<?> parent : BASE_CLS_WHITELIST) {
|
||||
for (Class<?> parent : BASE_CLS_TO_INCLUDE) {
|
||||
if (parent.isAssignableFrom(cls)) {
|
||||
hasParent = true;
|
||||
break;
|
||||
@@ -131,13 +131,13 @@ public class AAAPlusPlusVerifySysuiRequiredTestPropertiesTest extends SysuiTestC
|
||||
// with the main process dependency graph because it will not exist
|
||||
// at runtime and could lead to incorrect tests which assume
|
||||
// the main SystemUI process. Therefore, exclude this package
|
||||
// from the base class whitelist.
|
||||
// from the base class allowlist.
|
||||
filter.add(s -> !s.startsWith("com.android.systemui.screenshot"));
|
||||
return filter;
|
||||
}
|
||||
|
||||
private String getClsStr() {
|
||||
return TextUtils.join(",", Arrays.asList(BASE_CLS_WHITELIST)
|
||||
return TextUtils.join(",", Arrays.asList(BASE_CLS_TO_INCLUDE)
|
||||
.stream().map(cls -> cls.getSimpleName()).toArray());
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user