restore and fix some broken tests

..more to come, especially several of the
QueryResolver based tests are still broken

src/lib/element-tracker.hpp

@@ -45,6 +45,10 @@
  ** AutoRegistered::is_attached_to and AutoRegistered::deactivateRegistryLink
  ** can be used for detaching an Registry explicitly before destruction.
  ** Ignoring this might lead to segfault on application shutdown. /////////////////TICKET #574
+ ** @warning be sure to understand the implications of object identity,
+ ** especially when it comes to closing and opening the Session. Especially
+ ** note that Assets use a systematic identity, not an instance identity,
+ ** which might lead to removing an old instance from the new session.
  ** 
  ** @see session-element-tracker-test.cpp
  ** @see session-interface-modules.hpp

src/proc/mobject/session/sess-manager-impl.cpp

@@ -245,7 +245,8 @@ namespace session {
   SessManagerImpl::close ()
   {
     Lock sync(this);
-    lifecycle_->shutDown();
+    if (isUp())
+      lifecycle_->shutDown();
     pSess_.reset();
   }
   
@@ -258,7 +259,8 @@ namespace session {
   SessManagerImpl::reset ()
   {
     Lock sync(this);
-    lifecycle_->shutDown();
+    if (isUp())
+      lifecycle_->shutDown();
     lifecycle_->pullUp();
   }
   
@@ -268,7 +270,8 @@ namespace session {
   {
     UNIMPLEMENTED ("load serialised session");
     Lock sync(this);
-    lifecycle_->shutDown();
+    if (isUp())
+      lifecycle_->shutDown();
     lifecycle_->pullUp();
   }
   

tests/40components.tests

@@ -530,7 +530,7 @@ END
 
 
 TEST "Query support utilities" QueryUtils_test  <<END
-out-lit: ..original : a A AA dufte 1a _1 A_A BÄH    White	 space  §&Ω%€GΩ%€ar  ☠☠☠  baäääääge!!!!!  :
+out-lit: ..original : a A AA dufte 1a _1 A_A BÄH    White 	 space  §&Ω%€GΩ%€ar  ☠☠☠  baäääääge!!!!!  :
 out-lit: normalised : a a aA dufte o1a o_1 a_A bH o white_space gar_bage :
 return: 0
 END

tests/components/proc/mobject/session/defs-manager-test.cpp

@@ -112,9 +112,9 @@ namespace test    {
           CHECK (pipe2 == pipe1);
           pipe2 = Pipe::query ("default(X)");
           CHECK (pipe2 == pipe1);
-          pipe2 = Session::current->defaults(Query<Pipe> ());
+          pipe2 = Session::current->defaults(Query<Pipe> (""));
           CHECK (pipe2 == pipe1);
-          pipe2 = asset::Struct::retrieve (Query<Pipe> ());
+          pipe2 = asset::Struct::retrieve (Query<Pipe> (""));
           CHECK (pipe2 == pipe1);
           pipe2 = asset::Struct::retrieve (Query<Pipe> ("default(P)"));
           CHECK (pipe2 == pipe1);
@@ -143,7 +143,7 @@ namespace test    {
       void
       failureCreatesNewDefault()
         {
-          PPipe pipe1 = Session::current->defaults(Query<Pipe> ());   // "the default pipe"
+          PPipe pipe1 = Session::current->defaults(Query<Pipe> ("")); // "the default pipe"
           
           string new_pID (str (format ("dummy_%s_%i")
                                % pipe1->getPipeID()

tests/components/proc/mobject/session/session-element-tracker-test.cpp

@@ -211,11 +211,11 @@ namespace test    {
       /** @test ensure the asset cleanup doesn't interfere with session shutdown
        */
       void
-      verify_cleanup (PTimeline aTimeline_in_session)
+      verify_cleanup (PTimeline const& aTimeline_in_session)
         {
           CHECK (1 < aTimeline_in_session.use_count(), "test object should still be attached to session");
           Session::current.reset();
-          aTimeline_in_session->detach();              // should be a no-op and not cause any invalid access
+          CHECK (1 == aTimeline_in_session.use_count(), "session reset should have de-registered the test object");
         }
     };
   

tests/lib/query/query-utils-test.cpp

@@ -66,7 +66,7 @@ namespace test{
       virtual void
       run (Arg arg) 
         {
-          if (isnil(arg))  arg = Cmdline ("Query normaliseID extractID removeTerm countPred");
+          if (isnil(arg))  arg = Cmdline ("normaliseID extractID removeTerm countPred");
           
           if (contains (arg, "normaliseID"))   check_normaliseID();
           if (contains (arg, "extractID"  ))   check_extractID  ();
@@ -85,11 +85,11 @@ namespace test{
           tokens.push_back ("  White \t space ");
           tokens.push_back ("§&Ω%€GΩ%€ar  ☠☠☠  baäääääge!!!!! ");
           
-          cout << "..original : " << tokens << " :"<<endl;
+          cout << "..original : "<<tokens<<" :"<<endl;
           
           for_each (tokens, normaliseID, _1 );
           
-          cout << "normalised : " << tokens << " :"<<endl;
+          cout << "normalised : "<<tokens<<" :"<<endl;
         }