Review Request 129082: Reenable CSV importer tests

Łukasz Wojniłowicz lukasz.wojnilowicz at gmail.com
Sun Oct 2 10:48:33 UTC 2016


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

Review request for KMymoney.


Repository: kmymoney


Description
-------

This patch does what it's intended to do, but I'm not completly satisfied with it.

I wonder why I get lots of errors of this type
./kmymoney/kmymoney/plugins/csvimport/tests/symbol-test.cpp:34: undefined reference to `Parse::Parse()'

if I use **kmm_csvimport** instead of **csvimport_tests** in code block below

```
file(GLOB tests_sources "*-test.cpp")
ecm_add_tests(${tests_sources}
  NAME_PREFIX
    "csvimport-"
  LINK_LIBRARIES
    Qt5::Test
    csvimport_tests
)
```

if **kmm_csvimport** is **MODULE** instead of **STATIC**.
Can somebody explain it to me and knows how to do it without **csvimport_tests**?


Diffs
-----

  kmymoney/plugins/csvimport/tests/CMakeLists.txt f5b148e 

Diff: https://git.reviewboard.kde.org/r/129082/diff/


Testing
-------


Thanks,

Łukasz Wojniłowicz

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kmymoney-devel/attachments/20161002/1b78733a/attachment-0001.html>


More information about the KMyMoney-devel mailing list