[Merge] lp:~xnox/upstart/test-static into lp:upstart
Dmitrijs Ledkovs
launchpad at surgut.co.uk
Wed Dec 19 12:53:20 UTC 2012
Dmitrijs Ledkovs has proposed merging lp:~xnox/upstart/test-static into lp:upstart.
Requested reviews:
Upstart Reviewers (upstart-reviewers)
For more details, see:
https://code.launchpad.net/~xnox/upstart/test-static/+merge/140655
Unit-tests should not force one to make functions public.
There are no excuses to not unit-test static functions ;-)
Also unit-tests should not modify code.
=))))))
/me wants this, as I will be adding more static functions for override handling & want to unit test them.
--
https://code.launchpad.net/~xnox/upstart/test-static/+merge/140655
Your team Upstart Reviewers is requested to review the proposed merge of lp:~xnox/upstart/test-static into lp:upstart.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 5714 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/upstart-devel/attachments/20121219/fa1aee43/attachment.diff>
More information about the upstart-devel
mailing list