Re: [PATCH] Output xml results for tests with compilation problems

[Omair Majid <omajid@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 354 bytes --]

Lillian Angel wrote:
> Omair Majid wrote:
>> The attached patch fixes mauve so that a failure is reported in the xml
>> file. 
> 
> 
> Feel free to commit this.
> 

Thanks. Applied patch as

2009-02-19  Omair Majid  <omajid@redhat.com>

     * RunnerProcess (runtest): If a test fails to start, include it in 
the xml report as a failure.

Cheers,
Omair

[-- Attachment #2: xmloutput.patch --]
[-- Type: text/plain, Size: 1189 bytes --]

? xmloutput-on-failed-to-compile-tests.patch
? xmloutput.patch
Index: ChangeLog
===================================================================
RCS file: /cvs/mauve/mauve/ChangeLog,v
retrieving revision 1.2136
diff -u -r1.2136 ChangeLog
--- ChangeLog	13 Feb 2009 15:51:58 -0000	1.2136
+++ ChangeLog	19 Feb 2009 19:37:11 -0000
@@ -1,3 +1,8 @@
+2009-02-19  Omair Majid  <omajid@redhat.com>
+
+	* RunnerProcess (runtest): If a test fails to start, include it in the xml
+	report as a failure.
+
 2009-02-13  Omair Majid  <omajid@redhat.com>
 
 	* Harness.java (setupHarness): Check for -xmlout file argument.
Index: RunnerProcess.java
===================================================================
RCS file: /cvs/mauve/mauve/RunnerProcess.java,v
retrieving revision 1.18
diff -u -r1.18 RunnerProcess.java
--- RunnerProcess.java	11 Jun 2008 10:08:00 -0000	1.18
+++ RunnerProcess.java	19 Feb 2009 19:37:11 -0000
@@ -362,6 +362,10 @@
         if (verbose)
           System.out.println("TEST FAILED: exception when loading "
                              + stripPrefix(name));
+
+        if (report != null)
+          report.addTestResult(currentResult);
+
         return;
       }