Merge pull request #4 from StringCare/develop

Develop

Author: efraespada

README.md

@@ -14,7 +14,7 @@ root_project/build.gradle
 buildscript {
 
     ext {
-        stringcare_version = '0.4'
+        stringcare_version = '0.5'
     }
 
     repositories {

build.gradle

@@ -24,7 +24,7 @@ apply plugin: 'java'
 apply plugin: 'com.jfrog.bintray'
 
 group 'com.stringcare'
-version '0.4'
+version '0.5'
 
 sourceCompatibility = 1.8
 

src/main/groovy/CredentialUtils.java

@@ -75,7 +75,8 @@ private static void parseTrace(String module, String variant, String line, boole
                     until = key;
                 }
             } else if (debug){
-                PrintUtils.print(module, "Variant:" + variant, true);
+                PrintUtils.print(module, "Module: " + module, true);
+                PrintUtils.print(module, "Variant: " + variant, true);
             }
 
         } else if (line.toLowerCase().contains("sha") && moduleLocated && variantLocated) {

src/main/groovy/GradleHandlerCallback.java

@@ -1,6 +1,8 @@
 
 public interface GradleHandlerCallback {
 
+    boolean debug();
+
     void onDataFound(String module, String variant);
 
     void onMergeResourcesStarts(String module, String variant);

src/main/groovy/PrintUtils.java

@@ -20,6 +20,7 @@ public static void init(String module, String variant) {
     public static void print(String message) {
         print(message, false);
     }
+
     public static void print(String message, boolean tab) {
         if (variant != null && module != null) {
             if (!tab) {

src/main/groovy/StringCarePlugin.groovy

@@ -59,7 +59,13 @@ class StringCare implements Plugin<Project> {
                 }
             }
         }
-        this.project.gradle.addBuildListener(new TimingRecorder(this, new GradleHandlerCallback() {
+        this.project.gradle.addBuildListener(new TListener(this, new GradleHandlerCallback() {
+
+            @Override
+            boolean debug() {
+                return debug
+            }
+
             @Override
             void onDataFound(String module, String variant) {
                 // nothing to do here

src/main/groovy/TimingRecorder.groovy renamed to src/main/groovy/TaskListener.groovy

@@ -20,7 +20,7 @@ class Timing {
     }
 }
 
-class TimingRecorder extends BuildAndTaskExecutionListenerAdapter implements TaskExecutionListener {
+class TListener extends BuildAndTaskExecutionListenerAdapter implements TaskExecutionListener {
     private ClockT clock
     private List<Timing> timings = []
     private StringCare plugin
@@ -32,7 +32,7 @@ class TimingRecorder extends BuildAndTaskExecutionListenerAdapter implements Tas
     private static final String MERGE = "merge";
     private static final String RESOURCES = "Resources";
 
-    TimingRecorder(StringCare plugin, GradleHandlerCallback callback) {
+    TListener(StringCare plugin, GradleHandlerCallback callback) {
         this.plugin = plugin
         this.callback = callback
     }
@@ -41,16 +41,31 @@ class TimingRecorder extends BuildAndTaskExecutionListenerAdapter implements Tas
     void beforeExecute(Task task) {
         clock = new ClockT()
         if (task.name.contains(PRE) && task.name.contains(BUILD) && !task.name.equals(PRE + BUILD) && !task.name.contains(TEST)) {
-            callback.onDataFound(task.project.name, PrintUtils.uncapitalize(task.name.substring(PRE.length()).substring(0, task.name.substring(PRE.length()).length() - BUILD.length())));
+            String module = getName(task)
+            if (module != null) {
+                callback.onDataFound(module, PrintUtils.uncapitalize(task.name.substring(PRE.length()).substring(0, task.name.substring(PRE.length()).length() - BUILD.length())));
+            }
         } else if (task.name.contains(MERGE) && task.name.contains(RESOURCES) && !task.name.contains(TEST)) {
-            callback.onMergeResourcesStarts(task.project.name, PrintUtils.uncapitalize(task.name.substring(MERGE.length()).substring(0, task.name.substring(MERGE.length()).length() - RESOURCES.length())));
+            String module = getName(task)
+            if (module != null) {
+                if (callback.debug()) {
+                    // PrintUtils.print(module, "Module: " + module, true)
+                }
+                callback.onMergeResourcesStarts(module, PrintUtils.uncapitalize(task.name.substring(MERGE.length()).substring(0, task.name.substring(MERGE.length()).length() - RESOURCES.length())));
+
+            } else {
+                PrintUtils.print("not_needed", "🤖 module path not found, report an issue", true)
+            }
         }
     }
 
     @Override
     void afterExecute(Task task, TaskState taskState) {
         if (task.name.contains(MERGE) && task.name.contains(RESOURCES) && !task.name.contains(TEST)) {
-            callback.onMergeResourcesFinish(task.project.name, PrintUtils.uncapitalize(task.name.substring(MERGE.length()).substring(0, task.name.substring(MERGE.length()).length() - RESOURCES.length())));
+            String module = getName(task)
+            if (module != null) {
+                callback.onMergeResourcesFinish(module, PrintUtils.uncapitalize(task.name.substring(MERGE.length()).substring(0, task.name.substring(MERGE.length()).length() - RESOURCES.length())));
+            }
         }
         timings << new Timing(
                 clock.getTimeInMs(),
@@ -73,4 +88,10 @@ class TimingRecorder extends BuildAndTaskExecutionListenerAdapter implements Tas
     Timing getTiming(String path) {
         timings.find { it.path == path }
     }
+
+    String getName(Task task) {
+        String path = task.project.getPath()
+        return path == null || path.length() == 0 ? null : path.split(":")[path.split(":").length - 1]
+    }
+
 }