test_library_tables.cpp

Go to the documentation of this file.

/*
 * This program source code file is part of KiCad, a free EDA CAD application.
 *
 * Copyright The KiCad Developers, see AUTHORS.txt for contributors.
 * @author Jon Evans <[email protected]>
 *
 * This program is free software: you can redistribute it and/or modify it
 * under the terms of the GNU General Public License as published by the
 * Free Software Foundation, either version 3 of the License, or (at your
 * option) any later version.
 *
 * This program is distributed in the hope that it will be useful, but
 * WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
 * General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program.  If not, see <https://www.gnu.org/licenses/>.
 */
 
#include <algorithm>
#include <filesystem>
#include <fstream>
#include <optional>
#include <utility>
#include <vector>
 
#include <wx/ffile.h>
 
#include <mock_pgm_base.h>
#include <richio.h>
#include <io/kicad/kicad_io_utils.h>
#include <qa_utils/env_var_utils.h>
#include <qa_utils/file_utils.h>
#include <qa_utils/wx_utils/unit_test_utils.h>
#include <settings/settings_manager.h>
#include <pegtl/contrib/analyze.hpp>
 
#include <env_vars.h>
#include <pgm_base.h>
#include <settings/common_settings.h>
#include <libraries/library_manager.h>
#include <libraries/library_table.h>
#include <libraries/library_table_parser.h>
#include <libraries/library_table_grammar.h>
#include <scoped_set_reset.h>
#include <settings/kicad_settings.h>
#include <startwizard/startwizard_provider_libraries.h>
 
 
BOOST_AUTO_TEST_SUITE( LibraryTables )
 
 
BOOST_AUTO_TEST_CASE( Grammar )
{
    BOOST_REQUIRE( tao::pegtl::analyze< LIBRARY_TABLE_GRAMMAR::LIB_TABLE_FILE >( 1 ) == 0 );
}
 
 
BOOST_AUTO_TEST_CASE( EmptyString )
{
    LIBRARY_TABLE_PARSER parser;
    tl::expected<LIBRARY_TABLE_IR, LIBRARY_PARSE_ERROR> result = parser.ParseBuffer( "" );
    BOOST_REQUIRE( !result.has_value() );
}
 
 
BOOST_AUTO_TEST_CASE( ParseFromFile )
{
    std::vector<std::string> cases = {
        "sym-lib-table",
        "fp-lib-table"
    };
 
    std::filesystem::path p( KI_TEST::GetTestDataRootDir() );
    p.append( "libraries/" );
 
    LIBRARY_TABLE_PARSER parser;
 
    for( const std::string& path : cases )
    {
        p.remove_filename();
        p.append( path );
 
        auto result = parser.Parse( p );
 
        BOOST_REQUIRE( result.has_value() );
    }
}
 
 
BOOST_AUTO_TEST_CASE( ParseAndConstruct )
{
    struct TESTCASE
    {
        wxString filename;
        wxString expected_error;
        size_t expected_rows;
        bool check_formatted = true;
    };
 
    std::vector<TESTCASE> cases = {
        { .filename = "sym-lib-table", .expected_rows = 224 },
        { .filename = "fp-lib-table", .expected_rows = 146 },
        { .filename = "nested-symbols", .expected_rows = 6 },
        { .filename = "nested-disabled", .expected_rows = 4 },
        { .filename = "nested-hidden", .expected_rows = 4 },
        { .filename = "cycle", .expected_rows = 2 },
        { .filename = "sym-hand-edited", .expected_rows = 2, .check_formatted = false },
        { .filename = "corrupted", .expected_error = "Syntax error at line 6, column 9" },
        { .filename = "truncated", .expected_error = "Syntax error at line 11, column 1" }
    };
 
    wxFileName fn( KI_TEST::GetTestDataRootDir(), wxEmptyString );
    fn.AppendDir( "libraries" );
 
    for( const auto& [filename, expected_error, expected_rows, check_formatted] : cases )
    {
        BOOST_TEST_CONTEXT( filename )
        {
            fn.SetName( filename );
            LIBRARY_TABLE table( fn, LIBRARY_TABLE_SCOPE::GLOBAL );
 
            BOOST_REQUIRE( table.IsOk() == ( expected_error.IsEmpty() ) );
 
            BOOST_REQUIRE_MESSAGE( table.Rows().size() == expected_rows,
                                   wxString::Format( "Expected %zu rows but got %zu",
                                                     expected_rows, table.Rows().size() ) );
 
            BOOST_REQUIRE_MESSAGE( table.ErrorDescription() == expected_error,
                                   wxString::Format( "Expected error '%s' but got '%s'",
                                                     expected_error, table.ErrorDescription() ) );
 
            // Non-parsed tables can't be formatted
            if( !table.IsOk() || !check_formatted )
                continue;
 
            std::ifstream inFp;
            inFp.open( fn.GetFullPath().fn_str() );
            BOOST_REQUIRE( inFp.is_open() );
 
            std::stringstream inBuf;
            inBuf << inFp.rdbuf();
            std::string inData = inBuf.str();
 
            STRING_FORMATTER formatter;
            table.Format( &formatter );
            KICAD_FORMAT::Prettify( formatter.MutableString(),
                                    KICAD_FORMAT::FORMAT_MODE::LIBRARY_TABLE );
 
            if( formatter.GetString().compare( inData ) != 0 )
            {
                BOOST_TEST_MESSAGE( "--- original ---" );
                BOOST_TEST_MESSAGE( inData );
                BOOST_TEST_MESSAGE( "--- formatted ---" );
                BOOST_TEST_MESSAGE( formatter.GetString() );
            }
 
            BOOST_REQUIRE( formatter.GetString().compare( inData ) == 0 );
        }
    }
}
 
