comparison base/test/TestById.h @ 1752:6d09d68165a4 by-id

Further review of ById: make IDs only available when adding a model to the ById store, not by querying the item directly. This means any id encountered in the wild must have been added to the store at some point (even if later released), which simplifies reasoning about lifecycles

 
 using namespace std;
 
 struct WithoutId {};
 
-struct A : public WithTypedId<A> {};
+// We'll need to change access levels for getId() and getUntypedId()
+// to test the raw calls
+
+struct A : public WithTypedId<A> { public: using WithTypedId<A>::getId; };
 struct B1 : public A {};
 struct B2 : public A {};
 
 struct M {};
 
 typedef TypedById<A, A::Id> AById;
 
-struct X : virtual public WithId {};
+struct X : virtual public WithId { public: using WithId::getUntypedId; };
 struct Y : public X, public B2, public M {};
 
 class TestById : public QObject
 {
     Q_OBJECT

         QVERIFY(!p);
     }
 
     void anySimple() {
         auto a = std::make_shared<A>();
-        AnyById::add(a->getId().untyped, a);
-
-        auto aa = AnyById::getAs<A>(a->getId().untyped);
+        int id = AnyById::add(a);
+        QCOMPARE(id, a->getId().untyped);
+
+        auto aa = AnyById::getAs<A>(id);
         QVERIFY(!!aa);
         QCOMPARE(aa->getId(), a->getId());
         QCOMPARE(aa.get(), a.get()); // same object, not just same id!
-        AnyById::release(a->getId().untyped);
+        AnyById::release(id);
     }
     
     void typedEmpty() {
         auto p = AById::get({});
         QVERIFY(!p);

         QCOMPARE(aa->getId(), a->getId());
         QCOMPARE(aa.get(), a.get()); // same object, not just same id!
         AById::release(a);
     }
 
-    void typedRelease() {
-        auto a = std::make_shared<A>();
-        AById::add(a);
-
-        auto aa = AById::get(a->getId());
-        QVERIFY(!!aa);
-        AById::release(a);
-
-        aa = AById::get(a->getId());
+    void typedReleaseById() {
+        auto a = std::make_shared<A>();
+        auto aid = AById::add(a);
+
+        auto aa = AById::get(aid);
+        QVERIFY(!!aa);
+        AById::release(aid);
+
+        aa = AById::get(aid);
+        QVERIFY(!aa);
+    }
+
+    void typedReleaseByItem() {
+        auto a = std::make_shared<A>();
+        auto aid = AById::add(a);
+
+        auto aa = AById::get(aid);
+        QVERIFY(!!aa);
+        AById::release(a);
+
+        aa = AById::get(aid);
         QVERIFY(!aa);
     }
 
     void typedDowncast() {
         auto a = std::make_shared<A>();