more test include gardening

test/unit/core/box2d_test.cpp

@@ -1,6 +1,5 @@
 #include "catch.hpp"
 
-#include <iostream>
 #include <mapnik/coord.hpp>
 #include <mapnik/box2d.hpp>
 

test/unit/core/comparison_test.cpp

@@ -2,7 +2,6 @@
 
 #include <mapnik/value_types.hpp>
 #include <mapnik/value.hpp>
-#include <mapnik/unicode.hpp>
 
 TEST_CASE("comparison")
 {

test/unit/core/conversions_test.cpp

@@ -6,8 +6,6 @@
 #include <mapnik/util/conversions.hpp>
 
 #include <iostream>
-#include <vector>
-#include <algorithm>
 #include <unordered_map>
 #include <sstream>
 

test/unit/core/copy_move_test.cpp

@@ -6,9 +6,6 @@
 #include <mapnik/datasource_cache.hpp>
 #include <mapnik/util/fs.hpp>
 
-#include <vector>
-#include <algorithm>
-
 #include "catch.hpp"
 
 

test/unit/core/params_test.cpp

@@ -4,8 +4,6 @@
 #include <mapnik/value_types.hpp>
 #include <mapnik/params.hpp>
 #include <mapnik/boolean.hpp>
-#include <vector>
-#include <algorithm>
 
 #include <ostream>
 

test/unit/imaging/image.cpp

@@ -1,10 +1,10 @@
 #include "catch.hpp"
 
 // mapnik
-#include <mapnik/value.hpp>
+#include <mapnik/image.hpp>
+#include <mapnik/image_view.hpp>
 #include <mapnik/image_any.hpp>
 #include <mapnik/color.hpp>
-#include <mapnik/image_view_any.hpp>
 #include <mapnik/image_util.hpp>
 
 TEST_CASE("image class") {

test/unit/imaging/image_io_test.cpp

@@ -6,8 +6,6 @@
 #include <mapnik/image_util.hpp>
 #include <mapnik/image_util_jpeg.hpp>
 #include <mapnik/util/fs.hpp>
-#include <vector>
-#include <algorithm>
 #if defined(HAVE_CAIRO)
 #include <mapnik/cairo/cairo_context.hpp>
 #include <mapnik/cairo/cairo_image_util.hpp>

test/unit/imaging/image_set_pixel.cpp

@@ -1,9 +1,7 @@
 #include "catch.hpp"
 
 // mapnik
-#include <mapnik/image_any.hpp>
-#include <mapnik/image_view_any.hpp>
-#include <mapnik/color.hpp>
+#include <mapnik/image.hpp>
 #include <mapnik/image_util.hpp>
 
 

test/unit/imaging/image_view.cpp

@@ -1,7 +1,7 @@
 #include "catch.hpp"
 
 // mapnik
-#include <mapnik/image_any.hpp>
+#include <mapnik/image.hpp>
 #include <mapnik/color.hpp>
 #include <mapnik/image_view_any.hpp>
 #include <mapnik/image_util.hpp>

test/unit/imaging/tiff_io.cpp

@@ -7,10 +7,8 @@
 #include "catch.hpp"
 
 #include <mapnik/image_reader.hpp>
-#include <mapnik/image_util.hpp>
 #include <mapnik/util/file_io.hpp>
 
-#include <mapnik/tiff_io.hpp>
 #include "../../../src/tiff_reader.cpp"
 
 #define TIFF_ASSERT(filename) \

test/unit/imaging/webp_io.cpp

@@ -3,8 +3,7 @@
 
 #include "catch.hpp"
 
-#include <mapnik/image_util.hpp>
-#include <mapnik/image_view_any.hpp>
+#include <mapnik/image_view.hpp>
 #include <mapnik/webp_io.hpp>
 
 TEST_CASE("webp io") {

test/unit/pixel/agg_blend_src_over_test.cpp

@@ -1,12 +1,9 @@
 #include "catch.hpp"
 
 #include <iostream>
-#include <cstdio>
 #include <cstring>
 #include <sstream>
 #include <string>
-#include <vector>
-#include <algorithm>
 #include "agg_color_rgba.h"
 #include "agg_pixfmt_rgba.h"
 #include "agg_rendering_buffer.h"

test/unit/serialization/wkb_formats_test.cpp

@@ -1,15 +1,12 @@
 #include "catch.hpp"
 
 #include <iostream>
-#include <mapnik/params.hpp>
 #include <mapnik/wkb.hpp>
 #include <mapnik/feature.hpp>
 #include <mapnik/geometry_is_valid.hpp>
 #include <mapnik/geometry_is_simple.hpp>
 #include <mapnik/geometry_correct.hpp>
 #include <mapnik/feature_factory.hpp>
-#include <vector>
-#include <algorithm>
 #include <boost/version.hpp>
 
 TEST_CASE("geometry formats") {

test/unit/serialization/xml_parser_trim.cpp

@@ -1,10 +1,8 @@
 #include "catch.hpp"
 
-#include <mapnik/debug.hpp>
-#include <mapnik/value.hpp>
 #include <mapnik/xml_tree.hpp>
 #include <mapnik/xml_loader.hpp>
-#include <mapnik/attribute.hpp>
+#include <mapnik/attribute.hpp> // needed due to fwd declare in value_types.hpp
 
 TEST_CASE("xml parser") {
 

test/unit/svg/svg_parser_test.cpp

@@ -22,20 +22,18 @@
 
 #include "catch.hpp"
 
-#include <mapnik/version.hpp>
 #include <mapnik/debug.hpp>
 #include <mapnik/marker.hpp>
 #include <mapnik/marker_cache.hpp>
 #include <mapnik/vertex.hpp>
 #include <mapnik/svg/svg_parser.hpp>
-#include <mapnik/svg/svg_storage.hpp>
 #include <mapnik/svg/svg_converter.hpp>
 #include <mapnik/svg/svg_path_adapter.hpp>
 #include <mapnik/svg/svg_path_attributes.hpp>
 
 #include <cmath>
 #include <fstream>
-#include <streambuf>
+#include <iterator>
 
 namespace detail {
 

test/unit/symbolizer/symbolizer_test.cpp

@@ -2,8 +2,6 @@
 
 #include <iostream>
 #include <mapnik/symbolizer.hpp>
-#include <vector>
-#include <algorithm>
 
 using namespace mapnik;