Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1645)

Unified Diff: base/test/android/javatests/src/org/chromium/base/test/util/MinAndroidSdkLevelSkipCheck.java

Issue 2273553002: Create BaseJUnitClassRunner to run junit4 style tests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Do not obfuscate junit Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: base/test/android/javatests/src/org/chromium/base/test/util/MinAndroidSdkLevelSkipCheck.java
diff --git a/base/test/android/javatests/src/org/chromium/base/test/util/MinAndroidSdkLevelSkipCheck.java b/base/test/android/javatests/src/org/chromium/base/test/util/MinAndroidSdkLevelSkipCheck.java
index c95ae224364ba3065c1f47552efba0573d713ad6..99bd1acf2bbd8da8449f53dffbc0873157fa9f05 100644
--- a/base/test/android/javatests/src/org/chromium/base/test/util/MinAndroidSdkLevelSkipCheck.java
+++ b/base/test/android/javatests/src/org/chromium/base/test/util/MinAndroidSdkLevelSkipCheck.java
@@ -6,12 +6,10 @@ package org.chromium.base.test.util;
import android.os.Build;
-import junit.framework.TestCase;
+import org.junit.runners.model.FrameworkMethod;
import org.chromium.base.Log;
-import java.lang.reflect.Method;
-
/**
* Checks the device's SDK level against any specified minimum requirement.
*/
@@ -27,21 +25,18 @@ public class MinAndroidSdkLevelSkipCheck extends SkipCheck {
* @return true if the device's SDK level is below the specified minimum.
*/
@Override
- public boolean shouldSkip(TestCase testCase) {
- Class testClass = testCase.getClass();
- Method testMethod = getTestMethod(testCase);
-
+ public boolean shouldSkip(FrameworkMethod frameworkMethod) {
int minSdkLevel = 0;
- for (MinAndroidSdkLevel m : getAnnotations(testMethod, MinAndroidSdkLevel.class)) {
+ for (MinAndroidSdkLevel m : getAnnotations(frameworkMethod, MinAndroidSdkLevel.class)) {
minSdkLevel = Math.max(minSdkLevel, m.value());
}
-
if (Build.VERSION.SDK_INT < minSdkLevel) {
- Log.i(TAG, "Test " + testClass.getName() + "#" + testCase.getName()
- + " is not enabled at SDK level " + Build.VERSION.SDK_INT
- + ".");
+ Log.i(TAG, "Test " + frameworkMethod.getDeclaringClass().getName() + "#"
+ + frameworkMethod.getName() + " is not enabled at SDK level "
+ + Build.VERSION.SDK_INT + ".");
return true;
}
return false;
}
+
}

Powered by Google App Engine
This is Rietveld 408576698