changes for showing option to restart or recompile when a change is …

…detected in build.gradle

Signed-off-by: Arun Venmany <[email protected]>

src/main/groovy/io/openliberty/tools/gradle/tasks/DevTask.groovy

@@ -499,13 +499,23 @@ class DevTask extends AbstractFeatureTask {
         @Override
         public boolean updateArtifactPaths(ProjectModule projectModule, boolean redeployCheck, boolean generateFeatures, ThreadPoolExecutor executor)
                 throws PluginExecutionException {
-            // not supported for Gradle, only used for multi module Maven projects
+            // unable to identify the changes made, showing option for user. always return false because action is invoked manually
+            if (isMultiModuleProject()) {
+                info("We detected a change in build.gradle, but we cannot identify whether it’s a runtime or dependency change");
+                info("r - to restart server, type 'r' and press Enter.");
+                info("Press Enter to recompile the project and run tests manually");
+            }
             return false;
         }
 
         @Override
         public boolean updateArtifactPaths(File parentBuildFile) {
-            // not supported for Gradle, only used for multi module Maven projects
+            // unable to identify the changes made, showing option for user. always return false because action is invoked manually
+            if (isMultiModuleProject()) {
+                info("We detected a change in build.gradle, but we cannot identify whether it’s a runtime or dependency change");
+                info("r - to restart server, type 'r' and press Enter.");
+                info("Press Enter to recompile the project and run tests manually");
+            }
             return false;
         }
 
@@ -535,6 +545,13 @@ class DevTask extends AbstractFeatureTask {
             boolean installFeatures = false;
             boolean optimizeGenerateFeatures = false;
 
+            if (isMultiModuleProject()) {
+                info("We detected a change in build.gradle, but we cannot identify whether it’s a runtime or dependency change");
+                info("r - to restart server, type 'r' and press Enter.");
+                info("Press Enter to recompile the project and run tests manually");
+                return false;
+            }
+
             ProjectBuilder builder = ProjectBuilder.builder();
             Project newProject;
             try {

src/test/groovy/io/openliberty/tools/gradle/TestMultiModuleLooseEarAppDevMode.groovy

@@ -56,6 +56,24 @@ class TestMultiModuleLooseEarAppDevMode extends BaseDevTest {
 
     }
 
+    @Test
+    public void modifyUpdateGradleTest() throws Exception {
+
+        // modify a java file
+        File srcHelloWorld = new File(buildDir, "build.gradle");
+        assertTrue(srcHelloWorld.exists());
+
+        waitLongEnough();
+        String str = "// testing";
+        BufferedWriter javaWriter = new BufferedWriter(new FileWriter(srcHelloWorld, true));
+        javaWriter.append(' ');
+        javaWriter.append(str);
+        javaWriter.close();
+        if (!verifyLogMessage(2000,  "We detected a change in build.gradle, but we cannot identify whether it’s a runtime or dependency change")) {
+            assertTrue(verifyLogMessage(2000,  "We detected a change in build.gradle, but we cannot identify whether it’s a runtime or dependency change"));
+        }
+    }
+
     @AfterClass
     public static void cleanUpAfterClass() throws Exception {
         String stdout = getContents(logFile, "Dev mode std output");
@@ -64,4 +82,5 @@ class TestMultiModuleLooseEarAppDevMode extends BaseDevTest {
         System.out.println(stderr);
         cleanUpAfterClass(true);
     }
+
 }