DexLoaderTest.testPreviouslyLoadedFromLocalDataDir flaky |
||||
Issue descriptionAndroid Tests (dbg) failed on testPreviouslyLoadedFromLocalDataDir: Failed build: https://uberchromegw.corp.google.com/i/chromium.linux/builders/Android%20Tests%20(dbg)/builds/34429 Log: [FAIL] org.chromium.webapk.shell_apk.DexLoaderTest#testPreviouslyLoadedFromLocalDataDir: junit.framework.AssertionFailedError at org.chromium.webapk.shell_apk.DexLoaderTest.testPreviouslyLoadedFromLocalDataDir(DexLoaderTest.java:212) at java.lang.reflect.Method.invokeNative(Native Method) at android.test.InstrumentationTestCase.runMethod(InstrumentationTestCase.java:214) at android.test.InstrumentationTestCase.runTest(InstrumentationTestCase.java:199) at org.chromium.base.test.BaseTestResult.run(BaseTestResult.java:129) at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:191) at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:176) at android.test.InstrumentationTestRunner.onStart(InstrumentationTestRunner.java:554) at android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1701) It seems to be passing most of the time though.
,
May 19 2016
,
May 23 2016
Can we disable this test until it's fixed? It's flaking pretty frequently on Android Tests (dbg): http://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=chrome_public_test_apk&tests=org.chromium.webapk.shell_apk.DexLoaderTest%23testPreviouslyLoadedFromLocalDataDir
,
May 23 2016
I'm going to disable.
,
May 24 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/4be351834826fa6bc3576f63c144b88d75ea49d9 commit 4be351834826fa6bc3576f63c144b88d75ea49d9 Author: aelias <aelias@chromium.org> Date: Tue May 24 00:29:28 2016 Mark DexLoaderTest.testPreviouslyLoadedFromDataDir flaky. This test has been 25% flaky since it was added last week in r394656. (The other tests in the DexLoaderTest package are consistently green.) TBR=pkotwicz NOTRY=true BUG= 613116 Review-Url: https://codereview.chromium.org/2005073002 Cr-Commit-Position: refs/heads/master@{#395484} [modify] https://crrev.com/4be351834826fa6bc3576f63c144b88d75ea49d9/chrome/android/webapk/shell_apk/BUILD.gn [modify] https://crrev.com/4be351834826fa6bc3576f63c144b88d75ea49d9/chrome/android/webapk/shell_apk/javatests/src/org/chromium/webapk/shell_apk/DexLoaderTest.java
,
May 24 2016
Back to pkotwicz@ to track fixing and reenabling this.
,
May 24 2016
Thanks for disabling this.
,
May 25 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/498ec85734442ebbefca2874a934a5700e53e0aa commit 498ec85734442ebbefca2874a934a5700e53e0aa Author: pkotwicz <pkotwicz@chromium.org> Date: Wed May 25 21:27:53 2016 Remove flaky assertion in DexLoaderTest#testPreviouslyLoadedFromLocalDataDir Previously the test was checking whether canary.dex was being read - testing the internal implementation of BaseDexClassLoader. It is possible that BaseDexClassLoader reads the optimized version of canary.dex and not canary.dex BUG= 613116 Review-Url: https://codereview.chromium.org/2007553005 Cr-Commit-Position: refs/heads/master@{#395988} [modify] https://crrev.com/498ec85734442ebbefca2874a934a5700e53e0aa/chrome/android/webapk/shell_apk/javatests/src/org/chromium/webapk/shell_apk/DexLoaderTest.java
,
May 26 2016
,
May 28 2016
Pada tanggal 27 Mei 2016 02.44, "pkotwicz@chromium.org via Monorail" < monorail@chromium.org> menulis: |
||||
►
Sign in to add a comment |
||||
Comment 1 by peconn@chromium.org
, May 19 2016