buildr-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Boisvert (JIRA)" <j...@apache.org>
Subject [jira] Updated: (BUILDR-350) CLONE -Failures not reported correclty for ScalaTest (fix included)
Date Thu, 26 Nov 2009 16:08:39 GMT

     [ https://issues.apache.org/jira/browse/BUILDR-350?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Alex Boisvert updated BUILDR-350:
---------------------------------

    Fix Version/s:     (was: 1.3.5)
                   1.4

> CLONE -Failures not reported correclty for ScalaTest (fix included)
> -------------------------------------------------------------------
>
>                 Key: BUILDR-350
>                 URL: https://issues.apache.org/jira/browse/BUILDR-350
>             Project: Buildr
>          Issue Type: Bug
>          Components: Test frameworks
>    Affects Versions: 1.3.4, 1.3.5
>         Environment: all
>            Reporter: Alex Eagle
>            Assignee: Daniel Spiewak
>             Fix For: 1.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.


Mime
View raw message