allow hooks to be updated after archive if needed

Author: mcanoy

src/main/java/com/redhat/labs/lodestar/models/gitlab/HookConfig.java

@@ -26,5 +26,11 @@ public class HookConfig {
 
     @JsonbProperty("token")
     private String token;
+    
+    /**
+     * Should the webhook be enabled after an engagement is archived
+     */
+    @JsonbProperty("enabledAfterArchive")
+    private boolean enabledAfterArchive;
 
 }

src/main/java/com/redhat/labs/lodestar/service/ConfigService.java

@@ -122,7 +122,7 @@ void updateWebHooksInGitLab() {
         List<Project> projects = projectService.getProjectsByGroup(engagementRepositoryId, true);
         LOGGER.debug("number of projects found: {}", projects.size());
         projects.stream().filter(project -> project.getName().equals(IAC))
-                .filter(project -> !engagementIsArchived(project)).forEach(project -> {
+                .forEach(project -> {
 
                     Integer projectId = project.getId();
 
@@ -131,9 +131,12 @@ void updateWebHooksInGitLab() {
 
                     // remove existing webhooks for project
                     hookService.deleteProjectHooks(projectId);
+                    
+                    boolean isArchived = isEngagementArchived(project);
 
                     // create hooks from configuration
-                    hookConfigs.stream().forEach(hookC -> {
+                    hookConfigs.stream().filter(config -> config.isEnabledAfterArchive() || !isArchived)
+                        .forEach(hookC -> {
                         Hook hook = Hook.builder().projectId(projectId).pushEvents(true).url(hookC.getBaseUrl())
                                 .token(hookC.getToken()).build();
                         LOGGER.debug("\tcreating webhook {}", hook.getUrl());
@@ -154,7 +157,7 @@ void updateWebHooksInGitLab() {
      * @param project
      * @return
      */
-    boolean engagementIsArchived(Project project) {
+    boolean isEngagementArchived(Project project) {
 
         Optional<Engagement> engagement = engagementService.getEngagement(project, false, false);
         if (engagement.isPresent() && null != engagement.get().getArchiveDate()) {

src/test/java/com/redhat/labs/lodestar/model/ProjectTest.java

@@ -0,0 +1,32 @@
+package com.redhat.labs.lodestar.model;
+
+import java.util.ArrayList;
+
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.Test;
+
+import com.redhat.labs.lodestar.models.gitlab.Project;
+
+class ProjectTest {
+
+    @Test
+    void testPreserve() {
+
+        Project project = new Project();
+        Assertions.assertNull(project.getTagList());
+        
+        project.setTagList(new ArrayList<>());
+        Assertions.assertEquals(0, project.getTagList().size());
+
+        project.preserve();
+
+        Assertions.assertEquals(1, project.getTagList().size());
+        Assertions.assertEquals("DO_NOT_DELETE", project.getTagList().get(0));
+        
+        //Handle 2nd call properly
+        project.preserve();
+
+        Assertions.assertEquals(1, project.getTagList().size());
+        Assertions.assertEquals("DO_NOT_DELETE", project.getTagList().get(0));
+    }
+}

src/test/java/com/redhat/labs/lodestar/resource/ConfigResourceTest.java

@@ -122,13 +122,15 @@ void testGetHookFileSuccess() {
             .body(is("\n[\n" + 
                     "    {\n" + 
                     "        \"baseUrl\": \"https://labs.com/webhooks/\",\n" + 
+                    "        \"enabledAfterArchive\": false,\n" +
                     "        \"name\": \"labs\",\n" + 
                     "        \"pushEvent\": true,\n" + 
                     "        \"pushEventsBranchFilter\": \"master\",\n" + 
                     "        \"token\": \"abc\"\n" + 
                     "    },\n" + 
                     "    {\n" + 
                     "        \"baseUrl\": \"https://rht.com/hooks/\",\n" + 
+                    "        \"enabledAfterArchive\": true,\n" +
                     "        \"name\": \"rht\",\n" + 
                     "        \"pushEvent\": true,\n" + 
                     "        \"pushEventsBranchFilter\": \"master\",\n" + 

src/test/resources/webhooks.yaml

@@ -9,3 +9,4 @@
   pushEvent: true
   pushEventsBranchFilter: master
   token: def
+  enabledAfterArchive: true