cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daan Hoogland <>
Subject Re: [GitHub] cloudstack pull request: Removed duplicate code in CitrixResourceB...
Date Wed, 19 Aug 2015 06:04:38 GMT
On Wed, Aug 19, 2015 at 5:56 AM, anshul1886 <> wrote:

> If the purpose is to make sure that path is not modified by other
> developer then adding note/comment on top of that line makes more sense.
> Even adding note is kind of implicit as paths are kind of constants which
> any developer would think before changing. Tests are not meant for that
> purpose.

​Anshul, I hope I don't understand you when you say, 'tests are not meant
for that pupose'. When the hierarchy is changed and this leads to the
constants to be used in a different way in different classes, an error
occurs due to this that will be caught by these tests. This is exactly what
unit tests are for.​

class A has constant pth="/root".
class B:A has constant pth="/root/bla".
​class C:B has no constant hence pth="/root/bla".

now C is changed to C:A and its pth is there fore changed to "/root".​ This
is uninteded and a mistake that will be caught by such tests.


  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message