fix: no double quote and header for YAML

Author: kkweon

app/src/main/java/com/github/codingpot/github_org_member_manage_action/config/ConfigManager.java

@@ -2,6 +2,7 @@
 
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.dataformat.yaml.YAMLFactory;
+import com.fasterxml.jackson.dataformat.yaml.YAMLGenerator;
 import com.github.codingpot.github_org_member_manage_action.context.Context;
 import java.io.File;
 import java.io.IOException;
@@ -32,7 +33,11 @@ public void write(ConfigData data) throws IOException {
             return;
         }
         var filepath = this.context.getMembersFilePath();
-        ObjectMapper mapper = new ObjectMapper(new YAMLFactory());
+        ObjectMapper mapper =
+                new ObjectMapper(
+                        new YAMLFactory()
+                                .enable(YAMLGenerator.Feature.MINIMIZE_QUOTES)
+                                .disable(YAMLGenerator.Feature.WRITE_DOC_START_MARKER));
         mapper.writeValue(new File(filepath), data);
     }
 }

app/src/test/java/com/github/codingpot/github_org_member_manage_action/ConfigManagerTest.java

@@ -1,6 +1,7 @@
 package com.github.codingpot.github_org_member_manage_action;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
 
 import com.github.codingpot.github_org_member_manage_action.config.ConfigData;
 import com.github.codingpot.github_org_member_manage_action.config.ConfigManager;
@@ -25,7 +26,7 @@ void setUp() throws IOException {
         configManager =
                 new ConfigManager(
                         new Context(
-                                path.toUri().getPath(), "FAKE_GITHUB_TOKEN", true, AppMode.SYNC));
+                                path.toUri().getPath(), "FAKE_GITHUB_TOKEN", false, AppMode.SYNC));
 
         String yaml =
                 "org_name: orgName"
@@ -61,4 +62,21 @@ void readFromLocal() throws IOException {
                         .build(),
                 configData);
     }
+
+    @Test
+    void testYamlWriteDoesNotContainsUnnecessaryThings() throws IOException {
+        configManager.write(
+                ConfigData.builder()
+                        .orgName("orgName")
+                        .admins(Set.of("admin1"))
+                        .members(Set.of("member1"))
+                        .build());
+        final String s = Files.readString(path);
+
+        // should not double quote.
+        assertFalse(s.contains("\""));
+
+        // should not add a start header.
+        assertFalse(s.contains("---"));
+    }
 }