SONAR-8368 unset closeDate when reopening Closed issues

diff --git a/server/sonar-server-common/src/main/java/org/sonar/server/issue/workflow/IssueWorkflow.java b/server/sonar-server-common/src/main/java/org/sonar/server/issue/workflow/IssueWorkflow.java
index fb27f0d5853dbc6a969b450dedd814beead34fee..052acd43f9a7b3410e8f7aef8bf895dba49b1a6e 100644 (file)
--- a/server/sonar-server-common/src/main/java/org/sonar/server/issue/workflow/IssueWorkflow.java
+++ b/server/sonar-server-common/src/main/java/org/sonar/server/issue/workflow/IssueWorkflow.java
@@ -250,6 +250,7 @@ public class IssueWorkflow implements Startable {
           new PreviousStatusWas(Issue.STATUS_OPEN),
           new HasResolution(Issue.RESOLUTION_REMOVED, Issue.RESOLUTION_FIXED),
           IsNotHotspotNorManualVulnerability.INSTANCE)
+        .functions(UnsetCloseDate.INSTANCE)
         .automatic()
         .build())
       .transition(Transition.builder("automaticunclosereopen")
@@ -258,6 +259,7 @@ public class IssueWorkflow implements Startable {
           new PreviousStatusWas(Issue.STATUS_REOPENED),
           new HasResolution(Issue.RESOLUTION_REMOVED, Issue.RESOLUTION_FIXED),
           IsNotHotspotNorManualVulnerability.INSTANCE)
+        .functions(UnsetCloseDate.INSTANCE)
         .automatic()
         .build())
       .transition(Transition.builder("automaticuncloseconfirmed")
@@ -266,6 +268,7 @@ public class IssueWorkflow implements Startable {
           new PreviousStatusWas(Issue.STATUS_CONFIRMED),
           new HasResolution(Issue.RESOLUTION_REMOVED, Issue.RESOLUTION_FIXED),
           IsNotHotspotNorManualVulnerability.INSTANCE)
+        .functions(UnsetCloseDate.INSTANCE)
         .automatic()
         .build())
       .transition(Transition.builder("automaticuncloseresolved")
@@ -274,6 +277,7 @@ public class IssueWorkflow implements Startable {
           new PreviousStatusWas(Issue.STATUS_RESOLVED),
           new HasResolution(Issue.RESOLUTION_REMOVED, Issue.RESOLUTION_FIXED),
           IsNotHotspotNorManualVulnerability.INSTANCE)
+        .functions(UnsetCloseDate.INSTANCE)
         .automatic()
         .build())
 

diff --git a/server/sonar-server-common/src/test/java/org/sonar/server/issue/workflow/IssueWorkflowTest.java b/server/sonar-server-common/src/test/java/org/sonar/server/issue/workflow/IssueWorkflowTest.java
index c515234fd08ace4f595f0eecb62b7f35dad77307..c848d05955977412f89c32cca535c39207d73e2f 100644 (file)
--- a/server/sonar-server-common/src/test/java/org/sonar/server/issue/workflow/IssueWorkflowTest.java
+++ b/server/sonar-server-common/src/test/java/org/sonar/server/issue/workflow/IssueWorkflowTest.java
@@ -201,6 +201,8 @@ public class IssueWorkflowTest {
 
       assertThat(issue.status()).isEqualTo(previousStatus);
       assertThat(issue.updateDate()).isEqualTo(DateUtils.truncate(now, Calendar.SECOND));
+      assertThat(issue.closeDate()).isNull();
+      assertThat(issue.isChanged()).isTrue();
     });
   }