Hotglue the unit tests to work for now

The mocks never expected InitPgm to work and I accidentally fixed it for the kicad-cli

common/pgm_base.cpp

@@ -394,7 +394,7 @@ void PGM_BASE::BuildArgvUtf8()
 }
 
 
-bool PGM_BASE::InitPgm( bool aHeadless, bool aSkipPyInit )
+bool PGM_BASE::InitPgm( bool aHeadless, bool aSkipPyInit, bool aIsUnitTest )
 {
     // Just make sure we init precreate any folders early for later code
     // In particular, the user cache path is the most likely to be hit by startup code
@@ -466,6 +466,12 @@ bool PGM_BASE::InitPgm( bool aHeadless, bool aSkipPyInit )
 
     m_settings_manager = std::make_unique<SETTINGS_MANAGER>( aHeadless );
 
+    // Our unit test mocks break if we continue
+    // A bug caused InitPgm to terminate early in unit tests and the mocks are...simplistic
+    // TODO fix the unit tests so this can be removed
+    if( aIsUnitTest )
+        return false;
+
     // Something got in the way of settings load: can't continue
     if( !m_settings_manager->IsOK() )
         return false;

include/pgm_base.h

@@ -278,7 +278,7 @@ public:
      * Useful in application that do not use python, to disable python dependency at run time
      * @return true if success, false if failure and program is to terminate.
      */
-    bool InitPgm( bool aHeadless = false, bool aSkipPyInit = false );
+    bool InitPgm( bool aHeadless = false, bool aSkipPyInit = false, bool aIsUnitTest = false );
 
     // The PGM_* classes can have difficulties at termination if they
     // are not destroyed soon enough.  Relying on a static destructor can be

qa/unittests/eeschema/test_module.cpp

@@ -59,7 +59,7 @@ bool init_unit_test()
 
     wxSetAssertHandler( &wxAssertThrower );
 
-    Pgm().InitPgm( true, true );
+    Pgm().InitPgm( true, true, true );
 
     return ok;
 }