Review Request 108510: split out _data() method for TestBlockLayout::testTabs()

Friedrich W. H. Kossebau kossebau at kde.org
Sun Jan 20 17:50:36 GMT 2013


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

Review request for Calligra and C. Boemann.


Description
-------

When investigating the failing TestBlockLayout::testTabs() I split out the data delivery into a separate method.

Might be worth to keep it in the real codebase as well, as that way in case of fails the index of the test data will be reported, so one does not have to add debug statements to find out for which data it failed.

E.g. in http://build.kde.org/job/calligra_master/606/testReport/junit/(root)/TestSuite/libs_textlayout_TestBlockLayout/ one has no idea for which "i" it is failing. And if it would fail for more testdata.


Diffs
-----

  libs/textlayout/tests/TestBlockLayout.h c770bd3 
  libs/textlayout/tests/TestBlockLayout.cpp a642b4e 

Diff: http://git.reviewboard.kde.org/r/108510/diff/


Testing
-------


Thanks,

Friedrich W. H. Kossebau

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20130120/113777eb/attachment.htm>


More information about the calligra-devel mailing list