[prev in list] [next in list] [prev in thread] [next in thread] 

List:       koffice-devel
Subject:    RE: Review Request: Layout, first paragraph bottom position fix
From:       Korinek Pavol <pavol.korinek () ixonos ! com>
Date:       2010-09-23 10:50:53
Message-ID: 0D3E2052891577439DFFBFAC2CE5A23A03FBDD38 () HKIMAIL01 ! ixonos ! local
[Download RAW message or body]

Hello Cyrille,

well it's maybe not unit test as you have written. Problem is that bug which I have, \
is like between two classes and after some iterations and communication between them. \
Actually I have been working on it. Do you think, that I should try some different \
approach ? If you are interesting in, I can explain more in details. Interesting is \
that Thomas has marked this bug as 'works for me' yesterday. I have tried to test it. \
It looks quite well, but I see some other problem there. Header was not loaded or \
viewed as before. Probably it's related to previous problem. I have tried also other \
file with header and there was no problem.

Pavol

-----Original Message-----
From: Cyrille Berger [mailto:cberger@cberger.net] 
Sent: Thursday, September 23, 2010 11:49 AM
To: For developer's discussion about KOffice
Subject: Re: Review Request: Layout, first paragraph bottom position fix

Hi,

On Thursday 23 September 2010, Korinek Pavol wrote:
> Hello,
> 
> my name is Pavol. Not Matus ! I have experience with writing unit test :-)
Well your comment on that review lead to believe that you don't. Sorry for 
being blunt. 

"To simulate this problem I had to combine tests for application 
(KWTextDocumentLayout) and pluggin (Layout.cpp) which means don't write test 
for some unit or function, but test for whole layout."

This is *not* an unit test. A unit test is focused on a function of a class. 
Each class defines an API that can be tested.

> I haven't ask for any unit test course. This test was quite problematic
> and I tried to explain why. I have consult it with Boemann. He see problem
> to test layout as one unit too. I haven't said, that it's not possible.
> It's just too expensive. Fix is about one new line in source code. To
> write unit test for this bug could take few days. I should say, that
> simulation with unit test could not be successful. It's not trivial. I'd
> rather consult such step before I'll start work on complicated unit test.
> I'd rather spend time to solve new bugs in this case. I'll hope this
> brings more light to darkness ;-)

The number of lines of a fix does not matter. It is quiet possible that adding 
the line "firstParagraph = true;" is introducing a bug, meaning that someone 
else might come and remove the line. And suddently your bug is back. We can 
understand that this is quiet profitable for companies, but it is not benefical 
for the koffice project.

Also, I shall note that in a recent discussion on the subject, there was a 
wide agreement from the community that patch on reviewboard should come along 
an unit test. While this is not a formal policy, it can be considered a valid 
objection.

-- 
Cyrille Berger
_______________________________________________
koffice-devel mailing list
koffice-devel@kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel


_______________________________________________
koffice-devel mailing list
koffice-devel@kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel


[prev in list] [next in list] [prev in thread] [next in thread]