[Kmymoney-devel] Review Request: Fix some unit test linking issues on msvc

[Fernando Vilas]

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


If it is possible to do a #ifdef KDE4_BUILD_TESTS / #endif or similar around the friend declaration, I would prefer that. If not, then it looks good to me. In an effort to reduce the friend declarations, we should probably have a more in depth look at the tests after this is committed.

- Fernando


On 2010-07-03 18:10:15, Cristian Onet wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/4520/
> -----------------------------------------------------------
> 
> (Updated 2010-07-03 18:10:15)
> 
> 
> Review request for kmymoney.
> 
> 
> Summary
> -------
> 
> Replace the '#define' of all access qualifiers to public with proper 'friend class' declarations for unit tests. I know that this replaces the non-intrusive way of writing unit tests with a relatively intrusive way (if you can call the friend class declaration as such) but this should be done to fix the following linkage problem. Some compilers (like msvc) include the access qualifiers in the mangled method name so when we try to link the unit test we will fail because we are looking for a public method in a library where that method is actually private or protected. It took me a while to spot this error and I think we should really fix it. I don't see a problem with the fact that the unit test class is a friend of the real class. Any thoughts? Another solution would mean recompiling the mymoney cpp files and linking them in the test executable but I don't like this solution.
> 
> 
> Diffs
> -----
> 
>   trunk/extragear/office/kmymoney/kmymoney/converter/convertertest.cpp 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyaccount.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyaccounttest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyaccounttest.cpp 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyexceptiontest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyfile.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyfiletest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyfinancialcalculator.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyfinancialcalculatortest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyforecast.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyforecasttest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyinstitutiontest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneykeyvaluecontainer.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneykeyvaluecontainertest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneymoney.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneymoneytest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyobject.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyobjecttest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneypayeetest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneypricetest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyscheduled.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneyscheduletest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneysecuritytest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneysplittest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneytransaction.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/mymoneytransactiontest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/storage/mymoneydatabasemgr.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/storage/mymoneydatabasemgrtest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/storage/mymoneymaptest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/storage/mymoneyseqaccessmgr.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/storage/mymoneyseqaccessmgrtest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/mymoney/storage/mymoneystoragesql.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/reports/pivotgrid.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/reports/pivotgridtest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/reports/pivotgridtest.cpp 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/reports/pivottable.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/reports/pivottabletest.h 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/reports/pivottabletest.cpp 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/reports/querytabletest.cpp 1145134 
>   trunk/extragear/office/kmymoney/kmymoney/reports/reportstestcommon.cpp 1145134 
> 
> Diff: http://reviewboard.kde.org/r/4520/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Cristian
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kmymoney-devel/attachments/20100703/b9457b9f/attachment-0001.htm