BOOST_AUTO_TEST_CASE( Manager )
{
    LIBRARY_MANAGER manager;
    manager.LoadGlobalTables();
 
    BOOST_REQUIRE( manager.Rows( LIBRARY_TABLE_TYPE::SYMBOL ).size() == 3 );
    BOOST_REQUIRE( manager.Rows( LIBRARY_TABLE_TYPE::FOOTPRINT ).size() == 146 );
}
 
 
/*
 * Regression test for PCM library auto-add scan.
 *
 * The PCM auto-add scan is triggered by the presence of PCM packages in the 3RD_PARTY directory.
 * But when this happens, if there is no global design block table, this process shouldn't die.
 *
 * Sentry KICAD-17T5 was a startup crash when the design block table was missing.
 */
BOOST_AUTO_TEST_CASE( PcmScanWithoutDesignBlockTable )
{
    // Create a temporary directory with a PCM package containing a design block library
    KI_TEST::SCOPED_TEMP_DIR tempDir( wxS( "pcm-scan" ) );
 
    const bool dirsOk =
            std::filesystem::create_directories( tempDir.Path() / "design_blocks" / "test" / "block.kicad_blocks" );
    BOOST_REQUIRE( dirsOk );
 
    // PCM auto-add must be on
    bool& autoAddSetting = Pgm().GetSettingsManager().GetAppSettings<KICAD_SETTINGS>( "kicad" )->m_PcmLibAutoAdd;
    SCOPED_SET_RESET<bool> autoAddSettingReset( autoAddSetting, true );
 
    KI_TEST::SCOPED_PGM_ENV_VAR envVar( wxS( "KICAD10_3RD_PARTY" ), tempDir.PathStr() );
 
    LIBRARY_MANAGER manager;
 
    // Only the symbol table is loaded; the design block table is absent.
    manager.LoadGlobalTables( { LIBRARY_TABLE_TYPE::SYMBOL } );
 
    BOOST_REQUIRE( manager.Table( LIBRARY_TABLE_TYPE::SYMBOL, LIBRARY_TABLE_SCOPE::GLOBAL ).has_value() );
    BOOST_REQUIRE( !manager.Table( LIBRARY_TABLE_TYPE::DESIGN_BLOCK, LIBRARY_TABLE_SCOPE::GLOBAL ).has_value() );
}
 
 
// Regression coverage for the LoadGlobalTables guard: when a global library
// table file does not exist on disk, loadTables() never inserts an entry for
// that type into m_tables, so Table( aType, GLOBAL ) must return std::nullopt
// instead of an entry holding a null pointer. The cleanupRemovedPCMLibraries
// lambda inside LoadGlobalTables relies on this contract via .value_or(
// nullptr ) and must not dereference the result. Exercising the full PCM
// cleanup path requires extensive environment setup (3RD_PARTY env var plus
// PCM auto-remove enabled in user settings), so we test the underlying
// contract directly here.
BOOST_AUTO_TEST_CASE( ManagerTableReturnsNulloptForUnloadedType )
{
    LIBRARY_MANAGER manager;
 
    auto result = manager.Table( LIBRARY_TABLE_TYPE::SYMBOL, LIBRARY_TABLE_SCOPE::GLOBAL );
    BOOST_REQUIRE( !result.has_value() );
 
    LIBRARY_TABLE* table = result.value_or( nullptr );
    BOOST_REQUIRE( table == nullptr );
}
 
 
BOOST_AUTO_TEST_CASE( NestedTablesDisabledHidden )
{
    // Test that disabled and hidden nested library table rows are parsed correctly
    // This is a regression test for https://gitlab.com/kicad/code/kicad/-/issues/22784
    // Note that full end-to-end testing requires the library manager to process the tables,
    // but the parse test verifies the flag is correctly read from disk.
 
    wxFileName fn( KI_TEST::GetTestDataRootDir(), wxEmptyString );
    fn.AppendDir( "libraries" );
 
    // Test with the disabled nested table
    fn.SetName( "nested-disabled" );
    LIBRARY_TABLE disabledTable( fn, LIBRARY_TABLE_SCOPE::GLOBAL );
    BOOST_REQUIRE( disabledTable.IsOk() );
    BOOST_REQUIRE_MESSAGE( disabledTable.Rows().size() == 4,
                           wxString::Format( "Expected 4 rows but got %zu",
                                             disabledTable.Rows().size() ) );
 
    // Verify the disabled flag is parsed correctly on the nested table row
    bool foundDisabledRow = false;
 
    for( const LIBRARY_TABLE_ROW& row : disabledTable.Rows() )
    {
        if( row.Type() == LIBRARY_TABLE_ROW::TABLE_TYPE_NAME )
        {
            BOOST_REQUIRE_MESSAGE( row.Disabled(),
                                   "Nested table row should have disabled flag set" );
            foundDisabledRow = true;
        }
    }
 
    BOOST_REQUIRE_MESSAGE( foundDisabledRow,
                           "Disabled nested table row not found in parsed table" );
 
    // Test hidden nested table has same behavior
    fn.SetName( "nested-hidden" );
    LIBRARY_TABLE hiddenTable( fn, LIBRARY_TABLE_SCOPE::GLOBAL );
    BOOST_REQUIRE( hiddenTable.IsOk() );
    BOOST_REQUIRE_MESSAGE( hiddenTable.Rows().size() == 4,
                           wxString::Format( "Expected 4 rows but got %zu",
                                             hiddenTable.Rows().size() ) );
 
    bool foundHiddenRow = false;
 
    for( const LIBRARY_TABLE_ROW& row : hiddenTable.Rows() )
    {
        if( row.Type() == LIBRARY_TABLE_ROW::TABLE_TYPE_NAME )
        {
            BOOST_REQUIRE_MESSAGE( row.Hidden(),
                                   "Nested table row should have hidden flag set" );
            foundHiddenRow = true;
        }
    }
 
    BOOST_REQUIRE_MESSAGE( foundHiddenRow,
                           "Hidden nested table row not found in parsed table" );
}
 

