Remove the GetCommonSettings mock

There's some sort of internal issue in boost::test and the turtle mocks that causes test crashes

qa/mocks/include/mock_pgm_base.h

@@ -39,7 +39,6 @@ MOCK_BASE_CLASS( MOCK_PGM_BASE, PGM_BASE )
     virtual ~MOCK_PGM_BASE() {};
 
     MOCK_METHOD( MacOpenFile, 1, void( const wxString& ) );
-    MOCK_METHOD( GetCommonSettings, 0, COMMON_SETTINGS*() );
     MOCK_METHOD( SetTextEditor, 1, void( const wxString& ) );
     MOCK_METHOD( GetTextEditor, 1, wxString&( bool ) );
     MOCK_METHOD( AskUserForPreferredEditor, 1, const wxString( const wxString& ) );

qa/tests/api/test_api_module.cpp

@@ -46,8 +46,6 @@ bool init_unit_test()
 
         Pgm().InitPgm( true, true, true );
         const MOCK_PGM_BASE& program = static_cast<MOCK_PGM_BASE&>( Pgm() );
-        MOCK_EXPECT( program.GetCommonSettings )
-                .returns( Pgm().GetSettingsManager().GetCommonSettings() );
     }
 
     return ok;

qa/tests/pcbnew/test_module.cpp

@@ -58,8 +58,6 @@ bool init_unit_test()
         Pgm().GetSettingsManager().RegisterSettings( new PCBNEW_SETTINGS, false );
         Pgm().GetSettingsManager().Load();
         const MOCK_PGM_BASE& program = static_cast<MOCK_PGM_BASE&>( Pgm() );
-        MOCK_EXPECT( program.GetCommonSettings )
-                .returns( Pgm().GetSettingsManager().GetCommonSettings() );
     }
 
     return ok;