Issue 243537 in chromium: TestPageSet.testSimpleSet failing on windows

[chro...@googlecode.com]

Status: Unconfirmed
Owner: ----
CC: d...@chromium.org, nd...@chromium.org
Labels: OS-Linux Type-Bug Pri-2 Cr-Tests-Telemetry Telemetry-Test-Fixit

New issue 243537 by to...@chromium.org: TestPageSet.testSimpleSet failing
on windows
http://code.google.com/p/chromium/issues/detail?id=243537

[ RUN ] TestPageSet.testSimpleSet
Traceback (most recent call last):

File "E:\b\build\slave\Chromium_Windows_Perf_Annotator\build\src\tools\telemetry\telemetry\page\page_set_unittest.py",
line 39, in testSimpleSet
ps = page_set.PageSet.FromFile(f2.name)

File "E:\b\build\slave\Chromium_Windows_Perf_Annotator\build\src\tools\telemetry\telemetry\page\page_set.py",
line 35, in FromFile
with open(file_path, 'r') as f:
IOError: [Errno 13] Permission
denied: 'c:\\users\\chrome~2\\appdata\\local\\temp\\tmpqmrjsk'

[ FAILED ] TestPageSet.testSimpleSet (0 ms)

See:
http://build.chromium.org/p/chromium.fyi/builders/Chromium%20Windows%20Perf%20Annotator/builds/2825/steps/telemetry_unittests/logs/stdio

--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

[chro...@googlecode.com]

Comment #3 on issue 243537 by bugdro...@chromium.org:
TestPageSet.testSimpleSet failing on windows
http://code.google.com/p/chromium/issues/detail?id=243537#c3

------------------------------------------------------------------------
r202126 | to...@chromium.org | 2013-05-24T18:11:42.993642Z

Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/tools/telemetry/telemetry/page/page_set_unittest.py?r1=202126&r2=202125&pathrev=202126

[Telemetry] Fix TestPageSet.testSimpleSet on windows.

1. You cannot open a file for writing and reading at the same time on
windows.
So close the writing file before opening it for reading.
2. Fix json backslash escaping issue.

BUG=243537
TEST=python tools\telemetry\run_tests TestPageSet.testSimpleSet
NOTRY=True

Review URL: https://chromiumcodereview.appspot.com/15846013
------------------------------------------------------------------------

[chro...@googlecode.com]

Updates:
Status: Fixed

Comment #4 on issue 243537 by to...@chromium.org: TestPageSet.testSimpleSet

failing on windows
http://code.google.com/p/chromium/issues/detail?id=243537

(No comment was entered for this change.)