[Webkit-unassigned] [Bug 38388] Merge svn-apply and svn-unapply

[bugzilla-daemon at webkit.org]

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





--- Comment #3 from Chris Jerdonek <cjerdonek at webkit.org>  2010-04-30 09:45:11 PST ---
(In reply to comment #2)
> (In reply to comment #1)
> > Another approach that can be taken in conjunction with this approach is to
> > move/refactor methods defined in both files to a third location.
> 
> If we follow this direction the unit-testing can be more precise not by testing
> the whole file(s), but subroutines directly.

We can still do this either way.  For example, if you look at
VCSUtils_unittest/, you will see that it contains unit tests for individual
subroutines defined in VCSUtils.pm.

In the end, perhaps svn-apply can have a top-level "main()" method.  Then if
one wants to go the route of testing the whole file, testing that single
subroutine is almost as good from a code coverage standpoint as testing the
"whole file".  It won't be as necessary to test the script by invoking it as a
script.

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