AppFuse

Force locale for canoo web tests

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.8
  • Fix Version/s: 1.8.1
  • Component/s: Build, Test, or Deploy Process
  • Labels:
    None
  • Environment:

Description

"ant test-canoo -Dtestcase=Login" Fail with error beacause of locale (need to force local to en for test-canoo):

C:\developpement\appfuse\appfuse-1.8-src\appfuse>ant test-canoo -Dtestcase=Login
Buildfile: build.xml

ping-tomcat:
[echo] Verifying that tomcat is running...
[get] Getting: http://localhost:8080/appfuse

init:

prepare:
[echo] Preparing target directory 'C:\developpement\appfuse\appfuse-1.8-src\appfuse/build/appfuse'

db-load:
[dbunit] Executing operation: CLEAN_INSERT
[dbunit] on file: C:\developpement\appfuse\appfuse-1.8-src\appfuse\metadata\sql\sample-data.xml
[dbunit] with format: xml

test-canoo:
[delete] Deleting: C:\developpement\appfuse\appfuse-1.8-src\appfuse\build\test\data\web-tests-result.xml

Login:

BUILD FAILED
C:\developpement\appfuse\appfuse-1.8-src\appfuse\build.xml:568: The following error occurred while executing this line:
Test step verifytitle (C:\developpement\appfuse\appfuse-1.8-src\appfuse\test\web\login.xml:3: ) named "we should see the login title" failed with mes
sage "Wrong document title found! Expected "AppFuse ~ Login" but got "AppFuse ~ Connexion""

Total time: 3 seconds

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Matt Raible added a comment -

Can you confirm this issue still occurs with the CVS version. I believe it probably does, but want to make sure. Also, you might search the mailing list archives as one user pointed out how to force the local using a <sysproperty> tag in Ant.

Show
Matt Raible added a comment - Can you confirm this issue still occurs with the CVS version. I believe it probably does, but want to make sure. Also, you might search the mailing list archives as one user pointed out how to force the local using a <sysproperty> tag in Ant.
Hide
Permalink
Benjamin Francisoud added a comment -

I don't know how to force locale on <ant> tag.
I tried:
<ant antfile="test/web/web-tests.xml" target="$

{testcase}

">
<property name="user.country" value="US"/>
<property name="user.language" value="en"/>
</ant>
but it doesn't work.

From CVS, I can't copile appfuse. Can't find:
org.appfuse.webapp.form.UserForm

do you want me to open a new bug for it?

Also .classpath value for commons-codecs.jar is wrong should point to something like commons-codec-1.3.jar

Show
Benjamin Francisoud added a comment - I don't know how to force locale on <ant> tag. I tried: <ant antfile="test/web/web-tests.xml" target="$ {testcase} "> <property name="user.country" value="US"/> <property name="user.language" value="en"/> </ant> but it doesn't work. From CVS, I can't copile appfuse. Can't find: org.appfuse.webapp.form.UserForm do you want me to open a new bug for it? Also .classpath value for commons-codecs.jar is wrong should point to something like commons-codec-1.3.jar
Hide
Permalink
Benjamin Francisoud added a comment -

In:
.../appfuse/test/web/web-tests.xml
Changing:
<property file="web/WEB-INF/classes/ApplicationResources_en.properties"/>
to:
<property file="web/WEB-INF/classes/ApplicationResources_fr.properties"/>

Works fine!

How could we select locale automaticly ?!

Show
Benjamin Francisoud added a comment - In: .../appfuse/test/web/web-tests.xml Changing: <property file="web/WEB-INF/classes/ApplicationResources_en.properties"/> to: <property file="web/WEB-INF/classes/ApplicationResources_fr.properties"/> Works fine! How could we select locale automaticly ?!
Hide
Permalink
Benjamin Francisoud added a comment -

Simply replace with:
<property file="web/WEB-INF/classes/ApplicationResources_$

{user.language}.properties"/>

everything works fine!

CVS patch:

Index: web-tests.xml
===================================================================
RCS file: /cvs/appfuse/test/web/web-tests.xml,v
retrieving revision 1.26
diff -u -r1.26 web-tests.xml
— web-tests.xml 6 May 2005 21:18:45 -0000 1.26
+++ web-tests.xml 16 May 2005 14:57:09 -0000
@@ -8,7 +8,7 @@
<project basedir="." default="run-all-tests">
<!-- Include ApplicationResources.properties so we can test against
keys, rather than values -->
- <property file="web/WEB-INF/classes/ApplicationResources.properties"/>
+ <property file="web/WEB-INF/classes/ApplicationResources_${user.language}

.properties"/>

<!-- runs all targets -->
<target name="run-all-tests"

Show
Benjamin Francisoud added a comment - Simply replace with: <property file="web/WEB-INF/classes/ApplicationResources_$ {user.language}.properties"/> everything works fine! CVS patch: Index: web-tests.xml =================================================================== RCS file: /cvs/appfuse/test/web/web-tests.xml,v retrieving revision 1.26 diff -u -r1.26 web-tests.xml — web-tests.xml 6 May 2005 21:18:45 -0000 1.26 +++ web-tests.xml 16 May 2005 14:57:09 -0000 @@ -8,7 +8,7 @@ <project basedir="." default="run-all-tests"> <!-- Include ApplicationResources.properties so we can test against keys, rather than values --> - <property file="web/WEB-INF/classes/ApplicationResources.properties"/> + <property file="web/WEB-INF/classes/ApplicationResources_${user.language} .properties"/> <!-- runs all targets --> <target name="run-all-tests"
Hide
Permalink
David AMAR added a comment -

Hello benjamin, I reported in mailing lists a solution using <sysproperty> tag in junit and cargo tasks.

https://appfuse.dev.java.net/servlets/ReadMsg?listName=users&msgNo=6480

Regards (ou salut! en fr.),

David.

Show
David AMAR added a comment - Hello benjamin, I reported in mailing lists a solution using <sysproperty> tag in junit and cargo tasks. https://appfuse.dev.java.net/servlets/ReadMsg?listName=users&msgNo=6480 Regards (ou salut! en fr.), David.
Hide
Permalink
Matt Raible added a comment -

Fixed in CVS:

https://appfuse.dev.java.net/source/browse/appfuse/test/web/config.xml?r1=1.4&r2=1.5
https://appfuse.dev.java.net/source/browse/appfuse/test/web/web-tests.xml?r1=1.26&r2=1.27

Show
Matt Raible added a comment - Fixed in CVS: https://appfuse.dev.java.net/source/browse/appfuse/test/web/config.xml?r1=1.4&r2=1.5 https://appfuse.dev.java.net/source/browse/appfuse/test/web/web-tests.xml?r1=1.26&r2=1.27

People

  • Assignee:
    Matt Raible
    Reporter:
    Benjamin Francisoud
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: