Bug 26916 – bugzilla-tool needs scm unit tests

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Eric Seidel (no email)]

bugzilla-tool needs PyUnit testing.

Desperately.

I looked at this a little the other day.  I found that PyUnit (unitest module) uses a code style which does not conform to modern python style guides.  This was very confusing, but shouldn't block me using it.  I searched the web a little and did not find any guidance on the matter.

git cl (a similar tool which Chromium uses) has a whole set of test suites:
http://neugierig.org/software/git/?url=git-cl/tree/test

but they're bash files... which is also kinda nasty.  (elegant, but nasty).

[Eric Seidel (no email)]

I've added a few tests.  I'll keep this around for adding the scm_unittest.py (which are the tricky ones).

[Eric Seidel (no email)]

Created attachment 34078 [details]
Patch v1

[Eric Seidel (no email)]

I've broken off the bugzilla bits into bug 27991.

[Adam Barth]

Comment on attachment 34078 [details]
Patch v1

This looks like a good start.  We'll need to do more eventually, but I like the idea of building a fake repo in the tmp directory.

[Eric Seidel (no email)]

Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebKitTools/ChangeLog
	M	WebKitTools/Scripts/modules/scm.py
	A	WebKitTools/Scripts/modules/scm_unittest.py
Committed r46772
	M	WebKitTools/ChangeLog
	A	WebKitTools/Scripts/modules/scm_unittest.py
	M	WebKitTools/Scripts/modules/scm.py
r46772 = 2216d9a41da59a3e65c137e9fdd39b75776a7286 (trunk)
No changes between current HEAD and refs/remotes/trunk
Resetting to the latest refs/remotes/trunk
http://trac.webkit.org/changeset/46772