BOOST_AUTO_TEST_CASE( InsertRowPreservesExistingRowPointers )
{
    LIBRARY_TABLE table( true, wxEmptyString, LIBRARY_TABLE_SCOPE::PROJECT );
    table.SetType( LIBRARY_TABLE_TYPE::SYMBOL );
 
    // Seed with a few rows and snapshot pointers plus the expected URIs.
    std::vector<const LIBRARY_TABLE_ROW*> seededPointers;
    std::vector<wxString>                 seededUris;
 
    for( int i = 0; i < 4; ++i )
    {
        LIBRARY_TABLE_ROW& row = table.InsertRow();
        row.SetNickname( wxString::Format( wxS( "seed_%d" ), i ) );
        row.SetURI( wxString::Format( wxS( "${KIPRJMOD}/libs/seed_%d.kicad_sym" ), i ) );
        row.SetType( wxS( "KiCad" ) );
 
        seededPointers.push_back( &row );
        seededUris.push_back( row.URI() );
    }
 
    // Insert additional rows to force container growth that would reallocate
    // a std::vector, and verify the seeded pointers continue to resolve to the
    // same logical rows (same nickname and URI).
    for( int i = 0; i < 64; ++i )
    {
        LIBRARY_TABLE_ROW& row = table.InsertRow();
        row.SetNickname( wxString::Format( wxS( "extra_%d" ), i ) );
        row.SetURI( wxString::Format( wxS( "${KIPRJMOD}/libs/extra_%d.kicad_sym" ), i ) );
        row.SetType( wxS( "KiCad" ) );
 
        for( size_t j = 0; j < seededPointers.size(); ++j )
        {
            BOOST_REQUIRE_MESSAGE(
                    seededPointers[j]->URI() == seededUris[j],
                    wxString::Format(
                            wxS( "Seed row %zu pointer was invalidated after inserting %d rows: "
                                 "expected URI '%s', got '%s'" ),
                            j, i + 1, seededUris[j], seededPointers[j]->URI() ) );
        }
    }
}
 

