Simpletest should be able to output xml reporting for continuous integration needs

[rahal]

Here is a patch for this : ​https://bitbucket.org/rahal/jelix-trunk-patches/src/b21785c4e441/support_junit_test_reports_output .

This patch is to be applied after simpletest upgrade to 1.1

[laurentj]

some comments are in french. copyright holder are missing in some files. Some files have CRLF as line ending instead of LF. In your commit message, don't forget to add the ticket number.

You should simplify some code. Ex:

	protected function getReporterType($rep){
		if ($this->option('--junitoutput' )){
			$outputHandler  = new xmlOutPutHandler ($rep);
			$type = 'junittests~jjunitrespreporter' ;
		} else { 
			$outputHandler  = new txtOutPutHandler ($rep);
			$type = 'junittests~jtextrespreporter' ;
		}

		// Sets the output handler
		$this->setOutPutHandler($outputHandler);
		return $type;
	}


...

        $this->responseType=$this->getReporterType($rep);

It's better imho.

In the Makefile, I think we should not run tests.php when SIMPLETEST_OUTPUT and BUILDTESTLOG are empty.

[rahal]

ok, here is a reviewed patch ​https://bitbucket.org/rahal/jelix-trunk-patches/src/c17382a49022/support_junit_test_reports_output

[laurentj]

ok for the patch.

Before landing it, upgrade to simpletest 1.1 and verify that all tests are ok.

Should we really need to upgrade to simpletest 1.1 ?

[rahal]

Oups ! there is no simpletest 1.1 yet, simpletest's last version is 1.1alpha2 which is already integrated in jelix trunk ​https://bitbucket.org/jelix/jelix-trunk/changeset/65d3af7fef8a

[rahal]

Patch landed in trunk : ​https://bitbucket.org/jelix/jelix-trunk/changeset/c5f2904efdaa