PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Include com.android.settings CodeInspectorTest

Browse Source 
Don't ignore classes directly under the package com.android.settings
in CodeInspectorTest.

Test: cd tests/robotests && mma
Change-Id: I4a6b00231839729405dd96a270e2dcbfef3b73c4
This commit is contained in:
Maurice Lam
2017-05-02 22:12:09 -07:00
parent 82c039bcfe
commit b6da1d5b3f
3 changed files with 23 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
tests/robotests/src/com/android/settings/core/codeinspection/CodeInspector.java
View File

@@ -16,9 +16,6 @@
package com.android.settings.core.codeinspection;
import com.google.common.truth.Truth;
import org.junit.Assert;
import org.robolectric.shadows.ShadowApplication;
import java.io.BufferedReader;
@@ -56,7 +53,7 @@ public abstract class CodeInspector {
}
final String packageName = clazz.getPackage().getName();
// Classes that are not in Settings
if (!packageName.contains(PACKAGE_NAME + ".")) {
if (!packageName.contains(PACKAGE_NAME + ".") && !packageName.endsWith(PACKAGE_NAME)) {
return false;
}
final String className = clazz.getName();