BOOST_AUTO_TEST_CASE( IsPcmManagedRow_URITemplateMatching )
{
    struct CASE
    {
        wxString uri;
        bool     expectedPcmManaged;
        wxString description;
    };
 
    std::vector<CASE> cases = {
        { wxS( "${KICAD10_3RD_PARTY}/symbols/foo/foo.kicad_sym" ), true,
          wxS( "Versioned 3RD_PARTY template should be recognised as PCM-managed" ) },
        { wxS( "${KICAD9_3RD_PARTY}/symbols/legacy/legacy.kicad_sym" ), true,
          wxS( "Legacy versioned 3RD_PARTY template should still match the wildcard" ) },
        { wxS( "${KICAD10_3RD_PARTY}/footprints/bar/bar.pretty" ), true,
          wxS( "Footprint library using 3RD_PARTY template should match" ) },
        { wxS( "${KICAD10_3RD_PARTY}/design_blocks/baz/baz.kicad_blocks" ), true,
          wxS( "Design block library using 3RD_PARTY template should match" ) },
        { wxS( "${KICAD_USER_LIB}/symbols/test.kicad_sym" ), false,
          wxS( "Row using a different env var must not be flagged as PCM-managed" ) },
        { wxS( "${KIPRJMOD}/libs/local.kicad_sym" ), false,
          wxS( "Project-relative row must not be flagged as PCM-managed" ) },
        { wxS( "/abs/path/to/lib.kicad_sym" ), false,
          wxS( "Absolute path row must not be flagged as PCM-managed" ) },
        { wxS( "${}" ), false,
          wxS( "Malformed empty var name must not match" ) },
        { wxS( "${KICAD10_3RD_PARTY_EXTRA}/foo" ), false,
          wxS( "Similar-but-different var name must not match" ) },
        // Issue #23476: a user who repurposes KICADn_3RD_PARTY to point at their own
        // library collection adds libraries directly under that root (no PCM category
        // folder). Such rows must not be treated as PCM-managed or auto-remove deletes
        // them.
        { wxS( "${KICAD10_3RD_PARTY}/mylib.kicad_sym" ), false,
          wxS( "User library directly under repurposed 3RD_PARTY root must not match" ) },
        { wxS( "${KICAD10_3RD_PARTY}/MyLibs/mylib.kicad_sym" ), false,
          wxS( "User library under a non-PCM subfolder of 3RD_PARTY must not match" ) },
        { wxS( "${KICAD10_3RD_PARTY}/eagle/imported.pretty" ), false,
          wxS( "User footprint library under a non-PCM subfolder must not match" ) },
        { wxS( "${KICAD10_3RD_PARTY}/symbols" ), false,
          wxS( "3RD_PARTY/symbols with no nested package must not match" ) },
        { wxS( "${KICAD10_3RD_PARTY}/symbols/mylib.kicad_sym" ), false,
          wxS( "User symbol library directly in symbols folder (no package level) must not match" ) },
        { wxS( "${KICAD10_3RD_PARTY}symbols/foo/foo.kicad_sym" ), false,
          wxS( "Missing separator after env var must not match" ) },
        { wxS( "${KICAD10_3RD_PARTY}/design_blocks/baz/baz.kicad_dbl" ), false,
          wxS( "Wrong design-block library extension must not match" ) },
        { wxS( "${KICAD10_3RD_PARTY}/symbols/foo/foo.txt" ), false,
          wxS( "Non-library file in PCM symbols tree must not match" ) },
        { wxS( "${KICAD10_3RD_PARTY}\\symbols\\foo\\foo.kicad_sym" ), false,
          wxS( "Backslash-separated URI is never emitted by PCM and must not match" ) },
        { wxS( "${KICAD10_3RD_PARTY}/symbols//foo.kicad_sym" ), false,
          wxS( "Empty package-id component must not match" ) },
        { wxS( "${KICAD10_3RD_PARTY}/symbols/foo/.kicad_sym" ), false,
          wxS( "Extension-only leaf with empty library stem must not match" ) },
    };
 
    for( const CASE& c : cases )
    {
        LIBRARY_TABLE_ROW row;
        row.SetURI( c.uri );
 
        bool actual = LIBRARY_MANAGER::IsPcmManagedRow( row );
 
        BOOST_CHECK_MESSAGE(
                actual == c.expectedPcmManaged,
                wxString::Format( wxS( "%s: URI='%s' expected=%d actual=%d" ),
                                  c.description, c.uri, c.expectedPcmManaged ? 1 : 0,
                                  actual ? 1 : 0 ) );
    }
}
 
 
BOOST_AUTO_TEST_CASE( ReadOnlyTable )
{
    // Create a temporary copy of a library table and make it read-only
    wxFileName fn( KI_TEST::GetTestDataRootDir(), wxEmptyString );
    fn.AppendDir( "libraries" );
    fn.SetName( "sym-lib-table" );
 
    KI_TEST::SCOPED_TEMP_DIR tempDir( "kicad_test_ro_table" );
    wxFileName               tmpFn = tempDir.CreateChildFileStr( "sym-lib-table" );
 
    wxCopyFile( fn.GetFullPath(), tmpFn.GetFullPath() );
 
    // Verify a writable table is not read-only
    {
        LIBRARY_TABLE writableTable( tmpFn, LIBRARY_TABLE_SCOPE::GLOBAL );
        BOOST_REQUIRE( writableTable.IsOk() );
        BOOST_REQUIRE( !writableTable.IsReadOnly() );
    }
 
    // Make the file read-only
    tmpFn.SetPermissions( wxS_IRUSR | wxS_IRGRP | wxS_IROTH );
 
    // CI containers commonly run as root, where access(W_OK) succeeds regardless of the
    // permission bits, so a read-only file still reports as writable.  IsReadOnly() uses the
    // same IsFileWritable() check, so when the file remains writable here the read-only
    // assertions cannot hold and are not meaningful.
    if( wxFileName( tmpFn.GetFullPath() ).IsFileWritable() )
    {
        BOOST_TEST_MESSAGE( "Skipping read-only table checks; file remains writable despite "
                            "read-only permissions (running as root?)" );
    }
    else
    {
        LIBRARY_TABLE roTable( tmpFn, LIBRARY_TABLE_SCOPE::GLOBAL );
        BOOST_REQUIRE( roTable.IsOk() );
        BOOST_REQUIRE( roTable.IsReadOnly() );
 
        // Save should return an error for read-only tables
        LIBRARY_RESULT<void> result = roTable.Save();
        BOOST_REQUIRE( !result.has_value() );
    }
 
    // Clean up
    tmpFn.SetPermissions( wxS_IRUSR | wxS_IWUSR );
    wxRemoveFile( tmpFn.GetFullPath() );
}
 

