Merge pull request #176 from conductor-oss/lpeng/fix-set-workflow-tags-method

Fix setWorkflowTags to send PUT

Author: ling-senpeng13

conductor-client/src/main/java/io/orkes/conductor/client/http/TagsResource.java

@@ -128,7 +128,7 @@ public void setTaskTags(List<TagObject> tagObjects, String taskName) {
 
     public void setWorkflowTags(List<TagObject> tagObjects, String workflow) {
         ConductorClientRequest request = ConductorClientRequest.builder()
-                .method(Method.GET)
+                .method(Method.PUT)
                 .path("/metadata/workflow/{workflow}/tags")
                 .addPathParam("workflow", workflow)
                 .body(tagObjects)

tests/src/test/java/io/orkes/conductor/client/http/MetadataClientTests.java

@@ -108,9 +108,11 @@ void tagWorkflow() {
             }
         }
         metadataClient.addWorkflowTag(tagObject, Commons.WORKFLOW_NAME);
-        metadataClient.setWorkflowTags(List.of(tagObject), Commons.WORKFLOW_NAME);
+        // A different tag, so the assertion fails unless setWorkflowTags replaces the existing one.
+        TagObject replacement = new TagObject().key(tagObject.getKey()).value(tagObject.getValue() + "-replaced");
+        metadataClient.setWorkflowTags(List.of(replacement), Commons.WORKFLOW_NAME);
         List<TagObject> tags = metadataClient.getWorkflowTags(Commons.WORKFLOW_NAME);
-        Assertions.assertIterableEquals(List.of(tagObject), tags);
+        Assertions.assertIterableEquals(List.of(replacement), tags);
     }
 
     // ==================== Additional CRUD Tests ====================