Hi, On Mon, Jun 06, 2016 at 10:39:12PM +0200, David Sommerseth wrote: > > On top of that, adding unit tests into an existing code like openvpn > > will involve a lot of refactoring. The ROI on backporting any such tests > > to 2.3 does not look worth the effort. > > Yes, lots of the code may need to be refactored. But I am not currently > convinced everything will need refactoring. > > I am especially having the SSL abstraction layer in my mind. That did > already go through a massive amount of refactoring to make it possible > to swap between OpenSSL and mbedTLS/PolarSSL. Those code paths I see > great value of having tested with unit tests, also in 2.3.
This all sounds nice on paper, but please look more closely at what
we are supposed to *do* in the release/2.3 branch
- add bug fixes
- add security relevant changes, if they are small and localized
- aim for maximum stability
there is no "refactoring" here (the SSL refactoring happened in git
master before 2.3 was forked).
Unit tests in git master are good and welcome, and if we discover bugs
there, we will backport the bug fixes to 2.3 (or at least look carefully
if 2.3 is affected by the same bugs at all).
Given that we do not have plenty of developer time around, we should focus
*development* time on git master / 2.4 - and that includes adding testing
frameworks.
Ceterum censo: please reconsider whether having that stuff in 2.3 is
beneficial, or just eats up highly valuable developer time without
bringing enough benefits to compensate.
[..]
> Without
> having dug too much into the buffer.[ch] code lately - so my memory may
> be corrupted, but I think that is also code which can be adopted without
> too much refactoring - and these code paths are also quite crucial. But
> yes, the unit test coverage will not be as massive as it hopefully will
> develop to in master and 2.4.
"without too much refactoring" is already "way too much refactoring" - the
last times we thought we'd "clean up some stuff" we introduced memory
leaks and crashes that took us months to find and fix - look at
ad49fb5a74b2ac16298ee8383d514a950ece1097, for example.
Since I'm usually the one who gets to debug such fallout, I'm absolutely
opposed to doing any sort of "this should not change anything" or "just
a bit of cleanup" changes in release/2.3. Bug-fixes, and long-term
compatibility changes (if localized).
gert
--
USENET is *not* the non-clickable part of WWW!
//www.muc.de/~gert/
Gert Doering - Munich, Germany g...@greenie.muc.de
fax: +49-89-35655025 g...@net.informatik.tu-muenchen.de
signature.asc
Description: PGP signature
