fix exceptions in constructor not shown properly

They were previously hidden by the "BUG found" message in
ApplicationBase due to the missing call to shutdown(). This is now being
called automatically in the Application destructor if the application
was not yet started fully, in which case this is ok to do there
(calling shutdown before destroying the Application data members is
important only if threads have been started already).

include/Application.h

@@ -49,6 +49,8 @@ namespace ChimeraTK {
      * characters. Use only alphanumeric characters and underscores. */
     explicit Application(const std::string& name);
 
+    ~Application() override;
+
     using ApplicationBase::getName;
 
     /** This will remove the global pointer to the instance and allows creating

src/Application.cc

@@ -46,6 +46,16 @@ Application::Application(const std::string& name) : ApplicationBase(name), Modul
 
 /*********************************************************************************************************************/
 
+Application::~Application() {
+  if(lifeCycleState == LifeCycleState::initialisation && !hasBeenShutdown) {
+    // likely an exception has been thrown in the initialisation phase, in which case we better call shutdown to prevent
+    // ApplicationBase from complaining and hiding the exception
+    ApplicationBase::shutdown();
+  }
+}
+
+/*********************************************************************************************************************/
+
 void Application::enableTestableMode() {
   assert(not initialiseCalled);
   testableMode.enable();