[Webkit-unassigned] [Bug 26755] webkit-patch's commit messages are less readable than commit-log-editor's

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=26755





--- Comment #43 from Adam Barth <abarth at webkit.org>  2011-07-07 11:11:03 PST ---
> Both MockExecutive and MockExecutive2 just return canned output. Is that really what we want?

For most tests, yes.  If you specifically want to test integration with commit-log-editor, we can do that by explicitly passing a live Executive in the test.  Generally, we avoid unit tests that touch the real disk or spawn processes whenever possible.  We have a handful to make sure we continue to integrate correctly with external dependencies, but we don't need to test commit-log-editor exhaustively in test-webkitpy (we should just test that it works at all).

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.