Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test code assumes locale is "English (United States)" #1

Open
GoogleCodeExporter opened this issue Oct 17, 2015 · 1 comment
Open

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. Set your user locale to "English (United Kingdom)"
2. Run all tests
3. GoogleReferenceJSONStringTest2 fails on line 362, because the following 
expected and actual 
strings do not match (call to DateTime.format() produces localized string, but 
the expected string 
is hard coded in United States locale):

Expected:
{cols: [{id: "A", label: "NEW A", type: "string"},{id: "B", label: "B-label", 
type: "number"},{id: "C", 
label: "C-label", type: "date"}], rows: [{c:[{v: "a"},{v: "1"},{v: new 
Date(2008, 1, 28, 0, 31, 26), f: 
"2/28/2008 12:31 AM"}]},{c:[{v: "b"},{v: "2"},{v: new Date(2008, 2, 30, 0, 31, 
26), f: "3/30/2008 
12:31 AM"}]},{c:[{v: "c"},{v: "3"},{v: new Date(2008, 3, 30, 0, 31, 26), f: 
"4/30/2008 12:31 AM"}]}]}

Actual:
{cols: [{id: "A", label: "NEW A", type: "string"},{id: "B", label: "B-label", 
type: "number"},{id: "C", 
label: "C-label", type: "date"}], rows: [{c:[{v: "a"},{v: "1"},{v: new 
Date(2008, 1, 28, 0, 31, 26), f: 
"28/02/2008 00:31"}]},{c:[{v: "b"},{v: "2"},{v: new Date(2008, 2, 30, 0, 31, 
26), f: "30/03/2008 
00:31"}]},{c:[{v: "c"},{v: "3"},{v: new Date(2008, 3, 30, 0, 31, 26), f: 
"30/04/2008 00:31"}]}]} 

What is the expected output? What do you see instead?


What version of the product are you using? On what operating system?


Please provide any additional information below.


Original issue reported on code.google.com by [email protected] on 7 Jun 2010 at 4:50

@GoogleCodeExporter
Copy link
Author

Original comment by [email protected] on 7 Jun 2010 at 5:14

Attachments:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant