From fa81ebc2bff3db41c1d12d0be16ea835a666c030 Mon Sep 17 00:00:00 2001 From: Asheem Mamoowala Date: Wed, 11 Oct 2017 15:16:48 -0700 Subject: [core] Tests for CustomVectorSource and CustomTile --- cmake/test-files.cmake | 2 +- src/mbgl/style/custom_tile_loader.cpp | 17 +++- src/mbgl/style/custom_tile_loader.hpp | 3 + src/mbgl/tile/custom_tile.cpp | 1 + test/style/custom_vector_source.test.cpp | 12 --- test/style/source.test.cpp | 37 +++++++++ test/tile/custom_tile.test.cpp | 131 +++++++++++++++++++++++++++++++ 7 files changed, 187 insertions(+), 16 deletions(-) delete mode 100644 test/style/custom_vector_source.test.cpp create mode 100644 test/tile/custom_tile.test.cpp diff --git a/cmake/test-files.cmake b/cmake/test-files.cmake index 39b619a6ac..8d3e8bd346 100644 --- a/cmake/test-files.cmake +++ b/cmake/test-files.cmake @@ -89,7 +89,6 @@ set(MBGL_TEST_FILES test/style/conversion/stringify.test.cpp # style - test/style/custom_vector_source.test.cpp test/style/filter.test.cpp # style/function @@ -114,6 +113,7 @@ set(MBGL_TEST_FILES # tile test/tile/annotation_tile.test.cpp + test/tile/custom_tile.test.cpp test/tile/geojson_tile.test.cpp test/tile/geometry_tile_data.test.cpp test/tile/raster_tile.test.cpp diff --git a/src/mbgl/style/custom_tile_loader.cpp b/src/mbgl/style/custom_tile_loader.cpp index da4ade6734..942216d26e 100644 --- a/src/mbgl/style/custom_tile_loader.cpp +++ b/src/mbgl/style/custom_tile_loader.cpp @@ -11,7 +11,7 @@ CustomTileLoader::CustomTileLoader(const TileFunction& fetchTileFn, const TileFu void CustomTileLoader::fetchTile(const OverscaledTileID& tileID, ActorRef callbackRef) { auto cachedTileData = dataCache.find(tileID.canonical); if (cachedTileData == dataCache.end()) { - fetchTileFunction(tileID.canonical); + invokeTileFetch(tileID.canonical); } else { callbackRef.invoke(&SetTileDataFunction::operator(), *(cachedTileData->second)); } @@ -33,7 +33,7 @@ void CustomTileLoader::fetchTile(const OverscaledTileID& tileID, ActorRef> tileCallbackMap; diff --git a/src/mbgl/tile/custom_tile.cpp b/src/mbgl/tile/custom_tile.cpp index 710c6f6691..7662085499 100644 --- a/src/mbgl/tile/custom_tile.cpp +++ b/src/mbgl/tile/custom_tile.cpp @@ -3,6 +3,7 @@ #include #include #include +#include #include diff --git a/test/style/custom_vector_source.test.cpp b/test/style/custom_vector_source.test.cpp deleted file mode 100644 index 1d9be99a58..0000000000 --- a/test/style/custom_vector_source.test.cpp +++ /dev/null @@ -1,12 +0,0 @@ -#include - -#include -#include -#include -#include - -using namespace mbgl; - -//TODO: AHM: Add tests with real expectations -TEST(CustomVectorSource, EmptyTest) { -} diff --git a/test/style/source.test.cpp b/test/style/source.test.cpp index 919260ffe9..c9a6c13992 100644 --- a/test/style/source.test.cpp +++ b/test/style/source.test.cpp @@ -9,6 +9,7 @@ #include #include #include +#include #include #include @@ -547,3 +548,39 @@ TEST(Source, ImageSourceImageUpdate) { test.run(); } + +TEST(Source, CustomVectorSourceSetTileData) { + SourceTest test; + + CustomVectorSource source("source", CustomVectorSource::Options()); + source.loadDescription(test.fileSource); + + LineLayer layer("id", "source"); + layer.setSourceLayer("water"); + std::vector> layers {{ layer.baseImpl }}; + + test.renderSourceObserver.tileChanged = [&] (RenderSource& source_, const OverscaledTileID&) { + EXPECT_EQ("source", source_.baseImpl->id); + test.end(); + }; + + test.renderSourceObserver.tileError = [&] (RenderSource&, const OverscaledTileID&, std::exception_ptr) { + FAIL() << "Should never be called"; + }; + + auto renderSource = RenderSource::create(source.baseImpl); + renderSource->setObserver(&test.renderSourceObserver); + renderSource->update(source.baseImpl, + layers, + true, + true, + test.tileParameters); + + test.loop.invoke([&] () { + // Set Tile Data + source.setTileData(CanonicalTileID(0, 0, 0), GeoJSON{ FeatureCollection{} }); + }); + + test.run(); +} + diff --git a/test/tile/custom_tile.test.cpp b/test/tile/custom_tile.test.cpp new file mode 100644 index 0000000000..1842bee1f5 --- /dev/null +++ b/test/tile/custom_tile.test.cpp @@ -0,0 +1,131 @@ +#include +#include +#include +#include +#include +#include + +#include +#include +#include +#include +#include +#include +#include +#include +#include + +#include + +using namespace mbgl; +using namespace mbgl::style; + +class CustomTileTest { +public: + FakeFileSource fileSource; + TransformState transformState; + util::RunLoop loop; + ThreadPool threadPool { 1 }; + style::Style style { loop, fileSource, 1 }; + AnnotationManager annotationManager { style }; + ImageManager imageManager; + GlyphManager glyphManager { fileSource }; + + TileParameters tileParameters { + 1.0, + MapDebugOptions(), + transformState, + threadPool, + fileSource, + MapMode::Continuous, + annotationManager, + imageManager, + glyphManager, + 0 + }; +}; + +TEST(CustomTile, InvokeFetchTile) { + CustomTileTest test; + + CircleLayer layer("circle", "source"); + + mapbox::geometry::feature_collection features; + features.push_back(mapbox::geometry::feature { + mapbox::geometry::point(0, 0) + }); + CustomTileLoader loader([&](const CanonicalTileID& tileId) { + EXPECT_EQ(tileId, CanonicalTileID(0,0,0)); + test.loop.stop(); + }, [&](const CanonicalTileID&) { + + }); + auto mb =std::make_shared(*Scheduler::GetCurrent()); + ActorRef loaderActor(loader, mb); + + CustomTile tile(OverscaledTileID(0, 0, 0), "source", test.tileParameters, CustomVectorSource::TileOptions(), + loaderActor); + + tile.setNecessity(Resource::Necessity::Required); + + test.loop.run(); +} + +TEST(CustomTile, InvokeCancelTile) { + CustomTileTest test; + + CircleLayer layer("circle", "source"); + + mapbox::geometry::feature_collection features; + features.push_back(mapbox::geometry::feature { + mapbox::geometry::point(0, 0) + }); + + CustomTileLoader loader([&](const CanonicalTileID&) { }, [&](const CanonicalTileID& tileId) { + EXPECT_EQ(tileId, CanonicalTileID(0,0,0)); + test.loop.stop(); + }); + auto mb =std::make_shared(*Scheduler::GetCurrent()); + ActorRef loaderActor(loader, mb); + + CustomTile tile(OverscaledTileID(0, 0, 0), "source", test.tileParameters, CustomVectorSource::TileOptions(), + loaderActor); + + tile.setNecessity(Resource::Necessity::Required); + tile.setNecessity(Resource::Necessity::Optional); + test.loop.run(); +} + +TEST(CustomTile, InvokeTileChanged) { + CustomTileTest test; + + CircleLayer layer("circle", "source"); + + mapbox::geometry::feature_collection features; + features.push_back(mapbox::geometry::feature { + mapbox::geometry::point(0, 0) + }); + + CustomTileLoader loader(nullptr, nullptr); + auto mb =std::make_shared(*Scheduler::GetCurrent()); + ActorRef loaderActor(loader, mb); + + CustomTile tile(OverscaledTileID(0, 0, 0), "source", test.tileParameters, CustomVectorSource::TileOptions(), + loaderActor); + + StubTileObserver observer; + observer.tileChanged = [&] (const Tile&) { + // Once present, the bucket should never "disappear", which would cause + // flickering. + ASSERT_NE(nullptr, tile.getBucket(*layer.baseImpl)); + }; + + tile.setLayers({{ layer.baseImpl }}); + tile.setObserver(&observer); + tile.setPlacementConfig({}); + tile.setTileData(features); + + while (!tile.isComplete()) { + test.loop.runOnce(); + } +} -- cgit v1.2.1