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 12 years ago

Closed 12 years ago

Last modified 12 years ago

#699 closed bug (fixed)

testapp: some tests fail when run individually, due to wrong locale usage

Reported by: Julien Owned by: Julien
Priority: high Milestone: Jelix 1.1 beta 1
Component: app:testapp Version: trunk
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

When you run "all tests" in testapp, everything is OK, but trying to run some tests individually fails, because they use french strings to test results, and testapp uses english by default.

"All tests" works right because jLocale tests (which happen before the failing tests) do not always reset the default locale after changing it.

The tests that fail when run alone :

  • jforms: htmlbuilder
  • jforms: htmlbuilder2
  • jforms: htmllightbuilder
  • jforms: htmllightbuilder2

Attachments (2)

699-testapp-some-tests-fail-when-run-individually-due-to-wrong-locale-usage.diff (93.3 KB) - added by Julien 12 years ago.
699-testapp-some-tests-fail-when-run-individually-due-to-wrong-locale-usage.2.diff (93.8 KB) - added by Julien 12 years ago.
updated patch: changed default locale from en_US to en_EN

Download all attachments as: .zip

Change History (9)

comment:1 Changed 12 years ago by Julien

  • Status changed from new to assigned

comment:2 Changed 12 years ago by Julien

  • Version changed from 1.0.5 to trunk

Changed 12 years ago by Julien

updated patch: changed default locale from en_US to en_EN

comment:3 Changed 12 years ago by Julien

updated patch: changed default locale from en_US to en_EN as it is more generic and matches /lib/jelix-scripts/templates/var/config/defaultconfig.ini.php.tpl

comment:4 Changed 12 years ago by laurentj

  • review changed from review? to review+

comment:5 Changed 12 years ago by laurentj

it's ok

comment:6 Changed 12 years ago by laurentj

  • Resolution set to fixed
  • Status changed from assigned to closed

Commited, thanks for this patch.

comment:7 Changed 12 years ago by Julien

;)

Note: See TracTickets for help on using tickets.