BOOST_AUTO_TEST_CASE( RowsAddedToANewProjectTableAreEnumerated )
{
    std::error_code       ec;
    std::filesystem::path dir =
            std::filesystem::temp_directory_path( ec ) / std::filesystem::path( "kicad_qa_new_project_table" );
 
    std::filesystem::remove_all( dir, ec );
    std::filesystem::create_directories( dir, ec );
 
    std::filesystem::path proPath = dir / "new_project.kicad_pro";
 
    {
        std::ofstream proFile( proPath );
        proFile << R"({ "meta": { "filename": "new_project.kicad_pro", "version": 3 } })";
    }
 
    SETTINGS_MANAGER& settings = Pgm().GetSettingsManager();
    BOOST_REQUIRE( settings.LoadProject( wxString( proPath.string() ) ) );
 
    wxFileName tableFile( wxString( ( dir / "sym-lib-table" ).string() ) );
    BOOST_REQUIRE( !tableFile.FileExists() );
 
    LIBRARY_MANAGER manager;
 
    std::optional<LIBRARY_TABLE*> optTable = manager.Table( LIBRARY_TABLE_TYPE::SYMBOL, LIBRARY_TABLE_SCOPE::PROJECT );
    BOOST_REQUIRE( optTable.has_value() );
 
    LIBRARY_TABLE*     table = optTable.value();
    LIBRARY_TABLE_ROW& row = table->InsertRow();
 
    row.SetNickname( wxS( "NewLib" ) );
    row.SetURI( wxS( "${KIPRJMOD}/NewLib.kicad_sym" ) );
    row.SetType( wxS( "KiCad" ) );
 
    auto projectRowCount = [&]() -> size_t
    {
        return manager.Rows( LIBRARY_TABLE_TYPE::SYMBOL, LIBRARY_TABLE_SCOPE::PROJECT ).size();
    };
 
    BOOST_CHECK_MESSAGE( projectRowCount() == 1,
                         "A library added to a project with no library table must be visible to "
                         "Rows(), which is what fills the library tree" );
 
    BOOST_REQUIRE( table->Save().has_value() );
    BOOST_CHECK_EQUAL( projectRowCount(), 1 );
 
    settings.UnloadProject( &settings.Prj(), false );
    std::filesystem::remove_all( dir, ec );
}
 
 
BOOST_AUTO_TEST_CASE( LibOverrideSettings )
{
    // Test that LIB_OVERRIDE serialization in KICAD_SETTINGS works via the
    // LIBRARY_MANAGER override API.
    LIBRARY_MANAGER manager;
 
    wxString tablePath = wxT( "/some/read-only/path/sym-lib-table" );
    wxString nickname1 = wxT( "LibA" );
    wxString nickname2 = wxT( "LibB" );
 
    // Set an override
    manager.SetLibOverride( tablePath, nickname1, true, false );
    manager.SetLibOverride( tablePath, nickname2, false, true );
 
    // Verify overrides via settings
    SETTINGS_MANAGER& mgr = Pgm().GetSettingsManager();
    KICAD_SETTINGS*   settings = mgr.GetAppSettings<KICAD_SETTINGS>( "kicad" );
 
    BOOST_REQUIRE( settings != nullptr );
    BOOST_REQUIRE( settings->m_LibOverrides.count( tablePath ) == 1 );
    BOOST_REQUIRE( settings->m_LibOverrides[tablePath].count( nickname1 ) == 1 );
    BOOST_REQUIRE( settings->m_LibOverrides[tablePath][nickname1].disabled == true );
    BOOST_REQUIRE( settings->m_LibOverrides[tablePath][nickname1].hidden == false );
    BOOST_REQUIRE( settings->m_LibOverrides[tablePath][nickname2].disabled == false );
    BOOST_REQUIRE( settings->m_LibOverrides[tablePath][nickname2].hidden == true );
 
    // Clear override (both disabled and hidden are false)
    manager.ClearLibOverride( tablePath, nickname1 );
    BOOST_REQUIRE( settings->m_LibOverrides[tablePath].count( nickname1 ) == 0 );
 
    // Clear last entry should remove the table key too
    manager.ClearLibOverride( tablePath, nickname2 );
    BOOST_REQUIRE( settings->m_LibOverrides.count( tablePath ) == 0 );
 
    // SetLibOverride with both false should also clear
    manager.SetLibOverride( tablePath, nickname1, true, false );
    BOOST_REQUIRE( settings->m_LibOverrides.count( tablePath ) == 1 );
    manager.SetLibOverride( tablePath, nickname1, false, false );
    BOOST_REQUIRE( settings->m_LibOverrides.count( tablePath ) == 0 );
}
 

