Merge "Remove Ignore annotation on JSE tests" into androidx-main
NOKEYCHECK=True
GitOrigin-RevId: 7ff8f296936a0f3a062d1031d89316365ffc3879
diff --git a/androidTest/java/androidx/javascriptengine/WebViewJavaScriptSandboxTest.java b/androidTest/java/androidx/javascriptengine/WebViewJavaScriptSandboxTest.java
index 112cbb3..f28f89f 100644
--- a/androidTest/java/androidx/javascriptengine/WebViewJavaScriptSandboxTest.java
+++ b/androidTest/java/androidx/javascriptengine/WebViewJavaScriptSandboxTest.java
@@ -35,7 +35,6 @@
import org.junit.Assert;
import org.junit.Assume;
import org.junit.Before;
-import org.junit.Ignore;
import org.junit.Test;
import org.junit.runner.RunWith;
@@ -211,7 +210,6 @@
@Test
@MediumTest
- @Ignore("b/268212217")
public void testInfiniteLoop() throws Throwable {
final String code = "while(true){}";
Context context = ApplicationProvider.getApplicationContext();
@@ -240,7 +238,6 @@
@Test
@MediumTest
- @Ignore("b/268212217")
public void testMultipleInfiniteLoops() throws Throwable {
final String code = "while(true){}";
final int num_of_evaluations = 10;
@@ -275,7 +272,6 @@
@Test
@MediumTest
- @Ignore("b/268212217")
public void testSimpleArrayBuffer() throws Throwable {
final String provideString = "Hello World";
final byte[] bytes = provideString.getBytes(StandardCharsets.US_ASCII);
@@ -307,7 +303,6 @@
@Test
@MediumTest
- @Ignore("b/268212217")
public void testArrayBufferWasmCompilation() throws Throwable {
final String success = "success";
// The bytes of a minimal WebAssembly module, courtesy of v8/test/cctest/test-api-wasm.cc
@@ -341,7 +336,6 @@
@Test
@MediumTest
- @Ignore("b/268212217")
public void testPromiseReturn() throws Throwable {
final String code = "Promise.resolve(\"PASS\")";
final String expected = "PASS";
@@ -362,7 +356,6 @@
@Test
@MediumTest
- @Ignore("b/268212217")
public void testPromiseReturnLaterResolve() throws Throwable {
final String code1 = "var promiseResolve, promiseReject;"
+ "new Promise(function(resolve, reject){"
@@ -390,7 +383,6 @@
@Test
@MediumTest
- @Ignore("b/268212217")
public void testNestedConsumeNamedDataAsArrayBuffer() throws Throwable {
final String success = "success";
// The bytes of a minimal WebAssembly module, courtesy of v8/test/cctest/test-api-wasm.cc
@@ -433,7 +425,6 @@
@Test
@MediumTest
- @Ignore("b/268212217")
public void testPromiseEvaluationThrow() throws Throwable {
final String code = ""
+ "android.consumeNamedDataAsArrayBuffer(\"id-1\").catch((error) => {"
@@ -553,7 +544,6 @@
@Test
@MediumTest
- @Ignore("b/268212217")
public void testHeapSizeAdjustment() throws Throwable {
final String code = "\"PASS\"";
final String expected = "PASS";
@@ -593,7 +583,6 @@
@Test
@LargeTest
- @Ignore("b/268212217")
public void testHeapSizeEnforced() throws Throwable {
// WebView versions < 110.0.5438.0 do not contain OOM crashes to a single isolate and
// instead crash the whole sandbox process. This change is not tracked in a feature flag.
@@ -689,7 +678,6 @@
@Test
@LargeTest
- @Ignore("b/268212217")
public void testIsolateCreationAfterCrash() throws Throwable {
// WebView versions < 110.0.5438.0 do not contain OOM crashes to a single isolate and
// instead crash the whole sandbox process. This change is not tracked in a feature flag.
@@ -769,7 +757,6 @@
@Test
@MediumTest
- @Ignore("b/268212217")
public void testAsyncPromiseCallbacks() throws Throwable {
// Unlike testPromiseReturn and testPromiseEvaluationThrow, this test is guaranteed to
// exercise promises in an asynchronous way, rather than in ways which cause a promise to
@@ -838,7 +825,6 @@
@Test
@LargeTest
- @Ignore("b/268212217")
public void testLargeScriptJsEvaluation() throws Throwable {
String longString = Strings.repeat("a", 2000000);
final String code = ""
@@ -863,7 +849,6 @@
@Test
@LargeTest
- @Ignore("b/268212217")
public void testLargeScriptByteArrayJsEvaluation() throws Throwable {
final String longString = Strings.repeat("a", 2000000);
final String codeString = ""
@@ -889,7 +874,6 @@
@Test
@LargeTest
- @Ignore("b/268212217")
public void testLargeReturn() throws Throwable {
final String code = "'a'.repeat(2000000);";
final String expected = Strings.repeat("a", 2000000);
@@ -911,7 +895,6 @@
@Test
@LargeTest
- @Ignore("b/268212217")
public void testLargeError() throws Throwable {
final String longString = Strings.repeat("a", 2000000);
final String code = "throw \"" + longString + "\");";
@@ -937,7 +920,6 @@
@Test
@MediumTest
- @Ignore("b/268212217")
public void testResultSizeEnforced() throws Throwable {
final int maxSize = 100;
Context context = ApplicationProvider.getApplicationContext();
@@ -988,7 +970,6 @@
@Test
@LargeTest
- @Ignore("b/268212217")
public void testConsoleLogging() throws Throwable {
final class LoggingJavaScriptConsoleCallback implements JavaScriptConsoleCallback {
private final Object mLock = new Object();
@@ -1141,7 +1122,6 @@
@Test
@MediumTest
- @Ignore("b/268212217")
public void testConsoleCallbackCanCallService() throws Throwable {
// This checks that there is nothing intrinsically wrong with calling service APIs from a
// console client. Note that, in theory, Binder will reuse the same threads if code recurses
