summaryrefslogtreecommitdiff
path: root/test/storage
diff options
context:
space:
mode:
authorMikhail Pozdnyakov <mikhail.pozdnyakov@mapbox.com>2020-02-25 17:50:12 +0200
committerMikhail Pozdnyakov <mikhail.pozdnyakov@mapbox.com>2020-02-25 23:15:47 +0200
commit7c48611f9ee04e1320e1667ea0dae7c355a9935b (patch)
tree31b46f31184833165ee1d53f137e674fcc3069fb /test/storage
parent3c05bd984ada9e146e614165f009b9b45fb05992 (diff)
downloadqtlocation-mapboxgl-7c48611f9ee04e1320e1667ea0dae7c355a9935b.tar.gz
[core] Consolidate file source keys in the generic FileSource header
Thus we do not need to include specific headers to use `{set|get}Property` API with key constants.
Diffstat (limited to 'test/storage')
-rw-r--r--test/storage/offline_download.test.cpp2
-rw-r--r--test/storage/online_file_source.test.cpp10
2 files changed, 6 insertions, 6 deletions
diff --git a/test/storage/offline_download.test.cpp b/test/storage/offline_download.test.cpp
index a15c96d391..ff0965039e 100644
--- a/test/storage/offline_download.test.cpp
+++ b/test/storage/offline_download.test.cpp
@@ -385,7 +385,7 @@ TEST(OfflineDownload, DoesNotFloodTheFileSourceWithRequests) {
fileSource.respond(Resource::Kind::Style, test.response("style.json"));
test.loop.runOnce();
- EXPECT_EQ(*fileSource.getProperty("max-concurrent-requests").getUint(), fileSource.requests.size());
+ EXPECT_EQ(*fileSource.getProperty(MAX_CONCURRENT_REQUESTS_KEY).getUint(), fileSource.requests.size());
}
TEST(OfflineDownload, GetStatusNoResources) {
diff --git a/test/storage/online_file_source.test.cpp b/test/storage/online_file_source.test.cpp
index 88dbf519f8..9d8fbfab64 100644
--- a/test/storage/online_file_source.test.cpp
+++ b/test/storage/online_file_source.test.cpp
@@ -449,7 +449,7 @@ TEST(OnlineFileSource, TEST_REQUIRES_SERVER(LowHighPriorityRequests)) {
const std::size_t NUM_REQUESTS = 3;
NetworkStatus::Set(NetworkStatus::Status::Offline);
- fs.setProperty("max-concurrent-requests", 1u);
+ fs.setProperty(MAX_CONCURRENT_REQUESTS_KEY, 1u);
// After DefaultFileSource was split, OnlineFileSource lives on a separate
// thread. Pause OnlineFileSource, so that messages are queued for processing.
fs.pause();
@@ -492,7 +492,7 @@ TEST(OnlineFileSource, TEST_REQUIRES_SERVER(LowHighPriorityRequestsMany)) {
int correct_regular = 0;
NetworkStatus::Set(NetworkStatus::Status::Offline);
- fs.setProperty("max-concurrent-requests", 1u);
+ fs.setProperty(MAX_CONCURRENT_REQUESTS_KEY, 1u);
fs.pause();
std::vector<std::unique_ptr<AsyncRequest>> collector;
@@ -542,10 +542,10 @@ TEST(OnlineFileSource, TEST_REQUIRES_SERVER(MaximumConcurrentRequests)) {
util::RunLoop loop;
OnlineFileSource fs;
- ASSERT_EQ(*fs.getProperty("max-concurrent-requests").getUint(), 20u);
+ ASSERT_EQ(*fs.getProperty(MAX_CONCURRENT_REQUESTS_KEY).getUint(), 20u);
- fs.setProperty("max-concurrent-requests", 10u);
- ASSERT_EQ(*fs.getProperty("max-concurrent-requests").getUint(), 10u);
+ fs.setProperty(MAX_CONCURRENT_REQUESTS_KEY, 10u);
+ ASSERT_EQ(*fs.getProperty(MAX_CONCURRENT_REQUESTS_KEY).getUint(), 10u);
}
TEST(OnlineFileSource, TEST_REQUIRES_SERVER(RequestSameUrlMultipleTimes)) {