Fix/warnings stderr

lib/runners/java.js

@@ -75,17 +75,30 @@ module.exports.run = function run(opts, cb) {
         }
       });
 
+      buffer.stderr += stderr;
+      // let's not show the noisy "Try:" text when there are build errors
+      buffer.stderr = buffer.stderr.split(/\^* Try:/m)[0];
+
       // attach build output to beginning of stdout, also attach dependencies so everyone knows whats available
       // TODO: eventually we want to support this as its own buffer property, for now we are just embedding it
       // within the output in case its helpful for troubleshooting
       let buildOutput = `Dependencies:\n\n${loadedDependencies()}\n\nTasks:\n\n`;
       buildOutput += buildLines.join("\n").replace(/^Start > start.*/gm, '');
+
+      // lets strip out any STDERR compiler warnings and and save them for later
+      const notes = [];
+      buffer.stderr = buffer.stderr.replace(/^Note:.*\n/gm, function(t) {
+        notes.push(t);
+        return '';
+      });
+
+      // now lets move those warnings to the build output
+      if (notes.length) {
+        buildOutput += `\n\nCompiler Warnings:\n\n${notes.join("\n")}`;
+      }
+
       buildOutput = tagHelpers.log('-Build Output', buildOutput);
       buffer.stdout = buildOutput + stdout;
-
-      buffer.stderr += stderr;
-      // let's not show the noisy "Try:" text when there are build errors
-      buffer.stderr = buffer.stderr.split(/\^* Try:/m)[0];
     }
   });
 

test/runners/java_spec.js

@@ -249,6 +249,40 @@ describe('java runner', function() {
     });
   });
 
+  describe('compiler warnings as STDERR messages', function() {
+    it('should handle unchecked or unsafe operations', function(done) {
+      runner.run({
+        language: 'java',
+        code: `
+          import java.util.ArrayList;
+
+          public class Example {
+
+              public static String[] dirReduc(String[] arr) {
+                  ArrayList<String> result = new ArrayList();
+                  return result.toArray(new String[result.size()]);
+              }
+          }
+        `,
+        fixture: `
+          import org.junit.Test;
+          import static org.junit.Assert.assertEquals;
+
+          public class ExampleTest {
+            @Test
+            public void testRandomDirReduc() throws Exception {
+              assertEquals("I always pass!", 1, 1);
+            }
+          }`
+      }, function(buffer) {
+        console.log(`|${buffer.stderr}|`);
+        expect(buffer.stderr).to.equal('');
+        done();
+      });
+
+    });
+  });
+
   describe('spring', function() {
     it('should handle basic junit tests', function(done) {
       runner.run({