BOOST_AUTO_TEST_CASE( CreateGlobalTableEmptyWhenNoStockTable )
{
    wxFileName stockPath( LIBRARY_MANAGER::StockTablePath( LIBRARY_TABLE_TYPE::DESIGN_BLOCK ) );
 
    if( stockPath.IsFileReadable() )
    {
        BOOST_TEST_MESSAGE( "Skipping: stock design-block-lib-table exists; test only applies when "
                            "no stock table is installed" );
        return;
    }
 
    const wxString tablePath =
            LIBRARY_MANAGER::DefaultGlobalTablePath( LIBRARY_TABLE_TYPE::DESIGN_BLOCK );
 
    // RAII guard: restore the global table file to its original state after the test.
    struct FILE_RESTORE
    {
        wxString path;
        bool     existed = false;
        wxString contents;
 
        explicit FILE_RESTORE( const wxString& aPath ) : path( aPath )
        {
            existed = wxFileName::FileExists( path );
 
            if( existed )
            {
                wxFFile in( path, wxT( "rb" ) );
                in.ReadAll( &contents );
            }
        }
 
        ~FILE_RESTORE()
        {
            if( existed )
            {
                wxFFile out( path, wxT( "wb" ) );
                out.Write( contents );
            }
            else if( wxFileName::FileExists( path ) )
            {
                wxRemoveFile( path );
            }
        }
    } restore( tablePath );
 
    BOOST_REQUIRE( LIBRARY_MANAGER::CreateGlobalTable( LIBRARY_TABLE_TYPE::DESIGN_BLOCK, true ) );
 
    LIBRARY_TABLE reloaded( wxFileName( tablePath ), LIBRARY_TABLE_SCOPE::GLOBAL );
 
    BOOST_REQUIRE( reloaded.IsOk() );
    BOOST_CHECK_MESSAGE( reloaded.Rows().empty(),
                         "CreateGlobalTable must not write a dangling row when the stock table is absent" );
}
 

