summaryrefslogtreecommitdiff
path: root/render-test/runner.cpp
diff options
context:
space:
mode:
authorAlexander Shalamov <alexander.shalamov@mapbox.com>2019-12-05 17:26:03 +0200
committerAlexander Shalamov <alexander.shalamov@mapbox.com>2019-12-09 17:53:34 +0200
commit3d0001642ba2d6290252f05592d54fd928f56b63 (patch)
treea1fbdbb54f788ea7a038fba4d17b50c2d66bd4d6 /render-test/runner.cpp
parent7eee124677983a01bbbddaa8005b399c277df5dd (diff)
downloadqtlocation-mapboxgl-3d0001642ba2d6290252f05592d54fd928f56b63.tar.gz
[test-runner] Simplify update option and remove obsolete ifdefs
Diffstat (limited to 'render-test/runner.cpp')
-rw-r--r--render-test/runner.cpp8
1 files changed, 0 insertions, 8 deletions
diff --git a/render-test/runner.cpp b/render-test/runner.cpp
index 67cc3aea0d..953576cf78 100644
--- a/render-test/runner.cpp
+++ b/render-test/runner.cpp
@@ -186,7 +186,6 @@ bool TestRunner::checkQueryTestResults(mbgl::PremultipliedImage&& actualImage,
return false;
}
-#if !TEST_READ_ONLY
if (updateResults == UpdateResults::PLATFORM) {
mbgl::filesystem::create_directories(expectations.back());
mbgl::util::write_file(expectations.back().string() + "/expected.json", metadata.actualJson);
@@ -197,7 +196,6 @@ bool TestRunner::checkQueryTestResults(mbgl::PremultipliedImage&& actualImage,
}
mbgl::util::write_file(base + "/actual.json", metadata.actualJson);
-#endif
std::vector<std::string> expectedJsonPaths;
mbgl::filesystem::path expectedMetricsPath;
@@ -256,7 +254,6 @@ bool TestRunner::checkRenderTestResults(mbgl::PremultipliedImage&& actualImage,
return false;
}
-#if !TEST_READ_ONLY
if (updateResults == UpdateResults::PLATFORM) {
mbgl::filesystem::create_directories(expectations.back());
mbgl::util::write_file(expectations.back().string() + "/expected.png", mbgl::encodePNG(actualImage));
@@ -267,7 +264,6 @@ bool TestRunner::checkRenderTestResults(mbgl::PremultipliedImage&& actualImage,
}
mbgl::util::write_file(base + "/actual.png", metadata.actual);
-#endif
mbgl::PremultipliedImage expectedImage{actualImage.size};
mbgl::PremultipliedImage imageDiff{actualImage.size};
@@ -307,9 +303,7 @@ bool TestRunner::checkRenderTestResults(mbgl::PremultipliedImage&& actualImage,
metadata.diff = mbgl::encodePNG(imageDiff);
-#if !TEST_READ_ONLY
mbgl::util::write_file(base + "/diff.png", metadata.diff);
-#endif
metadata.difference = pixels / expectedImage.size.area();
if (metadata.difference <= metadata.allowed) {
@@ -323,13 +317,11 @@ bool TestRunner::checkRenderTestResults(mbgl::PremultipliedImage&& actualImage,
bool TestRunner::checkProbingResults(TestMetadata& metadata) {
if (metadata.metrics.isEmpty()) return true;
const std::vector<mbgl::filesystem::path>& expectedMetrics = metadata.paths.expectedMetrics;
-#if !TEST_READ_ONLY
if (updateResults == UpdateResults::METRICS) {
mbgl::filesystem::create_directories(expectedMetrics.back());
mbgl::util::write_file(expectedMetrics.back().string() + "/metrics.json", serializeMetrics(metadata.metrics));
return true;
}
-#endif
// Check the possible paths in reverse order, so that the default path with the test style will only be checked in
// the very end.