Summary: | [bzt] Speed up CredentialsTest.test_read_credentials_with_SVN to make run-webkit-unittest run faster | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Daniel Bates <dbates> | ||||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | abarth, eric, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
Daniel Bates
2009-12-29 21:49:28 PST
Created attachment 45638 [details]
Patch
Comment on attachment 45638 [details]
Patch
Why are we keeping around the disabled path?
style-queue ran check-webkit-style on attachment 45638 [details] without any errors.
Comment on attachment 45638 [details]
Patch
On second thought, we don't need the optional parameter shouldMakeActualSVNRepo and related code as it does add much if any value. Original thought was to make the unit test match the actual environment as close as possible. But it is sufficient to just create a temporary directory (one in which we know does not contain a not Git repo).
Created attachment 45639 [details]
Patch
Removed unnecessary code that made use of modules.scm_unittest.SVNTestRepository.
Also, renamed method test_read_credentials_with_SVN to test_read_credentials_without_Git_repo to better reflect what it is testing.
Comment on attachment 45639 [details]
Patch
LGTM.
Comment on attachment 45639 [details]
Patch
test_read_credentials_without_Git_repo
capitalization is a bit strange. I would have just called it "git"
(In reply to comment #7) > (From update of attachment 45639 [details]) > test_read_credentials_without_Git_repo > capitalization is a bit strange. I would have just called it "git" I'll fix this before I land. Committed in <http://trac.webkit.org/changeset/52644>. |