BOOST_AUTO_TEST_CASE( StockTableReferenceURIHonorsExternalDefinition )
{
    const wxString templateVar = ENV_VAR::GetVersionedEnvVarName( wxS( "TEMPLATE_DIR" ) );
 
    // Preserve and restore the original entry so neighbouring tests are unaffected.
    KI_TEST::SCOPED_PGM_ENV_VAR envVarGuard( templateVar, wxEmptyString );
 
    for( LIBRARY_TABLE_TYPE type : { LIBRARY_TABLE_TYPE::SYMBOL, LIBRARY_TABLE_TYPE::FOOTPRINT,
                                     LIBRARY_TABLE_TYPE::DESIGN_BLOCK } )
    {
        ENV_VAR_ITEM& entry = envVarGuard.GetItem();
 
        entry.SetDefinedExternally( false );
        BOOST_CHECK_EQUAL( LIBRARY_MANAGER::StockTableReferenceURI( type ),
                           LIBRARY_MANAGER::StockTablePath( type ) );
 
        entry.SetDefinedExternally( true );
        BOOST_CHECK_EQUAL( LIBRARY_MANAGER::StockTableReferenceURI( type ),
                           LIBRARY_MANAGER::StockTableTokenizedURI( type ) );
    }
}
 

static LIBRARY_TABLE makeImportedSymbolTable( const std::vector<std::pair<wxString, wxString>>& aUserRows )
{
    LIBRARY_TABLE table( true, wxEmptyString, LIBRARY_TABLE_SCOPE::GLOBAL );
    table.SetType( LIBRARY_TABLE_TYPE::SYMBOL );
 
    for( const auto& [nickname, uri] : aUserRows )
    {
        LIBRARY_TABLE_ROW& row = table.InsertRow();
        row.SetNickname( nickname );
        row.SetURI( uri );
        row.SetType( wxS( "KiCad" ) );
    }
 
    return table;
}
 
 
static size_t countChainedKiCadRows( const LIBRARY_TABLE& aTable )
{
    return std::ranges::count_if( aTable.Rows(),
            []( const LIBRARY_TABLE_ROW& aRow )
            {
                return aRow.Type() == LIBRARY_TABLE_ROW::TABLE_TYPE_NAME
                       && aRow.Nickname() == wxS( "KiCad" );
            } );
}
 

BOOST_AUTO_TEST_CASE( MigrateBuiltInLibraries_NoStockRefsAddsNothing )
{
    const wxString stockPath = wxS( "${KICAD10_SYMBOL_DIR}/sym-lib-table" );
 
    LIBRARY_TABLE table = makeImportedSymbolTable( {
        { wxS( "MyParts" ),   wxS( "${KIPRJMOD}/../libs/MyParts.kicad_sym" ) },
        { wxS( "MyPassives" ), wxS( "/home/user/kicad/MyPassives.kicad_sym" ) },
    } );
 
    const size_t rowsBefore = table.Rows().size();
 
    bool modified = STARTWIZARD_PROVIDER_LIBRARIES::MigrateBuiltInLibraries(
            table, LIBRARY_TABLE_TYPE::SYMBOL, stockPath, true );
 
    BOOST_CHECK_MESSAGE( !modified, "Table with no stock references should not be modified" );
    BOOST_CHECK_EQUAL( table.Rows().size(), rowsBefore );
    BOOST_CHECK_EQUAL( countChainedKiCadRows( table ), 0u );
}
 

