diff options
author | Young Gyu Park <younggyu@google.com> | 2018-11-08 11:35:02 -0800 |
---|---|---|
committer | android-build-merger <android-build-merger@google.com> | 2018-11-08 11:35:02 -0800 |
commit | 4256a75e55599292f76c2a989abb332c18844ddd (patch) | |
tree | 481b481a4b01936061fe7fa25af6373c49f73e98 | |
parent | cd08956f77d21bd04d647f7d7b2169538791ac7a (diff) | |
parent | 978b3421aefa1b060e0e2c19041c0a9b1cd1a8bf (diff) | |
download | dashboard-4256a75e55599292f76c2a989abb332c18844ddd.tar.gz |
Bug fix on creating TestCase inner class am: a01b6fc96c am: 1b9873360f
am: 978b3421ae
Change-Id: I4af34117742a209aee06b9502ebde0943944ea5c
4 files changed, 64 insertions, 16 deletions
diff --git a/src/main/java/com/android/vts/api/TestRunRestServlet.java b/src/main/java/com/android/vts/api/TestRunRestServlet.java index 2c724ac..90fd3f9 100644 --- a/src/main/java/com/android/vts/api/TestRunRestServlet.java +++ b/src/main/java/com/android/vts/api/TestRunRestServlet.java @@ -16,20 +16,10 @@ package com.android.vts.api; -import com.android.vts.entity.CodeCoverageEntity; import com.android.vts.entity.TestCaseRunEntity; import com.android.vts.entity.TestEntity; import com.android.vts.entity.TestRunEntity; -import com.android.vts.util.FilterUtil; import com.android.vts.util.TestRunDetails; -import com.google.appengine.api.datastore.DatastoreService; -import com.google.appengine.api.datastore.DatastoreServiceFactory; -import com.google.appengine.api.datastore.Entity; -import com.google.appengine.api.datastore.EntityNotFoundException; -import com.google.appengine.api.datastore.FetchOptions; -import com.google.appengine.api.datastore.Key; -import com.google.appengine.api.datastore.KeyFactory; -import com.google.appengine.api.datastore.Query; import com.google.gson.Gson; import java.io.IOException; import java.io.PrintWriter; @@ -38,7 +28,6 @@ import java.util.List; import java.util.Map; import java.util.Objects; import java.util.logging.Logger; -import javax.servlet.http.HttpServlet; import javax.servlet.http.HttpServletRequest; import javax.servlet.http.HttpServletResponse; @@ -108,10 +97,7 @@ public class TestRunRestServlet extends BaseApiServlet { } Map<com.googlecode.objectify.Key<TestCaseRunEntity>, TestCaseRunEntity> testCaseRunEntityKeyMap = ofy().load().keys(() -> testCaseKeyList.iterator()); - for (Map.Entry<com.googlecode.objectify.Key<TestCaseRunEntity>, TestCaseRunEntity> entry : - testCaseRunEntityKeyMap.entrySet()) { - details.addTestCase(entry.getValue()); - } + testCaseRunEntityKeyMap.forEach((key, value) -> details.addTestCase(value)); } return details; } diff --git a/src/main/java/com/android/vts/config/ObjectifyListener.java b/src/main/java/com/android/vts/config/ObjectifyListener.java index 5f35abd..02e0215 100644 --- a/src/main/java/com/android/vts/config/ObjectifyListener.java +++ b/src/main/java/com/android/vts/config/ObjectifyListener.java @@ -28,6 +28,7 @@ import com.android.vts.entity.ProfilingPointEntity; import com.android.vts.entity.ProfilingPointRunEntity; import com.android.vts.entity.ProfilingPointSummaryEntity; import com.android.vts.entity.RoleEntity; +import com.android.vts.entity.TestAcknowledgmentEntity; import com.android.vts.entity.TestCaseRunEntity; import com.android.vts.entity.TestCoverageStatusEntity; import com.android.vts.entity.TestEntity; diff --git a/src/main/java/com/android/vts/entity/TestCaseRunEntity.java b/src/main/java/com/android/vts/entity/TestCaseRunEntity.java index 30776d4..85a6c43 100644 --- a/src/main/java/com/android/vts/entity/TestCaseRunEntity.java +++ b/src/main/java/com/android/vts/entity/TestCaseRunEntity.java @@ -138,7 +138,9 @@ public class TestCaseRunEntity implements DashboardEntity { private void onLoad() { if (testCaseNames.size() == results.size()) { for (int index = 0; index < testCaseNames.size(); index++) { - this.addTestCase(testCaseNames.get(index), results.get(index).intValue()); + String name = testCaseNames.get(index); + int result = results.get(index).intValue(); + this.testCases.add(new TestCase(this.id, this.testCases.size(), name, result)); } } } diff --git a/src/test/java/com/android/vts/entity/TestCaseRunEntityTest.java b/src/test/java/com/android/vts/entity/TestCaseRunEntityTest.java new file mode 100644 index 0000000..d4abd38 --- /dev/null +++ b/src/test/java/com/android/vts/entity/TestCaseRunEntityTest.java @@ -0,0 +1,59 @@ +/* + * Copyright (C) 2018 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); you + * may not use this file except in compliance with the License. You may + * obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or + * implied. See the License for the specific language governing + * permissions and limitations under the License. + */ + +package com.android.vts.entity; + +import com.android.vts.util.ObjectifyTestBase; +import org.junit.jupiter.api.Test; + +import java.util.Arrays; +import java.util.List; + +import static com.googlecode.objectify.ObjectifyService.factory; +import static org.junit.Assert.assertEquals; + +public class TestCaseRunEntityTest extends ObjectifyTestBase { + + @Test + public void saveTest() { + + factory().register(TestCaseRunEntity.class); + + List<Integer> results = Arrays.asList(1, 1, 1, 1, 1, 1, 1); + List<String> testCaseNames = + Arrays.asList( + "AudioEffectsFactoryTest.EnumerateEffects(default)_32bit", + "AudioEffectsFactoryTest.CreateEffect(default)_32bit", + "AudioEffectsFactoryTest.GetDescriptor(default)_32bit", + "AudioEffectsFactoryTest.DebugDumpArgument(default)_32bit", + "AudioEffectTest.Close(default)_32bit", + "AudioEffectTest.GetDescriptor(default)_32bit", + "AudioEffectTest.GetSetConfig(default)_32bit"); + + TestCaseRunEntity testCaseRunEntity = new TestCaseRunEntity(); + for (int index = 0; index < results.size(); index++) { + String testCaseName = testCaseNames.get(index); + int result = results.get(index); + testCaseRunEntity.addTestCase(testCaseName, result); + } + TestCaseRunEntity loadedTestCaseRunEntity = saveClearLoad(testCaseRunEntity); + + assertEquals(loadedTestCaseRunEntity.getTestCases().size(), results.size()); + assertEquals( + (Integer) loadedTestCaseRunEntity.getTestCases().get(0).result, results.get(0)); + assertEquals(loadedTestCaseRunEntity.getTestCases().get(0).name, testCaseNames.get(0)); + } +} |