From 9e477afcec642e143c88c0261b554c3c17ef6dd3 Mon Sep 17 00:00:00 2001
From: Timo Pagel <github@timo-pagel.de>
Date: Sun, 14 Jan 2024 11:46:46 +0100
Subject: [PATCH] fix: do not clone in Init

---
 .../analyzer/deserialization/ApplicationDirector.java     | 3 +--
 .../metricca/analyzer/deserialization/YamlScanner.java    | 8 ++------
 .../DatePeriodWithSecurityRequirementsTest.java           | 1 +
 3 files changed, 4 insertions(+), 8 deletions(-)

diff --git a/src/main/java/org/owasp/dsomm/metricca/analyzer/deserialization/ApplicationDirector.java b/src/main/java/org/owasp/dsomm/metricca/analyzer/deserialization/ApplicationDirector.java
index daa250a..fe2cefa 100644
--- a/src/main/java/org/owasp/dsomm/metricca/analyzer/deserialization/ApplicationDirector.java
+++ b/src/main/java/org/owasp/dsomm/metricca/analyzer/deserialization/ApplicationDirector.java
@@ -64,9 +64,8 @@ private void initiateApplications(boolean enforceGitCloneIfTargetFolderExists) t
   }
 
   private List<SkeletonActivity> getDeserializeSkeletons(boolean enforceGitCloneIfTargetFolderExists) throws IOException, GitAPIException {
-    logger.info("yamlConfigurationFilePath: " + yamlScanner.getSkeletonYaml(enforceGitCloneIfTargetFolderExists));
-
     ObjectMapper mapper = new ObjectMapper(new YAMLFactory());
+    yamlScanner.initiate(enforceGitCloneIfTargetFolderExists);
     Map<?, ?> yamlActivityFileMap = YamlReader.convertYamlToJavaYaml(yamlScanner.getSkeletonYaml().getPath());
     String skeletonString = mapper.writeValueAsString(yamlActivityFileMap.get("activity definitions"));
     logger.info("skeletonString: " + skeletonString);
diff --git a/src/main/java/org/owasp/dsomm/metricca/analyzer/deserialization/YamlScanner.java b/src/main/java/org/owasp/dsomm/metricca/analyzer/deserialization/YamlScanner.java
index 39fbb07..486f2ba 100644
--- a/src/main/java/org/owasp/dsomm/metricca/analyzer/deserialization/YamlScanner.java
+++ b/src/main/java/org/owasp/dsomm/metricca/analyzer/deserialization/YamlScanner.java
@@ -85,8 +85,8 @@ public Collection<File> getApplicationYamls() throws IOException, GitAPIExceptio
           .collect(Collectors.toList());
     }
   }
-  public File getSkeletonYaml(boolean enforceGitCloneIfTargetFolderExists) throws IOException, GitAPIException {
-    this.initiate(enforceGitCloneIfTargetFolderExists);
+  public File getSkeletonYaml() throws IOException, GitAPIException {
+    this.initiate(false);
     logger.info("getYamlSkeletonFilePath() " + getYamlSkeletonFilePath());
     File skeletonConfig = new File(getYamlSkeletonFilePath());
     if (!skeletonConfig.exists()) throw new FileNotFoundException(getYamlSkeletonFilePath());
@@ -94,10 +94,6 @@ public File getSkeletonYaml(boolean enforceGitCloneIfTargetFolderExists) throws
     return skeletonConfig;
   }
 
-  public File getSkeletonYaml() throws IOException, GitAPIException {
-    return getSkeletonYaml(false);
-  }
-
   private boolean isGit(boolean enforceGitCloneIfTargetFolderExists) {
     if (yamlGitUrl == null || yamlGitUrl.isEmpty()) {
       return false;
diff --git a/src/test/java/org/owasp/dsomm/metricca/analyzer/deserialization/DatePeriodWithSecurityRequirementsTest.java b/src/test/java/org/owasp/dsomm/metricca/analyzer/deserialization/DatePeriodWithSecurityRequirementsTest.java
index a880d98..da7bcb5 100644
--- a/src/test/java/org/owasp/dsomm/metricca/analyzer/deserialization/DatePeriodWithSecurityRequirementsTest.java
+++ b/src/test/java/org/owasp/dsomm/metricca/analyzer/deserialization/DatePeriodWithSecurityRequirementsTest.java
@@ -39,6 +39,7 @@ public void setUp() throws Exception {
     constructor.setAccessible(true);
     applicationDirector = constructor.newInstance();
     YamlScanner yamlScanner = new YamlScanner();
+    yamlScanner.initiate(false);
     setPrivateField(yamlScanner, "yamlApplicationFolderPath", "src/test/resources/test-security-requirements/definitions");
     setPrivateField(yamlScanner, "yamlSkeletonFilePath", "src/test/resources/test-security-requirements/skeleton.yaml");
     yamlScanner.getApplicationYamls();