Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(261)

Issue 215039: Fix gclient unittests. (Closed)

Created:
11 years, 3 months ago by M-A Ruel
Modified:
9 years, 7 months ago
Reviewers:
Nicolas Sylvain
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Fix gclient unittests. BUG=none TEST=unittests

Patch Set 1 #

Total comments: 4
Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -11 lines) Patch
M tests/gclient_test.py View 4 chunks +23 lines, -11 lines 4 comments Download

Messages

Total messages: 2 (0 generated)
M-A Ruel
11 years, 3 months ago (2009-09-21 21:04:03 UTC) #1
Nicolas Sylvain
11 years, 3 months ago (2009-09-21 22:30:43 UTC) #2
http://codereview.chromium.org/215039/diff/1/2
File tests/gclient_test.py (right):

http://codereview.chromium.org/215039/diff/1/2#newcode1095
Line 1095: gclient_scm.RunSVNAndGetFileList(['update', '--revision', 'BASE'],
base_path,
80 chars

http://codereview.chromium.org/215039/diff/1/2#newcode1121
Line 1121: gclient_scm.RunSVNAndGetFileList(['update', '--revision', 'BASE'],
base_path,
80 chars

http://codereview.chromium.org/215039/diff/1/2#newcode1145
Line 1145: #gclient_scm.os.remove(file_path)
why?

http://codereview.chromium.org/215039/diff/1/2#newcode1148
Line 1148: gclient_scm.RunSVNAndGetFileList(['update', '--revision', 'BASE'],
base_path,
80 chars

Powered by Google App Engine
This is Rietveld 408576698