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

Issue 113964: Make AffectedFile member variables private. (Closed)

Created:
11 years, 6 months ago by M-A Ruel
Modified:
9 years, 7 months ago
Reviewers:
Jói Sigurðsson
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Make AffectedFile member variables private. Cache SvnAffectedFile.IsTextFile() return value. Improve the unit test by not/less relying on the local file system.

Patch Set 1 #

Patch Set 2 : Improved the unit tests #

Patch Set 3 : 80cols #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+101 lines, -39 lines) Patch
M presubmit_support.py View 1 3 chunks +41 lines, -33 lines 0 comments Download
M tests/presubmit_unittest.py View 1 2 12 chunks +60 lines, -6 lines 3 comments Download

Messages

Total messages: 4 (0 generated)
M-A Ruel
11 years, 6 months ago (2009-05-28 21:06:53 UTC) #1
Jói Sigurðsson
LGTM with nits http://codereview.chromium.org/113964/diff/3001/3003 File tests/presubmit_unittest.py (right): http://codereview.chromium.org/113964/diff/3001/3003#newcode698 Line 698: self.failIf(affected_file.IsDirectory()) duplicate? http://codereview.chromium.org/113964/diff/3001/3003#newcode708 Line 708: ...
11 years, 6 months ago (2009-05-28 21:30:51 UTC) #2
M-A Ruel
http://codereview.chromium.org/113964/diff/3001/3003 File tests/presubmit_unittest.py (right): http://codereview.chromium.org/113964/diff/3001/3003#newcode698 Line 698: self.failIf(affected_file.IsDirectory()) On 2009/05/28 21:30:51, Jói wrote: > duplicate? ...
11 years, 6 months ago (2009-05-29 00:59:55 UTC) #3
Jói Sigurðsson
11 years, 6 months ago (2009-05-29 13:13:52 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld 408576698