Mailing-List: contact commits-help@buildr.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@buildr.apache.org
Message-ID: <882375769.1252262877921.JavaMail.jira@brutus>
Date: Sun, 6 Sep 2009 11:47:57 -0700 (PDT)
From: "Jeremie Lenfant-Engelmann (JIRA)" <jira@apache.org>
To: buildr-commits@incubator.apache.org
Subject: [jira] Created: (BUILDR-307) Failures not reported correclty for
 ScalaTest (fix included)
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

Failures not reported correclty for ScalaTest (fix included)
------------------------------------------------------------

                 Key: BUILDR-307
                 URL: https://issues.apache.org/jira/browse/BUILDR-307
             Project: Buildr
          Issue Type: Bug
          Components: Test frameworks
    Affects Versions: 1.3.4, 1.3.5
         Environment: all
            Reporter: Jeremie Lenfant-Engelmann
             Fix For: 1.3.5, 1.3.4


Today when a ScalaTest fails Buildr doesn't report it correctly and just ends without reporting the correct error or returning an error code.

There is 2 problems:
- The regexp used to now if a test failed is wrong
- The loop reading the reportFile breaks too early, it breaks on the sentence 'Run completed.' but 'TEST FAILED' is on the following line

Here is a fix for it:

Index: lib/buildr/scala/tests.rb
===================================================================
--- lib/buildr/scala/tests.rb	(revision 811837)
+++ lib/buildr/scala/tests.rb	(working copy)
@@ -126,9 +126,9 @@
         while (!completed) do
           File.open(reportFile, "r") do |input|
             while (line = input.gets) do
-              failed = (line =~ /(TEST FAILED -)|(RUN STOPPED)|(RUN ABORTED)/) unless failed
+              failed = (line =~ /(TESTS? FAILED)|(RUN STOPPED)|(RUN ABORTED)/) unless failed
               completed |= (line =~ /Run completed\./)
-              break if (failed || completed)
+              break if (failed)
             end
           end
           wait += 1


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.