diff options
author | Simon Brandhof <simon.brandhof@sonarsource.com> | 2016-11-10 12:02:26 +0100 |
---|---|---|
committer | Simon Brandhof <simon.brandhof@sonarsource.com> | 2016-11-14 12:18:51 +0100 |
commit | feda9c29f81ed7a1628afb9fbb5e47824f2f0c35 (patch) | |
tree | 3ed1a95241090a13c021ef150d650928cca9c504 /server | |
parent | 34cdbe891e9b45f42d35ee6deb2776cdfe50b431 (diff) | |
download | sonarqube-feda9c29f81ed7a1628afb9fbb5e47824f2f0c35.tar.gz sonarqube-feda9c29f81ed7a1628afb9fbb5e47824f2f0c35.zip |
SONAR-8353 purge webhook deliveries when deleting project
Diffstat (limited to 'server')
-rw-r--r-- | server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/webhook/WebhookDeliveryStorageTest.java | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/webhook/WebhookDeliveryStorageTest.java b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/webhook/WebhookDeliveryStorageTest.java index 5b5fe6e999e..145a411f5bc 100644 --- a/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/webhook/WebhookDeliveryStorageTest.java +++ b/server/sonar-server/src/test/java/org/sonar/server/computation/task/projectanalysis/webhook/WebhookDeliveryStorageTest.java @@ -20,8 +20,6 @@ package org.sonar.server.computation.task.projectanalysis.webhook; import java.io.IOException; -import java.util.List; -import java.util.Map; import org.junit.Rule; import org.junit.Test; import org.sonar.api.utils.System2; @@ -31,10 +29,11 @@ import org.sonar.db.DbSession; import org.sonar.db.DbTester; import org.sonar.db.webhook.WebhookDeliveryDto; -import static org.apache.commons.lang.RandomStringUtils.randomAlphanumeric; import static org.assertj.core.api.Assertions.assertThat; import static org.mockito.Mockito.mock; import static org.mockito.Mockito.when; +import static org.sonar.db.webhook.WebhookDbTesting.newWebhookDeliveryDto; +import static org.sonar.db.webhook.WebhookDbTesting.selectAllDeliveryUuids; public class WebhookDeliveryStorageTest { @@ -96,9 +95,8 @@ public class WebhookDeliveryStorageTest { underTest.purge("PROJECT_1"); - List<Map<String, Object>> uuids = dbTester.select(dbSession, "select uuid as \"uuid\" from webhook_deliveries"); // do not purge another project PROJECT_2 - assertThat(uuids).extracting(column -> column.get("uuid")).containsOnly("D2", "D3"); + assertThat(selectAllDeliveryUuids(dbTester, dbSession)).containsOnly("D2", "D3"); } private static WebhookDelivery.Builder newBuilderTemplate() { @@ -111,14 +109,9 @@ public class WebhookDeliveryStorageTest { } private static WebhookDeliveryDto newDto(String uuid, String componentUuid, long at) { - return new WebhookDeliveryDto() + return newWebhookDeliveryDto() .setUuid(uuid) .setComponentUuid(componentUuid) - .setCeTaskUuid(randomAlphanumeric(40)) - .setName("Jenkins") - .setUrl("http://jenkins") - .setSuccess(true) - .setPayload("{json}") .setCreatedAt(at); } } |