junit5: BaseQueueServiceTest annotation exceptions refactoring

This commit is contained in:
Oleksandra Matviienko 2023-03-06 13:34:22 +01:00
parent 25e046dc79
commit 8c253ec352

View File

@ -19,6 +19,7 @@ import org.junit.After;
import org.junit.Assert; import org.junit.Assert;
import org.junit.Before; import org.junit.Before;
import org.junit.Test; import org.junit.Test;
import org.junit.jupiter.api.Assertions;
import org.thingsboard.server.common.data.DataConstants; import org.thingsboard.server.common.data.DataConstants;
import org.thingsboard.server.common.data.Tenant; import org.thingsboard.server.common.data.Tenant;
import org.thingsboard.server.common.data.TenantProfile; import org.thingsboard.server.common.data.TenantProfile;
@ -139,7 +140,7 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queueService.deleteQueue(tenantId, foundQueue.getId()); queueService.deleteQueue(tenantId, foundQueue.getId());
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithEmptyName() { public void testSaveQueueWithEmptyName() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -149,10 +150,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setPackProcessingTimeout(2000); queue.setPackProcessingTimeout(2000);
queue.setSubmitStrategy(createTestSubmitStrategy()); queue.setSubmitStrategy(createTestSubmitStrategy());
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithInvalidName() { public void testSaveQueueWithInvalidName() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -163,10 +166,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setPackProcessingTimeout(2000); queue.setPackProcessingTimeout(2000);
queue.setSubmitStrategy(createTestSubmitStrategy()); queue.setSubmitStrategy(createTestSubmitStrategy());
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithEmptyTopic() { public void testSaveQueueWithEmptyTopic() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -176,10 +181,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setPackProcessingTimeout(2000); queue.setPackProcessingTimeout(2000);
queue.setSubmitStrategy(createTestSubmitStrategy()); queue.setSubmitStrategy(createTestSubmitStrategy());
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithInvalidTopic() { public void testSaveQueueWithInvalidTopic() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -190,10 +197,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setPackProcessingTimeout(2000); queue.setPackProcessingTimeout(2000);
queue.setSubmitStrategy(createTestSubmitStrategy()); queue.setSubmitStrategy(createTestSubmitStrategy());
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithEmptyPollInterval() { public void testSaveQueueWithEmptyPollInterval() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -203,10 +212,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setPackProcessingTimeout(2000); queue.setPackProcessingTimeout(2000);
queue.setSubmitStrategy(createTestSubmitStrategy()); queue.setSubmitStrategy(createTestSubmitStrategy());
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithEmptyPartitions() { public void testSaveQueueWithEmptyPartitions() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -216,10 +227,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setPackProcessingTimeout(2000); queue.setPackProcessingTimeout(2000);
queue.setSubmitStrategy(createTestSubmitStrategy()); queue.setSubmitStrategy(createTestSubmitStrategy());
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithEmptyPackProcessingTimeout() { public void testSaveQueueWithEmptyPackProcessingTimeout() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -229,10 +242,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setPartitions(1); queue.setPartitions(1);
queue.setSubmitStrategy(createTestSubmitStrategy()); queue.setSubmitStrategy(createTestSubmitStrategy());
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithEmptySubmitStrategy() { public void testSaveQueueWithEmptySubmitStrategy() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -242,10 +257,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setPartitions(1); queue.setPartitions(1);
queue.setPackProcessingTimeout(2000); queue.setPackProcessingTimeout(2000);
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithEmptyProcessingStrategy() { public void testSaveQueueWithEmptyProcessingStrategy() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -255,10 +272,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setPartitions(1); queue.setPartitions(1);
queue.setPackProcessingTimeout(2000); queue.setPackProcessingTimeout(2000);
queue.setSubmitStrategy(createTestSubmitStrategy()); queue.setSubmitStrategy(createTestSubmitStrategy());
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithEmptySubmitStrategyType() { public void testSaveQueueWithEmptySubmitStrategyType() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -270,10 +289,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setSubmitStrategy(createTestSubmitStrategy()); queue.setSubmitStrategy(createTestSubmitStrategy());
queue.getSubmitStrategy().setType(null); queue.getSubmitStrategy().setType(null);
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithEmptySubmitStrategyBatchSize() { public void testSaveQueueWithEmptySubmitStrategyBatchSize() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -286,10 +307,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.getSubmitStrategy().setType(SubmitStrategyType.BATCH); queue.getSubmitStrategy().setType(SubmitStrategyType.BATCH);
queue.getSubmitStrategy().setBatchSize(0); queue.getSubmitStrategy().setBatchSize(0);
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithEmptyProcessingStrategyType() { public void testSaveQueueWithEmptyProcessingStrategyType() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -301,10 +324,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setSubmitStrategy(createTestSubmitStrategy()); queue.setSubmitStrategy(createTestSubmitStrategy());
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
queue.getProcessingStrategy().setType(null); queue.getProcessingStrategy().setType(null);
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithNegativeProcessingStrategyRetries() { public void testSaveQueueWithNegativeProcessingStrategyRetries() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -316,10 +341,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setSubmitStrategy(createTestSubmitStrategy()); queue.setSubmitStrategy(createTestSubmitStrategy());
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
queue.getProcessingStrategy().setRetries(-1); queue.getProcessingStrategy().setRetries(-1);
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithNegativeProcessingStrategyFailurePercentage() { public void testSaveQueueWithNegativeProcessingStrategyFailurePercentage() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -331,10 +358,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setSubmitStrategy(createTestSubmitStrategy()); queue.setSubmitStrategy(createTestSubmitStrategy());
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
queue.getProcessingStrategy().setFailurePercentage(-1); queue.getProcessingStrategy().setFailurePercentage(-1);
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithNegativeProcessingStrategyPauseBetweenRetries() { public void testSaveQueueWithNegativeProcessingStrategyPauseBetweenRetries() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -346,10 +375,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setSubmitStrategy(createTestSubmitStrategy()); queue.setSubmitStrategy(createTestSubmitStrategy());
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
queue.getProcessingStrategy().setPauseBetweenRetries(-1); queue.getProcessingStrategy().setPauseBetweenRetries(-1);
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithProcessingStrategyPauseBetweenRetriesBiggerThenMaxPauseBetweenRetries() { public void testSaveQueueWithProcessingStrategyPauseBetweenRetriesBiggerThenMaxPauseBetweenRetries() {
Queue queue = new Queue(); Queue queue = new Queue();
queue.setTenantId(tenantId); queue.setTenantId(tenantId);
@ -361,10 +392,12 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setSubmitStrategy(createTestSubmitStrategy()); queue.setSubmitStrategy(createTestSubmitStrategy());
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
queue.getProcessingStrategy().setPauseBetweenRetries(100); queue.getProcessingStrategy().setPauseBetweenRetries(100);
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} }
@Test(expected = DataValidationException.class) @Test
public void testSaveQueueWithNotIsolatedTenant() { public void testSaveQueueWithNotIsolatedTenant() {
Tenant tenant = new Tenant(); Tenant tenant = new Tenant();
tenant.setTitle("Not isolated tenant"); tenant.setTitle("Not isolated tenant");
@ -381,7 +414,9 @@ public abstract class BaseQueueServiceTest extends AbstractServiceTest {
queue.setSubmitStrategy(createTestSubmitStrategy()); queue.setSubmitStrategy(createTestSubmitStrategy());
queue.setProcessingStrategy(createTestProcessingStrategy()); queue.setProcessingStrategy(createTestProcessingStrategy());
try { try {
Assertions.assertThrows(DataValidationException.class, () -> {
queueService.saveQueue(queue); queueService.saveQueue(queue);
});
} finally { } finally {
tenantService.deleteTenant(savedTenant.getId()); tenantService.deleteTenant(savedTenant.getId());
} }