diff options
author | Christian Mollekopf <chrigi_1@fastmail.fm> | 2016-11-04 12:40:24 +0100 |
---|---|---|
committer | Christian Mollekopf <chrigi_1@fastmail.fm> | 2016-11-04 12:40:24 +0100 |
commit | ba94c4300c52dd80774ed7affc2ef9b4508cb7be (patch) | |
tree | 3c8bf6ce042a4544e81901d901a3a3759d0a128a /tests | |
parent | 98b682acb8c57cadaea5ff0ac6709d21a591b97b (diff) | |
download | sink-ba94c4300c52dd80774ed7affc2ef9b4508cb7be.tar.gz sink-ba94c4300c52dd80774ed7affc2ef9b4508cb7be.zip |
Don't expose the live query flag directly.
Diffstat (limited to 'tests')
-rw-r--r-- | tests/accountstest.cpp | 4 | ||||
-rw-r--r-- | tests/clientapitest.cpp | 9 | ||||
-rw-r--r-- | tests/databasepopulationandfacadequerybenchmark.cpp | 1 | ||||
-rw-r--r-- | tests/dummyresourcetest.cpp | 2 | ||||
-rw-r--r-- | tests/mailquerybenchmark.cpp | 1 | ||||
-rw-r--r-- | tests/modelinteractivitytest.cpp | 2 | ||||
-rw-r--r-- | tests/querytest.cpp | 15 | ||||
-rw-r--r-- | tests/resourceconfigtest.cpp | 2 |
8 files changed, 12 insertions, 24 deletions
diff --git a/tests/accountstest.cpp b/tests/accountstest.cpp index 8216210..88b13f3 100644 --- a/tests/accountstest.cpp +++ b/tests/accountstest.cpp | |||
@@ -99,7 +99,7 @@ private slots: | |||
99 | Store::create(account).exec().waitForFinished(); | 99 | Store::create(account).exec().waitForFinished(); |
100 | 100 | ||
101 | Query query; | 101 | Query query; |
102 | query.liveQuery = true; | 102 | query.setFlags(Query::LiveQuery); |
103 | auto model = Store::loadModel<SinkAccount>(query); | 103 | auto model = Store::loadModel<SinkAccount>(query); |
104 | QTRY_COMPARE(model->rowCount(QModelIndex()), 1); | 104 | QTRY_COMPARE(model->rowCount(QModelIndex()), 1); |
105 | 105 | ||
@@ -130,7 +130,7 @@ private slots: | |||
130 | VERIFYEXEC(Sink::Store::create(res)); | 130 | VERIFYEXEC(Sink::Store::create(res)); |
131 | { | 131 | { |
132 | Sink::Query query; | 132 | Sink::Query query; |
133 | query.liveQuery = true; | 133 | query.setFlags(Query::LiveQuery); |
134 | query.request<Sink::ApplicationDomain::SinkAccount::Status>(); | 134 | query.request<Sink::ApplicationDomain::SinkAccount::Status>(); |
135 | 135 | ||
136 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::SinkAccount>(query); | 136 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::SinkAccount>(query); |
diff --git a/tests/clientapitest.cpp b/tests/clientapitest.cpp index 94c78a7..3f500b7 100644 --- a/tests/clientapitest.cpp +++ b/tests/clientapitest.cpp | |||
@@ -107,7 +107,6 @@ private slots: | |||
107 | 107 | ||
108 | Sink::Query query; | 108 | Sink::Query query; |
109 | query.resourceFilter("dummyresource.instance1"); | 109 | query.resourceFilter("dummyresource.instance1"); |
110 | query.liveQuery = false; | ||
111 | 110 | ||
112 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::Event>(query); | 111 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::Event>(query); |
113 | QTRY_VERIFY(model->data(QModelIndex(), Sink::Store::ChildrenFetchedRole).toBool()); | 112 | QTRY_VERIFY(model->data(QModelIndex(), Sink::Store::ChildrenFetchedRole).toBool()); |
@@ -118,7 +117,6 @@ private slots: | |||
118 | { | 117 | { |
119 | Sink::Query query; | 118 | Sink::Query query; |
120 | query.resourceFilter("nonexisting.resource"); | 119 | query.resourceFilter("nonexisting.resource"); |
121 | query.liveQuery = false; | ||
122 | 120 | ||
123 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::Event>(query); | 121 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::Event>(query); |
124 | QTRY_VERIFY(model->data(QModelIndex(), Sink::Store::ChildrenFetchedRole).toBool()); | 122 | QTRY_VERIFY(model->data(QModelIndex(), Sink::Store::ChildrenFetchedRole).toBool()); |
@@ -132,7 +130,6 @@ private slots: | |||
132 | 130 | ||
133 | Sink::Query query; | 131 | Sink::Query query; |
134 | query.resourceFilter("dummyresource.instance1"); | 132 | query.resourceFilter("dummyresource.instance1"); |
135 | query.liveQuery = false; | ||
136 | 133 | ||
137 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::Folder>(query); | 134 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::Folder>(query); |
138 | QTRY_COMPARE(model->rowCount(), 1); | 135 | QTRY_COMPARE(model->rowCount(), 1); |
@@ -150,7 +147,6 @@ private slots: | |||
150 | // Test | 147 | // Test |
151 | Sink::Query query; | 148 | Sink::Query query; |
152 | query.resourceFilter("dummyresource.instance1"); | 149 | query.resourceFilter("dummyresource.instance1"); |
153 | query.liveQuery = false; | ||
154 | query.parentProperty = "parent"; | 150 | query.parentProperty = "parent"; |
155 | 151 | ||
156 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::Folder>(query); | 152 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::Folder>(query); |
@@ -173,7 +169,6 @@ private slots: | |||
173 | // Test | 169 | // Test |
174 | Sink::Query query; | 170 | Sink::Query query; |
175 | query.resourceFilter("dummyresource.instance1"); | 171 | query.resourceFilter("dummyresource.instance1"); |
176 | query.liveQuery = false; | ||
177 | query.parentProperty = "parent"; | 172 | query.parentProperty = "parent"; |
178 | 173 | ||
179 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::Folder>(query); | 174 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::Folder>(query); |
@@ -196,7 +191,7 @@ private slots: | |||
196 | // Test | 191 | // Test |
197 | Sink::Query query; | 192 | Sink::Query query; |
198 | query.resourceFilter("dummyresource.instance1"); | 193 | query.resourceFilter("dummyresource.instance1"); |
199 | query.liveQuery = true; | 194 | query.setFlags(Sink::Query::LiveQuery); |
200 | query.parentProperty = "parent"; | 195 | query.parentProperty = "parent"; |
201 | 196 | ||
202 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::Folder>(query); | 197 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::Folder>(query); |
@@ -247,7 +242,6 @@ private slots: | |||
247 | ResourceConfig::addResource("dummyresource.instance2", "dummyresource"); | 242 | ResourceConfig::addResource("dummyresource.instance2", "dummyresource"); |
248 | 243 | ||
249 | Sink::Query query; | 244 | Sink::Query query; |
250 | query.liveQuery = false; | ||
251 | 245 | ||
252 | int childrenFetchedCount = 0; | 246 | int childrenFetchedCount = 0; |
253 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::Event>(query); | 247 | auto model = Sink::Store::loadModel<Sink::ApplicationDomain::Event>(query); |
@@ -271,7 +265,6 @@ private slots: | |||
271 | 265 | ||
272 | Sink::Query query; | 266 | Sink::Query query; |
273 | query.resourceFilter("dummyresource.instance1"); | 267 | query.resourceFilter("dummyresource.instance1"); |
274 | query.liveQuery = false; | ||
275 | 268 | ||
276 | bool gotValue = false; | 269 | bool gotValue = false; |
277 | auto result = Sink::Store::fetchOne<Sink::ApplicationDomain::Event>(query) | 270 | auto result = Sink::Store::fetchOne<Sink::ApplicationDomain::Event>(query) |
diff --git a/tests/databasepopulationandfacadequerybenchmark.cpp b/tests/databasepopulationandfacadequerybenchmark.cpp index 834ce30..f1904ad 100644 --- a/tests/databasepopulationandfacadequerybenchmark.cpp +++ b/tests/databasepopulationandfacadequerybenchmark.cpp | |||
@@ -90,7 +90,6 @@ class DatabasePopulationAndFacadeQueryBenchmark : public QObject | |||
90 | const auto startingRss = getCurrentRSS(); | 90 | const auto startingRss = getCurrentRSS(); |
91 | 91 | ||
92 | Sink::Query query; | 92 | Sink::Query query; |
93 | query.liveQuery = false; | ||
94 | query.requestedProperties << "uid" | 93 | query.requestedProperties << "uid" |
95 | << "summary"; | 94 | << "summary"; |
96 | 95 | ||
diff --git a/tests/dummyresourcetest.cpp b/tests/dummyresourcetest.cpp index 0883a13..f8d8543 100644 --- a/tests/dummyresourcetest.cpp +++ b/tests/dummyresourcetest.cpp | |||
@@ -238,7 +238,7 @@ private slots: | |||
238 | void testWriteModifyDeleteLive() | 238 | void testWriteModifyDeleteLive() |
239 | { | 239 | { |
240 | auto query = Query().resourceFilter("sink.dummy.instance1"); | 240 | auto query = Query().resourceFilter("sink.dummy.instance1"); |
241 | query.liveQuery = true; | 241 | query.setFlags(Query::LiveQuery); |
242 | query.filter<Event::Uid>("testuid"); | 242 | query.filter<Event::Uid>("testuid"); |
243 | 243 | ||
244 | auto model = Sink::Store::loadModel<Event>(query); | 244 | auto model = Sink::Store::loadModel<Event>(query); |
diff --git a/tests/mailquerybenchmark.cpp b/tests/mailquerybenchmark.cpp index 90cc4ba..e2c6864 100644 --- a/tests/mailquerybenchmark.cpp +++ b/tests/mailquerybenchmark.cpp | |||
@@ -152,7 +152,6 @@ private slots: | |||
152 | void test50k() | 152 | void test50k() |
153 | { | 153 | { |
154 | Sink::Query query; | 154 | Sink::Query query; |
155 | query.liveQuery = false; | ||
156 | query.request<Mail::Uid>() | 155 | query.request<Mail::Uid>() |
157 | .request<Mail::Subject>() | 156 | .request<Mail::Subject>() |
158 | .request<Mail::Date>(); | 157 | .request<Mail::Date>(); |
diff --git a/tests/modelinteractivitytest.cpp b/tests/modelinteractivitytest.cpp index 6d1c51c..5231c1a 100644 --- a/tests/modelinteractivitytest.cpp +++ b/tests/modelinteractivitytest.cpp | |||
@@ -78,7 +78,7 @@ private slots: | |||
78 | 78 | ||
79 | Sink::Query query; | 79 | Sink::Query query; |
80 | query.resourceFilter("sink.dummy.instance1"); | 80 | query.resourceFilter("sink.dummy.instance1"); |
81 | query.liveQuery = true; | 81 | query.setFlags(Sink::Query::LiveQuery); |
82 | 82 | ||
83 | VERIFYEXEC(Sink::ResourceControl::flushMessageQueue(QByteArrayList() << "sink.dummy.instance1")); | 83 | VERIFYEXEC(Sink::ResourceControl::flushMessageQueue(QByteArrayList() << "sink.dummy.instance1")); |
84 | 84 | ||
diff --git a/tests/querytest.cpp b/tests/querytest.cpp index 9ae3c74..a5e1caf 100644 --- a/tests/querytest.cpp +++ b/tests/querytest.cpp | |||
@@ -50,7 +50,7 @@ private slots: | |||
50 | // Test | 50 | // Test |
51 | Sink::Query query; | 51 | Sink::Query query; |
52 | query.resourceFilter("foobar"); | 52 | query.resourceFilter("foobar"); |
53 | query.liveQuery = true; | 53 | query.setFlags(Query::LiveQuery); |
54 | 54 | ||
55 | // We fetch before the data is available and rely on the live query mechanism to deliver the actual data | 55 | // We fetch before the data is available and rely on the live query mechanism to deliver the actual data |
56 | auto model = Sink::Store::loadModel<Mail>(query); | 56 | auto model = Sink::Store::loadModel<Mail>(query); |
@@ -70,7 +70,7 @@ private slots: | |||
70 | // Test | 70 | // Test |
71 | Sink::Query query; | 71 | Sink::Query query; |
72 | query.resourceFilter("sink.dummy.instance1"); | 72 | query.resourceFilter("sink.dummy.instance1"); |
73 | query.liveQuery = true; | 73 | query.setFlags(Query::LiveQuery); |
74 | 74 | ||
75 | // We fetch before the data is available and rely on the live query mechanism to deliver the actual data | 75 | // We fetch before the data is available and rely on the live query mechanism to deliver the actual data |
76 | auto model = Sink::Store::loadModel<Mail>(query); | 76 | auto model = Sink::Store::loadModel<Mail>(query); |
@@ -88,7 +88,6 @@ private slots: | |||
88 | // Test | 88 | // Test |
89 | Sink::Query query; | 89 | Sink::Query query; |
90 | query.resourceFilter("sink.dummy.instance1"); | 90 | query.resourceFilter("sink.dummy.instance1"); |
91 | query.liveQuery = false; | ||
92 | 91 | ||
93 | // Ensure all local data is processed | 92 | // Ensure all local data is processed |
94 | VERIFYEXEC(Sink::ResourceControl::flushMessageQueue(QByteArrayList() << "sink.dummy.instance1")); | 93 | VERIFYEXEC(Sink::ResourceControl::flushMessageQueue(QByteArrayList() << "sink.dummy.instance1")); |
@@ -119,7 +118,7 @@ private slots: | |||
119 | // Test | 118 | // Test |
120 | Sink::Query query; | 119 | Sink::Query query; |
121 | query.resourceFilter("sink.dummy.instance1"); | 120 | query.resourceFilter("sink.dummy.instance1"); |
122 | query.liveQuery = true; | 121 | query.setFlags(Query::LiveQuery); |
123 | query.filter<Mail::Folder>("folder1"); | 122 | query.filter<Mail::Folder>("folder1"); |
124 | 123 | ||
125 | // We fetch before the data is available and rely on the live query mechanism to deliver the actual data | 124 | // We fetch before the data is available and rely on the live query mechanism to deliver the actual data |
@@ -182,7 +181,7 @@ private slots: | |||
182 | // Test | 181 | // Test |
183 | Sink::Query query; | 182 | Sink::Query query; |
184 | query.resourceFilter("sink.dummy.instance1"); | 183 | query.resourceFilter("sink.dummy.instance1"); |
185 | query.liveQuery = true; | 184 | query.setFlags(Query::LiveQuery); |
186 | 185 | ||
187 | // We fetch before the data is available and rely on the live query mechanism to deliver the actual data | 186 | // We fetch before the data is available and rely on the live query mechanism to deliver the actual data |
188 | auto model = Sink::Store::loadModel<Folder>(query); | 187 | auto model = Sink::Store::loadModel<Folder>(query); |
@@ -253,7 +252,6 @@ private slots: | |||
253 | // Test | 252 | // Test |
254 | Sink::Query query; | 253 | Sink::Query query; |
255 | query.resourceFilter("sink.dummy.instance1"); | 254 | query.resourceFilter("sink.dummy.instance1"); |
256 | query.liveQuery = false; | ||
257 | query.filter<Mail::Uid>("test1"); | 255 | query.filter<Mail::Uid>("test1"); |
258 | 256 | ||
259 | // Ensure all local data is processed | 257 | // Ensure all local data is processed |
@@ -412,7 +410,6 @@ private slots: | |||
412 | query.filter<Mail::Folder>(*folderEntity); | 410 | query.filter<Mail::Folder>(*folderEntity); |
413 | query.sort<Mail::Date>(); | 411 | query.sort<Mail::Date>(); |
414 | query.limit = 1; | 412 | query.limit = 1; |
415 | query.liveQuery = false; | ||
416 | 413 | ||
417 | // Ensure all local data is processed | 414 | // Ensure all local data is processed |
418 | VERIFYEXEC(Sink::ResourceControl::flushMessageQueue(QByteArrayList() << "sink.dummy.instance1")); | 415 | VERIFYEXEC(Sink::ResourceControl::flushMessageQueue(QByteArrayList() << "sink.dummy.instance1")); |
@@ -433,7 +430,7 @@ private slots: | |||
433 | void testReactToNewResource() | 430 | void testReactToNewResource() |
434 | { | 431 | { |
435 | Sink::Query query; | 432 | Sink::Query query; |
436 | query.liveQuery = true; | 433 | query.setFlags(Query::LiveQuery); |
437 | auto model = Sink::Store::loadModel<Folder>(query); | 434 | auto model = Sink::Store::loadModel<Folder>(query); |
438 | QTRY_COMPARE(model->rowCount(QModelIndex()), 0); | 435 | QTRY_COMPARE(model->rowCount(QModelIndex()), 0); |
439 | 436 | ||
@@ -560,7 +557,7 @@ private slots: | |||
560 | Query query; | 557 | Query query; |
561 | query.filter<Mail::Folder>(Sink::Query().containsFilter<Folder::SpecialPurpose>("purpose1")); | 558 | query.filter<Mail::Folder>(Sink::Query().containsFilter<Folder::SpecialPurpose>("purpose1")); |
562 | query.request<Mail::Uid>(); | 559 | query.request<Mail::Uid>(); |
563 | query.liveQuery = true; | 560 | query.setFlags(Query::LiveQuery); |
564 | 561 | ||
565 | auto model = Sink::Store::loadModel<Mail>(query); | 562 | auto model = Sink::Store::loadModel<Mail>(query); |
566 | QTRY_COMPARE(model->rowCount(), 1); | 563 | QTRY_COMPARE(model->rowCount(), 1); |
diff --git a/tests/resourceconfigtest.cpp b/tests/resourceconfigtest.cpp index c06b0fb..df98a61 100644 --- a/tests/resourceconfigtest.cpp +++ b/tests/resourceconfigtest.cpp | |||
@@ -86,7 +86,7 @@ private slots: | |||
86 | VERIFYEXEC(Sink::Store::create(res)); | 86 | VERIFYEXEC(Sink::Store::create(res)); |
87 | { | 87 | { |
88 | Sink::Query query; | 88 | Sink::Query query; |
89 | query.liveQuery = true; | 89 | query.setFlags(Query::LiveQuery); |
90 | query.request<SinkResource::Status>(); | 90 | query.request<SinkResource::Status>(); |
91 | auto model = Sink::Store::loadModel<SinkResource>(query); | 91 | auto model = Sink::Store::loadModel<SinkResource>(query); |
92 | QTRY_COMPARE(model->rowCount(QModelIndex()), 1); | 92 | QTRY_COMPARE(model->rowCount(QModelIndex()), 1); |