diff options
author | Alexander Shalamov <alexander.shalamov@mapbox.com> | 2020-02-15 12:33:31 +0200 |
---|---|---|
committer | Alexander Shalamov <alexander.shalamov@mapbox.com> | 2020-02-17 10:14:05 +0200 |
commit | d0dfd40b0abe32615d33f867f5b07f8d490e146e (patch) | |
tree | f899b398ec6da5bb11c0ba2991cd26f24ecc0117 /test/map | |
parent | b97f40469b6f3d386ae93bc9a23722ff017162f0 (diff) | |
download | qtlocation-mapboxgl-d0dfd40b0abe32615d33f867f5b07f8d490e146e.tar.gz |
[core] Add unit test for Style::Impl::onSpriteLoaded
Diffstat (limited to 'test/map')
-rw-r--r-- | test/map/map.test.cpp | 55 |
1 files changed, 53 insertions, 2 deletions
diff --git a/test/map/map.test.cpp b/test/map/map.test.cpp index e00f84eff0..0ec97fd3bc 100644 --- a/test/map/map.test.cpp +++ b/test/map/map.test.cpp @@ -10,12 +10,14 @@ #include <mbgl/gl/context.hpp> #include <mbgl/map/map_options.hpp> #include <mbgl/math/log2.hpp> +#include <mbgl/renderer/update_parameters.hpp> #include <mbgl/storage/file_source_manager.hpp> #include <mbgl/storage/main_resource_loader.hpp> #include <mbgl/storage/network_status.hpp> #include <mbgl/storage/online_file_source.hpp> #include <mbgl/storage/resource_options.hpp> #include <mbgl/style/image.hpp> +#include <mbgl/style/image_impl.hpp> #include <mbgl/style/layers/background_layer.hpp> #include <mbgl/style/layers/raster_layer.hpp> #include <mbgl/style/layers/symbol_layer.hpp> @@ -36,13 +38,13 @@ using namespace mbgl; using namespace mbgl::style; using namespace std::literals::string_literals; -template <class FileSource = StubFileSource> +template <class FileSource = StubFileSource, class Frontend = HeadlessFrontend> class MapTest { public: util::RunLoop runLoop; std::shared_ptr<FileSource> fileSource; StubMapObserver observer; - HeadlessFrontend frontend; + Frontend frontend; MapAdapter map; MapTest(float pixelRatio = 1, MapMode mode = MapMode::Static) @@ -1253,3 +1255,52 @@ TEST(Map, PrefetchDeltaOverrideCustomSource) { // and current zoom level + parent tiles would be re-requested. EXPECT_EQ(8, requestedTiles); } + +// Test verifies that Style::Impl::onSpriteLoaded does not create duplicate +// images when we get new spritesheet from the server and we merge it with +// currently used spritesheet. +TEST(Map, TEST_REQUIRES_SERVER(ExpiredSpriteSheet)) { + class ForwardingHeadlessFrontend : public HeadlessFrontend { + public: + using HeadlessFrontend::HeadlessFrontend; + ~ForwardingHeadlessFrontend() override = default; + void update(std::shared_ptr<UpdateParameters> params) override { + if (checkParams) checkParams(params); + HeadlessFrontend::update(std::move(params)); + } + std::function<void(std::shared_ptr<UpdateParameters>)> checkParams; + }; + + MapTest<MainResourceLoader, ForwardingHeadlessFrontend> test{":memory:", ".", 1, MapMode::Continuous}; + + auto makeResponse = [](const std::string& path, bool expires = false) { + Response response; + response.data = std::make_shared<std::string>(util::read_file("test/fixtures/map/online/"s + path)); + response.expires = expires ? Timestamp{Seconds(0)} : Timestamp::max(); + return response; + }; + + test.observer.didFinishLoadingMapCallback = [&test] { + test.frontend.checkParams = [](std::shared_ptr<UpdateParameters> params) { + EXPECT_TRUE(std::is_sorted(params->images->begin(), params->images->end())); + EXPECT_TRUE(params->images->size() < 4u); + }; + + NetworkStatus::Set(NetworkStatus::Status::Online); + + test.observer.didBecomeIdleCallback = [&test] { test.runLoop.stop(); }; + }; + + NetworkStatus::Set(NetworkStatus::Status::Offline); + const std::string prefix = "http://127.0.0.1:3000/online/"; + auto dbfs = FileSourceManager::get()->getFileSource(FileSourceType::Database, ResourceOptions{}); + dbfs->forward(Resource::style(prefix + "style.json"), makeResponse("style.json")); + dbfs->forward(Resource::source(prefix + "streets.json"), makeResponse("streets.json")); + dbfs->forward(Resource::spriteJSON(prefix + "sprite", 1.0), makeResponse("sprite.json", true)); + dbfs->forward(Resource::spriteImage(prefix + "sprite", 1.0), makeResponse("sprite.png", true)); + dbfs->forward(Resource::tile(prefix + "{z}-{x}-{y}.vector.pbf", 1.0, 0, 0, 0, Tileset::Scheme::XYZ), + makeResponse("0-0-0.vector.pbf"), + [&] { test.map.getStyle().loadURL(prefix + "style.json"); }); + + test.runLoop.run(); +} |