From bdb3d3db25bdb56f595394d1cfa70e94ba9e7227 Mon Sep 17 00:00:00 2001 From: Volodymyr Babak Date: Tue, 4 Jul 2017 14:46:54 +0300 Subject: [PATCH] Fixes for JPA DAO tests --- .../server/dao/sql/JpaAbstractDao.java | 2 -- .../server/dao/sql/asset/AssetRepository.java | 2 +- .../server/dao/JpaDaoTestSuite.java | 2 +- .../server/dao/JpaDbunitTestConfig.java | 1 - .../server/dao/sql/alarm/JpaAlarmDaoTest.java | 13 ---------- .../resources/dbunit/device_credentials.xml | 8 +++--- dao/src/test/resources/dbunit/event.xml | 16 ++++++------ dao/src/test/resources/dbunit/rule.xml | 16 ++++++------ dao/src/test/resources/dbunit/user.xml | 26 +++++++++---------- .../resources/dbunit/user_credentials.xml | 8 +++--- dao/src/test/resources/dbunit/widget_type.xml | 16 ++++++------ .../test/resources/dbunit/widgets_bundle.xml | 20 +++++++------- 12 files changed, 57 insertions(+), 73 deletions(-) diff --git a/dao/src/main/java/org/thingsboard/server/dao/sql/JpaAbstractDao.java b/dao/src/main/java/org/thingsboard/server/dao/sql/JpaAbstractDao.java index 2fd6c97689..8571bd6408 100644 --- a/dao/src/main/java/org/thingsboard/server/dao/sql/JpaAbstractDao.java +++ b/dao/src/main/java/org/thingsboard/server/dao/sql/JpaAbstractDao.java @@ -28,8 +28,6 @@ import org.thingsboard.server.dao.model.BaseEntity; import java.util.List; import java.util.UUID; -import static org.springframework.transaction.annotation.Propagation.REQUIRES_NEW; - /** * @author Valerii Sosliuk */ diff --git a/dao/src/main/java/org/thingsboard/server/dao/sql/asset/AssetRepository.java b/dao/src/main/java/org/thingsboard/server/dao/sql/asset/AssetRepository.java index bc6a4d74e1..f6979a30d2 100644 --- a/dao/src/main/java/org/thingsboard/server/dao/sql/asset/AssetRepository.java +++ b/dao/src/main/java/org/thingsboard/server/dao/sql/asset/AssetRepository.java @@ -20,8 +20,8 @@ import org.springframework.data.jpa.repository.Query; import org.springframework.data.repository.CrudRepository; import org.springframework.data.repository.query.Param; import org.thingsboard.server.common.data.asset.TenantAssetType; -import org.thingsboard.server.dao.util.SqlDao; import org.thingsboard.server.dao.model.sql.AssetEntity; +import org.thingsboard.server.dao.util.SqlDao; import java.util.List; import java.util.UUID; diff --git a/dao/src/test/java/org/thingsboard/server/dao/JpaDaoTestSuite.java b/dao/src/test/java/org/thingsboard/server/dao/JpaDaoTestSuite.java index decd74b95b..4d9de47cd6 100644 --- a/dao/src/test/java/org/thingsboard/server/dao/JpaDaoTestSuite.java +++ b/dao/src/test/java/org/thingsboard/server/dao/JpaDaoTestSuite.java @@ -24,7 +24,7 @@ import java.util.Arrays; @RunWith(ClasspathSuite.class) @ClassnameFilters({ - "org.thingsboard.server.dao.sql.*AASDTest" + "org.thingsboard.server.dao.sql.alarm.JpaAlarmDaoTest" }) public class JpaDaoTestSuite { diff --git a/dao/src/test/java/org/thingsboard/server/dao/JpaDbunitTestConfig.java b/dao/src/test/java/org/thingsboard/server/dao/JpaDbunitTestConfig.java index 507eb4d4af..01c53b6883 100644 --- a/dao/src/test/java/org/thingsboard/server/dao/JpaDbunitTestConfig.java +++ b/dao/src/test/java/org/thingsboard/server/dao/JpaDbunitTestConfig.java @@ -51,7 +51,6 @@ public class JpaDbunitTestConfig { DatabaseDataSourceConnectionFactoryBean databaseDataSourceConnectionFactoryBean = new DatabaseDataSourceConnectionFactoryBean(); databaseDataSourceConnectionFactoryBean.setDatabaseConfig(databaseConfigBean()); databaseDataSourceConnectionFactoryBean.setDataSource(dataSource); - databaseDataSourceConnectionFactoryBean.setSchema("public"); return databaseDataSourceConnectionFactoryBean; } } diff --git a/dao/src/test/java/org/thingsboard/server/dao/sql/alarm/JpaAlarmDaoTest.java b/dao/src/test/java/org/thingsboard/server/dao/sql/alarm/JpaAlarmDaoTest.java index 265185479b..992cbba829 100644 --- a/dao/src/test/java/org/thingsboard/server/dao/sql/alarm/JpaAlarmDaoTest.java +++ b/dao/src/test/java/org/thingsboard/server/dao/sql/alarm/JpaAlarmDaoTest.java @@ -18,8 +18,6 @@ package org.thingsboard.server.dao.sql.alarm; import com.google.common.util.concurrent.ListenableFuture; import org.junit.Test; import org.springframework.beans.factory.annotation.Autowired; -import org.springframework.data.domain.PageRequest; -import org.thingsboard.server.common.data.EntityType; import org.thingsboard.server.common.data.alarm.Alarm; import org.thingsboard.server.common.data.alarm.AlarmId; import org.thingsboard.server.common.data.alarm.AlarmStatus; @@ -27,7 +25,6 @@ import org.thingsboard.server.common.data.id.DeviceId; import org.thingsboard.server.common.data.id.TenantId; import org.thingsboard.server.dao.AbstractJpaDaoTest; import org.thingsboard.server.dao.alarm.AlarmDao; -import org.thingsboard.server.dao.model.sql.AlarmEntity; import java.util.UUID; import java.util.concurrent.ExecutionException; @@ -43,8 +40,6 @@ public class JpaAlarmDaoTest extends AbstractJpaDaoTest { @Autowired private AlarmDao alarmDao; - @Autowired - private AlarmRepository alarmRepository; @Test public void testFindLatestByOriginatorAndType() throws ExecutionException, InterruptedException { @@ -59,9 +54,6 @@ public class JpaAlarmDaoTest extends AbstractJpaDaoTest { saveAlarm(alarm2Id, tenantId, originator1Id, "TEST_ALARM"); saveAlarm(alarm3Id, tenantId, originator2Id, "TEST_ALARM"); assertEquals(3, alarmDao.find().size()); - AlarmEntity alarmEntity = alarmRepository.findLatestByOriginatorAndType( - tenantId, originator1Id, EntityType.DEVICE, "TEST_ALARM", new PageRequest(0, 1)).get(0); - assertNotNull(alarmEntity); ListenableFuture future = alarmDao .findLatestByOriginatorAndType(new TenantId(tenantId), new DeviceId(originator1Id), "TEST_ALARM"); Alarm alarm = future.get(); @@ -69,11 +61,6 @@ public class JpaAlarmDaoTest extends AbstractJpaDaoTest { assertEquals(alarm2Id, alarm.getId().getId()); } - @Test - public void testFindAlarmByIdAsync() { - // TODO: implement - } - private void saveAlarm(UUID id, UUID tenantId, UUID deviceId, String type) { Alarm alarm = new Alarm(); alarm.setId(new AlarmId(id)); diff --git a/dao/src/test/resources/dbunit/device_credentials.xml b/dao/src/test/resources/dbunit/device_credentials.xml index 01ccd6615b..5f813e26d9 100644 --- a/dao/src/test/resources/dbunit/device_credentials.xml +++ b/dao/src/test/resources/dbunit/device_credentials.xml @@ -1,14 +1,14 @@ \ No newline at end of file diff --git a/dao/src/test/resources/dbunit/rule.xml b/dao/src/test/resources/dbunit/rule.xml index 5040ff6941..dfe1054785 100644 --- a/dao/src/test/resources/dbunit/rule.xml +++ b/dao/src/test/resources/dbunit/rule.xml @@ -1,31 +1,31 @@ - - - - -