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

Issue 2189001: Add real gclient_smoke tests now (Closed)

Created:
10 years, 7 months ago by M-A Ruel
Modified:
9 years, 7 months ago
Reviewers:
Nasser Grainawi, yaar
CC:
chromium-reviews
Visibility:
Public.

Description

Add real gclient_smoke tests now gclient sync, revert and status are tested. GClientSmokeGIT.testRevertAndStatus is known to fail since git-backed revert doesn't work. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=48239

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+465 lines, -118 lines) Patch
M tests/fake_repos.py View 5 chunks +243 lines, -96 lines 0 comments Download
M tests/gclient_smoketest.py View 6 chunks +222 lines, -22 lines 2 comments Download

Messages

Total messages: 3 (0 generated)
M-A Ruel
10 years, 7 months ago (2010-05-25 20:37:40 UTC) #1
yaar1
LGTM, though I'm not too familiar with the code base. http://codereview.chromium.org/2189001/diff/1/3 File tests/gclient_smoketest.py (right): http://codereview.chromium.org/2189001/diff/1/3#newcode249 ...
10 years, 7 months ago (2010-05-26 01:37:24 UTC) #2
M-A Ruel
10 years, 7 months ago (2010-05-26 01:56:02 UTC) #3
http://codereview.chromium.org/2189001/diff/1/3
File tests/gclient_smoketest.py (right):

http://codereview.chromium.org/2189001/diff/1/3#newcode249
tests/gclient_smoketest.py:249: results = self.gclient(['sync', '--deps',
'mac'])
On 2010/05/26 01:37:24, yaar1 wrote:
> Is this only going to work on Mac?

No but I'm enforcing the platform so the test behaves consistently on every
platforms.

Powered by Google App Engine
This is Rietveld 408576698