[Marble-devel] Review Request 116702: Extend unit test KML files to cover various KML Style tags

Cruceru Calin-Cristian crucerucalincristian at gmail.com
Mon Mar 10 23:58:18 UTC 2014

This is an automatically generated e-mail. To reply, visit:

Review request for Marble.

Bugs: 331532

Repository: marble


It was intended to extend the .kml files in tests/data/ to include at least one tag for each with non-default values of BalloonStyle, ColorStyle, IconStyle, LineStyle, ListStyle, PolyStyle. 
I made few modifications because most of the tags are already covered appropriately, mainly due to this bug I solved: https://bugs.kde.org/show_bug.cgi?id=331766 . This happened maybe because this is an older bug, which has been reassigned. Their coverage can be checked out here: http://nienhueser.de/marble/lcov/sources/src/lib/marble/geodata/writers/kml/index.html .

If I misunderstood the requirements for this bug, please tell me and I will continue the work on it.


  tests/data/ListStyleTest1.kml a0fc232 
  tests/data/ListStyleTest2.kml ce29ba2 
  tests/data/MackyModel.kml 1de8911 
  tests/data/NetworkLinkDocument.kml 0ea73d1 

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


I've test it using tests/TestGeoDataWriter and everything works as expected.


Cruceru Calin-Cristian

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20140310/c6b0243a/attachment.html>

More information about the Marble-devel mailing list