[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