summaryrefslogtreecommitdiffstats
path: root/tests/auto
diff options
context:
space:
mode:
authorMårten Nordheim <marten.nordheim@qt.io>2021-01-12 17:25:03 +0100
committerMårten Nordheim <marten.nordheim@qt.io>2021-01-14 10:00:01 +0000
commitd2e2d0d3a6249ccbc17ec92acb32fec9a0d71807 (patch)
tree30238f5e8aa22f0d11a884700df13af71ecf8e46 /tests/auto
parentcb4f853cb05bd57c0a00d9ca41b7e30bdd13bf9a (diff)
QCache: when overwriting, store the new cost instead of the delta
The delta was clearly intended to be used on the total (and still is) but it also wound up getting stored in the cache, which wouldn't be a big problem unless the object was removed, in which case we could incidentally 'free up more space' than intended. Pick-to: 6.0 Change-Id: Ib2b0f072d30da6d16a93dce60e4c5f6080c109fc Reviewed-by: Lars Knoll <lars.knoll@qt.io>
Diffstat (limited to 'tests/auto')
-rw-r--r--tests/auto/corelib/tools/qcache/tst_qcache.cpp14
1 files changed, 14 insertions, 0 deletions
diff --git a/tests/auto/corelib/tools/qcache/tst_qcache.cpp b/tests/auto/corelib/tools/qcache/tst_qcache.cpp
index 8fb7644b65..f8b0aba2dc 100644
--- a/tests/auto/corelib/tools/qcache/tst_qcache.cpp
+++ b/tests/auto/corelib/tools/qcache/tst_qcache.cpp
@@ -49,6 +49,7 @@ private slots:
void axioms_on_key_type();
void largeCache();
void internalChainOrderAfterEntryUpdate();
+ void emplaceLowerCost();
};
@@ -431,5 +432,18 @@ void tst_QCache::internalChainOrderAfterEntryUpdate()
QCOMPARE(cache.size(), 0);
}
+void tst_QCache::emplaceLowerCost()
+{
+ QCache<QString, int> cache;
+ cache.setMaxCost(5);
+ cache.insert("a", new int, 3); // insert high cost
+ cache.insert("a", new int, 1); // and then exchange it with a lower-cost object
+ QCOMPARE(cache.totalCost(), 1);
+ cache.remove("a"); // then remove the object
+ // The cache should now have a cost == 0 and be empty.
+ QCOMPARE(cache.totalCost(), 0);
+ QVERIFY(cache.isEmpty());
+}
+
QTEST_APPLESS_MAIN(tst_QCache)
#include "tst_qcache.moc"