add cache for tags

Author: Yang Guo

platform-api/src/main/java/com/flow/platform/api/controller/FlowController.java

@@ -271,7 +271,7 @@ public List<String> listBranches(@RequestParam(required = false) Boolean refresh
     @GetMapping("/{root}/tags")
     public List<String> listTags() {
         Node root = nodeService.find(currentNodePath.get());
-        return gitService.tags(root);
+        return gitService.tags(root, false);
     }
 
     /**

platform-api/src/main/java/com/flow/platform/api/service/GitService.java

@@ -66,7 +66,7 @@ interface ProgressListener {
     /**
      * Fetch tags from git repo
      */
-    List<String> tags(Node node);
+    List<String> tags(Node node, boolean refresh);
 
     /**
      * Fetch latest commit from git repo

platform-api/src/main/java/com/flow/platform/api/service/GitServiceImpl.java

@@ -113,7 +113,8 @@ public List<String> branches(Node node, boolean refresh) {
     }
 
     @Override
-    public List<String> tags(Node node) {
+    @Cacheable(value = "git.tags", key = "#node.getPath()", condition = "#refresh == false")
+    public List<String> tags(Node node, boolean refresh) {
         GitClient client = gitClientInstance(node);
         try {
             return client.tags();

platform-api/src/test/java/com/flow/platform/api/test/service/GitServiceTest.java

@@ -110,7 +110,7 @@ public void should_list_branches_of_git_repo() {
 
     @Test
     public void should_list_tags_of_git_repo() {
-        List<String> tags = gitService.tags(node);
+        List<String> tags = gitService.tags(node, false);
         Assert.assertNotNull(tags);
     }