[Kde-pim] Review Request 109325: fixing tests in messageviewer

[Sandro Knauß]

> On March 7, 2013, 6:36 a.m., Laurent Montel wrote:
> > messageviewer/objecttreeparser.cpp, line 3111
> > <http://git.reviewboard.kde.org/r/109325/diff/1/?file=117580#file117580line3111>
> >
> >     Before to fix objecttreeparser which is a special class, you need to show a bug report about it.
> >     you replace a += to = it will change a lot of thing I think. so you need to show bug about it thanks.
> >

The tests themself:
http://build.kde.org/job/kdepim_master/1212/testReport/junit/(root)/TestSuite/messageviewer_objecttreeparsertest/
[...]
Actual (otp.plainTextContent()): This is the encapsulating message.This is the encapsulating message.

Expected (QString( "This is the encapsulating message." )): This is the encapsulating message.
[...]
I think it is not intended to repeat the message.

No it isn't used a lot:
cd kdepim && grep -rl 'plainTextContent\W' 

messageviewer/objecttreeparser.cpp
messageviewer/objecttreeparser.h
messageviewer/tests/unencryptedmessagetest.cpp
messageviewer/tests/objecttreeparsertest.cpp
composereditor-ng/composereditor.cpp  ( not related )
composereditor-ng/composereditor.h    ( not related )
messagecomposer/composerviewbase.cpp  ( no tests available )
templateparser/templateparser.cpp: ( tested via templateparsertest )
templateparser/tests/templateparsertest.cpp: QVERIFY( !otp.plainTextContent().isEmpty() );
blogilo/src/postentry.h               ( not related )
blogilo/src/postentry.cpp             ( not related )

So there are one two places, where it used
* Unfortuantelly templateparsertest is broken *grmpf* 
* I can't say what will be happen is messagecomposer/composerviewbase.cpp.


- Sandro


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109325/#review28745
-----------------------------------------------------------


On March 6, 2013, 11:33 p.m., Sandro Knauß wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109325/
> -----------------------------------------------------------
> 
> (Updated March 6, 2013, 11:33 p.m.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Description
> -------
> 
> fixed messageviewer-objecttreeparsertest
> fixed messageviewer-unencryptedmessagetest
> 
> one problem is still occures: gpgsm will open a pinentry window that has to be closed via mouse click.
> 
> 
> Diffs
> -----
> 
>   messageviewer/objecttreeparser.cpp d41e77f65adc20374b2ec358a80d9aebf6e9fed4 
>   messageviewer/tests/objecttreeparsertest.cpp ded141a6444a98392096984ad8f29fc4ea2b2210 
>   messageviewer/tests/unencryptedmessagetest.cpp 6b3dc7649b171df1746a8b43842da0690127a6bb 
> 
> Diff: http://git.reviewboard.kde.org/r/109325/diff/
> 
> 
> Testing
> -------
> 
> cd messageviewer && make test
> Running tests...
> Test project /home/kdetest/kde/build/kdepim/messageviewer
>     Start 1: messageviewer-htmlquotecolorertest
> 1/5 Test #1: messageviewer-htmlquotecolorertest .....   Passed    0.61 sec
>     Start 2: messageviewer-objecttreeparsertest
> 2/5 Test #2: messageviewer-objecttreeparsertest .....   Passed    0.54 sec
>     Start 3: messageviewer-rendertest
> 3/5 Test #3: messageviewer-rendertest ...............***Failed    0.99 sec
>     Start 4: messageviewer-unencryptedmessagetest
> 4/5 Test #4: messageviewer-unencryptedmessagetest ...   Passed    2.79 sec
>     Start 5: accessiblemailwebviewtest
> 5/5 Test #5: accessiblemailwebviewtest ..............   Passed    0.41 sec
> 
> 80% tests passed, 1 tests failed out of 5
> 
> Total Test time (real) =   5.36 sec
> 
> The following tests FAILED:
>           3 - messageviewer-rendertest (Failed)
> 
> 
> one problem is still occures: gpgsm will open a pinentry window that has to be closed via mouse click.
> 
> 
> Thanks,
> 
> Sandro Knauß
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/