Merging tag list related mods from PR #172

Author: gmessner

src/main/java/org/gitlab4j/api/ProjectApi.java

@@ -649,10 +649,20 @@ public Project createProject(Project project, String importUrl) throws GitLabApi
         if (isApiVersion(ApiVersion.V3)) {
             boolean isPublic = (project.getPublic() != null ? project.getPublic() : project.getVisibility() == Visibility.PUBLIC);
             formData.withParam("public", isPublic);
+            
+            if (project.getTagList() != null && !project.getTagList().isEmpty()) {
+                // What would be the preferred way to deal with this, as the V3 API doesn't
+                // appear to do anything if you send in the tag_list? Could either just ignore,
+                // or throw an exception.
+            }
         } else {
             Visibility visibility = (project.getVisibility() != null ? project.getVisibility() :
                 project.getPublic() == Boolean.TRUE ? Visibility.PUBLIC : null);
             formData.withParam("visibility", visibility);
+            
+            if (project.getTagList() != null && !project.getTagList().isEmpty()) {
+                formData.withParam("tag_list", String.join(",", project.getTagList()));
+            }
         }
 
         if (project.getNamespace() != null) {
@@ -880,10 +890,20 @@ public Project updateProject(Project project) throws GitLabApiException {
             formData.withParam("visibility_level", project.getVisibilityLevel());
             boolean isPublic = (project.getPublic() != null ? project.getPublic() : project.getVisibility() == Visibility.PUBLIC);
             formData.withParam("public", isPublic);
+            
+            if (project.getTagList() != null && !project.getTagList().isEmpty()) {
+                // What would be the preferred way to deal with this, as the V3 API doesn't
+                // appear to do anything if you send in the tag_list? Could either just ignore,
+                // or throw an exception.
+            }
         } else {
             Visibility visibility = (project.getVisibility() != null ? project.getVisibility() :
                 project.getPublic() == Boolean.TRUE ? Visibility.PUBLIC : null);
             formData.withParam("visibility", visibility);
+            
+            if (project.getTagList() != null && !project.getTagList().isEmpty()) {
+                formData.withParam("tag_list", String.join(",", project.getTagList()));
+            }
         }
 
         Response response = putWithFormData(Response.Status.OK, formData, "projects", id);

src/test/java/org/gitlab4j/api/TestProjectApi.java

@@ -29,6 +29,7 @@
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assume.assumeTrue;
 
+import java.util.Arrays;
 import java.util.List;
 import java.util.Optional;
 
@@ -169,7 +170,8 @@ public void testCreate() throws GitLabApiException {
                 .withMergeRequestsEnabled(true)
                 .withWikiEnabled(true)
                 .withSnippetsEnabled(true)
-                .withVisibility(Visibility.PUBLIC);
+                .withVisibility(Visibility.PUBLIC)
+                .withTagList(Arrays.asList("tag1","tag2"));
 
         Project newProject = gitLabApi.getProjectApi().createProject(project);
         assertNotNull(newProject);
@@ -179,6 +181,7 @@ public void testCreate() throws GitLabApiException {
         assertEquals(project.getMergeRequestsEnabled(), newProject.getMergeRequestsEnabled());
         assertEquals(project.getWikiEnabled(), newProject.getWikiEnabled());
         assertEquals(project.getSnippetsEnabled(), newProject.getSnippetsEnabled());
+        assertEquals(project.getTagList(), newProject.getTagList());
         assertTrue(Visibility.PUBLIC == newProject.getVisibility() || Boolean.TRUE == newProject.getPublic());
     }
 
@@ -192,7 +195,8 @@ public void testUpdate() throws GitLabApiException {
                 .withMergeRequestsEnabled(true)
                 .withWikiEnabled(true)
                 .withSnippetsEnabled(true)
-                .withVisibility(Visibility.PUBLIC);
+                .withVisibility(Visibility.PUBLIC)
+                .withTagList(Arrays.asList("tag1","tag2"));
 
         Project newProject = gitLabApi.getProjectApi().createProject(project);
         assertNotNull(newProject);
@@ -202,6 +206,7 @@ public void testUpdate() throws GitLabApiException {
         assertEquals(project.getMergeRequestsEnabled(), newProject.getMergeRequestsEnabled());
         assertEquals(project.getWikiEnabled(), newProject.getWikiEnabled());
         assertEquals(project.getSnippetsEnabled(), newProject.getSnippetsEnabled());
+        assertEquals(project.getTagList(), newProject.getTagList());
         assertTrue(Visibility.PUBLIC == newProject.getVisibility() || Boolean.TRUE == newProject.getPublic());
 
         project = new Project()