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

Issue 2092012: Add basic gclient smoke tests. (Closed)

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

Description

Add basic gclient smoke tests. The unit tests are unbearable and the next change forces me to trash them. Thus adding smoke tests before. The current tests aren't involved enough yet but are a good base for further testing. I plan to only test the most important functionalities, the ones used in the continuous build and try server. TEST=new smoke test BUG=23328 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=47580

Patch Set 1 #

Patch Set 2 : Rename the file to a better name #

Unified diffs Side-by-side diffs Delta from patch set Stats (+339 lines, -0 lines) Patch
M .gitignore View 1 chunk +1 line, -0 lines 0 comments Download
A tests/fake_repos.py View 1 chunk +203 lines, -0 lines 0 comments Download
A tests/gclient_smoketest.py View 1 chunk +135 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
M-A Ruel
10 years, 7 months ago (2010-05-18 20:55:24 UTC) #1
Nasser Grainawi
10 years, 7 months ago (2010-05-18 22:45:15 UTC) #2
On 2010/05/18 20:55:24, Marc-Antoine Ruel wrote:
> 

this will make testing SOOO much easier. Thank you!
lgtm

Powered by Google App Engine
This is Rietveld 408576698