diff options
author | Hal Canary <halcanary@google.com> | 2019-06-26 07:16:26 -0700 |
---|---|---|
committer | android-build-merger <android-build-merger@google.com> | 2019-06-26 07:16:26 -0700 |
commit | 66b4348e7d56e4ff50d3f2b5595d826d2b5d8c6a (patch) | |
tree | 207e510b2ea56aefff5ef3318934712511f8774c /platform_tools | |
parent | acc973735cb87bcb78d4322f0298cd92996cd417 (diff) | |
parent | 172d9d067a9b372957c48391ce5862280bddd089 (diff) | |
download | platform_external_skqp-66b4348e7d56e4ff50d3f2b5595d826d2b5d8c6a.tar.gz platform_external_skqp-66b4348e7d56e4ff50d3f2b5595d826d2b5d8c6a.tar.bz2 platform_external_skqp-66b4348e7d56e4ff50d3f2b5595d826d2b5d8c6a.zip |
SkQP: fix Filterable.filter am: c0b8bf2b74
am: 172d9d067a
Change-Id: I4b42b8101457e504e58cc7cbba73010d2dd07506
Diffstat (limited to 'platform_tools')
-rw-r--r-- | platform_tools/android/apps/skqp/src/main/java/org/skia/skqp/SkQPRunner.java | 42 |
1 files changed, 21 insertions, 21 deletions
diff --git a/platform_tools/android/apps/skqp/src/main/java/org/skia/skqp/SkQPRunner.java b/platform_tools/android/apps/skqp/src/main/java/org/skia/skqp/SkQPRunner.java index 8e360c8840..e65d618d7d 100644 --- a/platform_tools/android/apps/skqp/src/main/java/org/skia/skqp/SkQPRunner.java +++ b/platform_tools/android/apps/skqp/src/main/java/org/skia/skqp/SkQPRunner.java @@ -29,6 +29,7 @@ import org.junit.runner.notification.RunNotifier; public class SkQPRunner extends Runner implements Filterable { private int mShouldRunTestCount; private Description[] mTests; + private Description mDescription; private boolean[] mShouldSkipTest; private String mOutputDirectory; private SkQP mImpl; @@ -52,8 +53,9 @@ public class SkQPRunner extends Runner implements Filterable { AssetManager assetManager = context.getResources().getAssets(); mImpl.nInit(assetManager, mOutputDirectory); - mTests = new Description[this.testCount()]; - mShouldSkipTest = new boolean[mTests.length]; // = {false, false, ....}; + int totalCount = mImpl.mUnitTests.length + mImpl.mGMs.length * mImpl.mBackends.length; + mTests = new Description[totalCount]; + mShouldSkipTest = new boolean[totalCount]; // = {false, false, ....}; int index = 0; for (int backend = 0; backend < mImpl.mBackends.length; backend++) { for (int gm = 0; gm < mImpl.mGMs.length; gm++) { @@ -65,37 +67,35 @@ public class SkQPRunner extends Runner implements Filterable { mTests[index++] = Description.createTestDescription(SkQPRunner.class, "unitTest_" + mImpl.mUnitTests[unitTest]); } - assert(index == mTests.length); - mShouldRunTestCount = mTests.length; + assert(index == totalCount); + this.updateDescription(null); } - @Override - public void filter(Filter filter) throws NoTestsRemainException { - int count = 0; + private void updateDescription(Filter filter) { + mShouldRunTestCount = 0; + mDescription = Description.createSuiteDescription(SkQP.class); + assert(mTests.length == mShouldSkipTest.length); for (int i = 0; i < mTests.length; ++i) { - mShouldSkipTest[i] = !filter.shouldRun(mTests[i]); - if (!mShouldSkipTest[i]) { - ++count; + boolean doRunTest = filter != null ? filter.shouldRun(mTests[i]) : true; + mShouldSkipTest[i] = !doRunTest; + if (doRunTest) { + mDescription.addChild(mTests[i]); + ++mShouldRunTestCount; } } - mShouldRunTestCount = count; - if (0 == count) { - throw new NoTestsRemainException(); - } } @Override - public Description getDescription() { - Description d = Description.createSuiteDescription(SkQP.class); - for (int i = 0; i < mTests.length; ++i) { - d.addChild(mTests[i]); + public void filter(Filter filter) throws NoTestsRemainException { + this.updateDescription(filter); + if (0 == mShouldRunTestCount) { + throw new NoTestsRemainException(); } - return d; } @Override - public int testCount() { - return mImpl.mUnitTests.length + mImpl.mGMs.length * mImpl.mBackends.length; + public Description getDescription() { + return mDescription; } @Override |