Closed
Bug 396059
Opened 17 years ago
Closed 12 years ago
Create new Talos suite: run basic startup tests from non-ASCII paths
Categories
(Testing :: Talos, defect)
Testing
Talos
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: ted, Unassigned)
Details
Given bugs like bug 396052, we should at least be trying to launch the browser from a non-ASCII path. We could possibly do this on the perftest machines. It'd be nice to avoid future bustage like this.
Reporter | ||
Comment 1•17 years ago
|
||
Also it looks like we're broken on Win32 with a non-ascii username, so that'd be good to test too.
Reporter | ||
Comment 2•17 years ago
|
||
See also bug 396199.
Reporter | ||
Updated•16 years ago
|
Component: Testing → General
Product: Core → Testing
QA Contact: testing → general
Updated•15 years ago
|
Summary: should run basic startup tests from non-ASCII paths → Create new Talos suite: run basic startup tests from non-ASCII paths
Comment 4•14 years ago
|
||
Is this worth having a whole test devoted to it? Sounds more like a unit test then a talos test, is all.
Reporter | ||
Comment 5•14 years ago
|
||
My original summary was simply "should run basic startup tests from non-ASCII paths". It's kind of a PITA to write tests that have to start the browser specially right now, since we don't have a harness for that, but I don't really care if this were to be Talos or just some other new harness.
Comment 6•13 years ago
|
||
This is actually not very complicated. If you add "!!python/str" tag in front of the browser_path definition in the yml file and ensure that the yml file is encoded in utf-8, it works correctly.
One other way is to loop over all the parsed yaml values and decode them to utf-8 encoded strings instead of unicode strings.
Changing setEnvironmentVars to use str decoding is not a good option since the parameters passed can be instances of int or some other class.
Comment 7•13 years ago
|
||
Adding jhammel since this may be related with his efforts on PerfConfigurator.
Comment 8•12 years ago
|
||
this isn't needed for a performance test, lets resolve as a unittest.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•