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-
comment:2 Changed 10 years ago by rahal
- review changed from review- to review?
ok, here is a reviewed patch https://bitbucket.org/rahal/jelix-trunk-patches/src/c17382a49022/support_junit_test_reports_output
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
Patch landed in trunk : https://bitbucket.org/jelix/jelix-trunk/changeset/c5f2904efdaa
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:
It's better imho.
In the Makefile, I think we should not run tests.php when SIMPLETEST_OUTPUT and BUILDTESTLOG are empty.