Chromium Code Reviews

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 Stats (+339 lines, -0 lines)
M .gitignore View 1 chunk +1 line, -0 lines 0 comments
A tests/fake_repos.py View 1 chunk +203 lines, -0 lines 0 comments
A tests/gclient_smoketest.py View 1 chunk +135 lines, -0 lines 0 comments

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