developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 10 years ago

Closed 10 years ago

#1304 closed enhancement (fixed)

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

Reported by: rahal Owned by: rahal
Priority: normal Milestone: Jelix 1.3 beta 1
Component: module:junittests Version: 1.2.1
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

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

Change History (5)

comment:1 Changed 10 years ago by laurentj

  • Milestone set to Jelix 1.3
  • review changed from review? to review-

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.

comment:2 Changed 10 years ago by rahal

  • review changed from review- to review?

comment:3 Changed 10 years ago by laurentj

  • review changed from review? to review+

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 ?

comment:4 Changed 10 years ago by rahal

  • Owner set to rahal
  • Status changed from new to assigned

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

comment:5 Changed 10 years ago by rahal

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.