BOOST_AUTO_TEST_CASE( MigrateBuiltInLibraries_DirectStockRowsBecomeChained )
{
    const wxString stockPath = wxS( "${KICAD10_SYMBOL_DIR}/sym-lib-table" );
 
    LIBRARY_TABLE table = makeImportedSymbolTable( {
        { wxS( "Device" ),  wxS( "${KICAD9_SYMBOL_DIR}/Device.kicad_sym" ) },
        { wxS( "MyParts" ), wxS( "${KIPRJMOD}/../libs/MyParts.kicad_sym" ) },
    } );
 
    bool modified = STARTWIZARD_PROVIDER_LIBRARIES::MigrateBuiltInLibraries(
            table, LIBRARY_TABLE_TYPE::SYMBOL, stockPath, true );
 
    BOOST_CHECK( modified );
    BOOST_CHECK_EQUAL( countChainedKiCadRows( table ), 1u );
 
    // The user's own row must survive and the direct stock row must be gone.
    BOOST_CHECK( table.Row( wxS( "MyParts" ) ).has_value() );
    BOOST_CHECK( !table.Row( wxS( "Device" ) ).has_value() );
}
 

BOOST_AUTO_TEST_CASE( MigrateBuiltInLibraries_ChainedRowMigratedInPlace )
{
    const wxString stockPath = wxS( "${KICAD10_SYMBOL_DIR}/sym-lib-table" );
 
    LIBRARY_TABLE table( true, wxEmptyString, LIBRARY_TABLE_SCOPE::GLOBAL );
    table.SetType( LIBRARY_TABLE_TYPE::SYMBOL );
 
    LIBRARY_TABLE_ROW& chained = table.InsertRow();
    chained.SetType( LIBRARY_TABLE_ROW::TABLE_TYPE_NAME );
    chained.SetNickname( wxS( "KiCad" ) );
    chained.SetURI( wxS( "${KICAD9_SYMBOL_DIR}/sym-lib-table" ) );
 
    LIBRARY_TABLE_ROW& mine = table.InsertRow();
    mine.SetNickname( wxS( "MyParts" ) );
    mine.SetURI( wxS( "${KIPRJMOD}/../libs/MyParts.kicad_sym" ) );
    mine.SetType( wxS( "KiCad" ) );
 
    bool modified = STARTWIZARD_PROVIDER_LIBRARIES::MigrateBuiltInLibraries(
            table, LIBRARY_TABLE_TYPE::SYMBOL, stockPath, true );
 
    BOOST_CHECK( modified );
    BOOST_CHECK_EQUAL( countChainedKiCadRows( table ), 1u );
 
    auto migrated = table.Row( wxS( "KiCad" ) );
    BOOST_REQUIRE( migrated.has_value() );
    BOOST_CHECK_EQUAL( ( *migrated )->URI(), stockPath );
}
 

BOOST_AUTO_TEST_CASE( QuotedTextRoundTripsEscapes )
{
    LIBRARY_TABLE_PARSER parser;
 
    const std::string escaped =
            "(sym_lib_table (lib (name \"x\") (descr \"say \\\"hi\\\"\\\\line1\\nline2\\r\")))";
 
    tl::expected<LIBRARY_TABLE_IR, LIBRARY_PARSE_ERROR> result = parser.ParseBuffer( escaped );
    BOOST_REQUIRE( result.has_value() );
    BOOST_REQUIRE_EQUAL( result->rows.size(), 1 );
    BOOST_CHECK_EQUAL( result->rows[0].description, "say \"hi\"\\line1\nline2\r" );
 
    // Round-trip: format the parsed value back out and parse it again.
    LIBRARY_TABLE table( true, wxEmptyString, LIBRARY_TABLE_SCOPE::GLOBAL );
    table.SetType( LIBRARY_TABLE_TYPE::SYMBOL );
 
    LIBRARY_TABLE_ROW& row = table.InsertRow();
    row.SetNickname( wxS( "x" ) );
    row.SetType( wxS( "KiCad" ) );
    row.SetURI( wxS( "${KIPRJMOD}/x.kicad_sym" ) );
    row.SetDescription( wxS( "say \"hi\"\\line1\nline2\r" ) );
 
    STRING_FORMATTER formatter;
    table.Format( &formatter );
 
    tl::expected<LIBRARY_TABLE_IR, LIBRARY_PARSE_ERROR> reparsed =
            parser.ParseBuffer( formatter.GetString() );
    BOOST_REQUIRE_MESSAGE( reparsed.has_value(),
                           "a formatted table containing escaped quotes/newlines must re-parse" );
 
    if( reparsed.has_value() && !reparsed->rows.empty() )
        BOOST_CHECK_EQUAL( reparsed->rows[0].description, row.Description() );
}
 
 
BOOST_AUTO_TEST_SUITE_END()