test_drc_stub_length.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.
 *
 * 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 2
 * 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 <boost/test/unit_test.hpp>
 
#include <pcbnew_utils/board_file_utils.h>
 
#include <filesystem>
#include <fstream>
 
#include <base_units.h>
#include <board.h>
#include <board_design_settings.h>
#include <drc/drc_engine.h>
#include <drc/drc_item.h>
#include <footprint.h>
#include <netinfo.h>
#include <pad.h>
#include <pcb_marker.h>
#include <pcb_track.h>
#include <pcbnew/pcb_io/kicad_sexpr/pcb_io_kicad_sexpr.h>
#include <drc/drc_rule_parser.h>
#include <reporter.h>
 
 
// Classifier-only fixture: the stub-length check today picks trunk/stub by
// terminal-pad ownership, not by routed topology, so the three nets need only
// be tagged into the same chain and given individually routed traces. Pads
// UA-pad1 and UC-pad1 are assigned as the chain's terminal pads, leaving
// MID_B as the only chain member without an endpoint assignment. Once real
// topological stub detection lands the segments will need to actually meet at
// series components.
static const char* BOARD_FILE = "net_chains/stub_length.kicad_pcb";
 
 
// stub_length 0..5 mm. The 30 mm middle net is well over budget; the 2 mm
// endpoints are within budget *and* are protected by trunk membership anyway.
static const char* DRU_TEXT = R"KICAD((version 1)
 
(rule "StubBudget"
    (condition "A.NetClass == 'Default'")
    (constraint stub_length (min 0mm) (max 5mm))
)
)KICAD";
 
 
BOOST_AUTO_TEST_SUITE( DRCStubLength )
 
 
BOOST_AUTO_TEST_CASE( StubLengthFiresOnIntermediateNetOnly )
{
    namespace fs = std::filesystem;
 
    fs::path tmpDir = fs::temp_directory_path() / "kicad_drc_stub_length";
    fs::create_directories( tmpDir );
 
    fs::path druPath = tmpDir / "stub_length.kicad_dru";
 
    {
        std::ofstream druOut( druPath );
        druOut << DRU_TEXT;
    }
 
    PCB_IO_KICAD_SEXPR     plugin;
    std::unique_ptr<BOARD> board = plugin.LoadBoard( KI_TEST::GetPcbnewTestDataDir() + BOARD_FILE );
    board->BuildConnectivity();
 
    NETINFO_ITEM* netA = board->FindNet( wxS( "/TRUNK_A" ) );
    NETINFO_ITEM* netB = board->FindNet( wxS( "/MID_B" ) );
    NETINFO_ITEM* netC = board->FindNet( wxS( "/TRUNK_C" ) );
 
    BOOST_REQUIRE( netA );
    BOOST_REQUIRE( netB );
    BOOST_REQUIRE( netC );
 
    netA->SetNetChain( wxS( "SIG" ) );
    netB->SetNetChain( wxS( "SIG" ) );
    netC->SetNetChain( wxS( "SIG" ) );
 
    PAD* padA = nullptr;
    PAD* padC = nullptr;
 
    for( FOOTPRINT* fp : board->Footprints() )
    {
        for( PAD* pad : fp->Pads() )
        {
            if( pad->GetNetCode() == netA->GetNetCode() )
                padA = pad;
 
            if( pad->GetNetCode() == netC->GetNetCode() )
                padC = pad;
        }
    }
 
    BOOST_REQUIRE( padA );
    BOOST_REQUIRE( padC );
 
    netA->SetTerminalPad( 0, padA );
    netC->SetTerminalPad( 1, padC );
 
    BOARD_DESIGN_SETTINGS& bds = board->GetDesignSettings();
 
    auto       drcEngine = std::make_shared<DRC_ENGINE>( board.get(), &bds );
    wxFileName ruleFile( druPath.string() );
    drcEngine->InitEngine( ruleFile );
    bds.m_DRCEngine = drcEngine;
 
    bds.m_DRCSeverities[DRCE_INVALID_OUTLINE] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_UNCONNECTED_ITEMS] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_LIB_FOOTPRINT_ISSUES] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_LIB_FOOTPRINT_MISMATCH] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_DANGLING_VIA] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_DRILL_OUT_OF_RANGE] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_NET_CHAIN_STUB_TOO_LONG] = SEVERITY::RPT_SEVERITY_ERROR;
 
    std::vector<wxString> stubMessages;
 
    drcEngine->SetViolationHandler(
            [&]( const std::shared_ptr<DRC_ITEM>& aItem, const VECTOR2I&, int,
                 const std::function<void( PCB_MARKER* )>& )
            {
                if( aItem->GetErrorCode() == DRCE_NET_CHAIN_STUB_TOO_LONG )
                    stubMessages.push_back( aItem->GetErrorMessage( false ) );
            } );
 
    drcEngine->RunTests( EDA_UNITS::MM, true, false );
 
    auto matchesNet = [&]( const wxString& netName )
    {
        for( const wxString& msg : stubMessages )
        {
            if( msg.Contains( wxString::Format( wxS( "'%s'" ), netName ) ) )
                return true;
        }
 
        return false;
    };
 
    bool aFlagged = matchesNet( netA->GetNetname() );
    bool bFlagged = matchesNet( netB->GetNetname() );
    bool cFlagged = matchesNet( netC->GetNetname() );
 
    BOOST_CHECK_MESSAGE( bFlagged,
                         "Intermediate stub net MID_B should fire stub_length violation" );
    BOOST_CHECK_MESSAGE( !aFlagged,
                         "Trunk endpoint net TRUNK_A (owns terminal_pad_0) must not fire" );
    BOOST_CHECK_MESSAGE( !cFlagged,
                         "Trunk endpoint net TRUNK_C (owns terminal_pad_1) must not fire" );
 
    std::error_code ec;
    fs::remove( druPath, ec );
}
 
 
BOOST_AUTO_TEST_CASE( StubLengthQuietOnTwoNetChain )
{
    namespace fs = std::filesystem;
 
    fs::path tmpDir = fs::temp_directory_path() / "kicad_drc_stub_length";
    fs::create_directories( tmpDir );
 
    fs::path druPath = tmpDir / "stub_length_2net.kicad_dru";
 
    // Two-net chain: every member owns a terminal pad assignment, so no member
    // is a stub. With the 30 mm trace on each net, a per-net stub_length would
    // fire if the trunk classifier wrongly excluded one of them.
    static const char* TWO_NET_BOARD_FILE = "net_chains/stub_length_two_net.kicad_pcb";
 
    static const char* TWO_NET_DRU = R"KICAD((version 1)
 
(rule "StubBudget"
    (condition "A.NetClass == 'Default'")
    (constraint stub_length (min 0mm) (max 5mm))
)
)KICAD";
 
    {
        std::ofstream druOut( druPath );
        druOut << TWO_NET_DRU;
    }
 
    PCB_IO_KICAD_SEXPR     plugin;
    std::unique_ptr<BOARD> board = plugin.LoadBoard( KI_TEST::GetPcbnewTestDataDir() + TWO_NET_BOARD_FILE );
    board->BuildConnectivity();
 
    NETINFO_ITEM* netA = board->FindNet( wxS( "/A" ) );
    NETINFO_ITEM* netB = board->FindNet( wxS( "/B" ) );
 
    BOOST_REQUIRE( netA );
    BOOST_REQUIRE( netB );
 
    netA->SetNetChain( wxS( "SIG" ) );
    netB->SetNetChain( wxS( "SIG" ) );
 
    PAD* padA = nullptr;
    PAD* padB = nullptr;
 
    for( FOOTPRINT* fp : board->Footprints() )
    {
        for( PAD* pad : fp->Pads() )
        {
            if( pad->GetNetCode() == netA->GetNetCode() )
                padA = pad;
 
            if( pad->GetNetCode() == netB->GetNetCode() )
                padB = pad;
        }
    }
 
    BOOST_REQUIRE( padA );
    BOOST_REQUIRE( padB );
 
    netA->SetTerminalPad( 0, padA );
    netB->SetTerminalPad( 1, padB );
 
    BOARD_DESIGN_SETTINGS& bds = board->GetDesignSettings();
 
    auto       drcEngine = std::make_shared<DRC_ENGINE>( board.get(), &bds );
    wxFileName ruleFile( druPath.string() );
    drcEngine->InitEngine( ruleFile );
    bds.m_DRCEngine = drcEngine;
 
    bds.m_DRCSeverities[DRCE_INVALID_OUTLINE] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_UNCONNECTED_ITEMS] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_LIB_FOOTPRINT_ISSUES] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_LIB_FOOTPRINT_MISMATCH] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_DANGLING_VIA] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_DRILL_OUT_OF_RANGE] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_NET_CHAIN_STUB_TOO_LONG] = SEVERITY::RPT_SEVERITY_ERROR;
 
    int stubCount = 0;
 
    drcEngine->SetViolationHandler(
            [&]( const std::shared_ptr<DRC_ITEM>& aItem, const VECTOR2I&, int,
                 const std::function<void( PCB_MARKER* )>& )
            {
                if( aItem->GetErrorCode() == DRCE_NET_CHAIN_STUB_TOO_LONG )
                    ++stubCount;
            } );
 
    drcEngine->RunTests( EDA_UNITS::MM, true, false );
 
    BOOST_CHECK_MESSAGE( stubCount == 0,
                         "Two-net chain with both ends owning terminal pads must not "
                         "fire any stub_length violation, got "
                                 << stubCount );
 
    std::error_code ec;
    fs::remove( druPath, ec );
}
 
 
BOOST_AUTO_TEST_CASE( StubLengthIncludesPadToDie )
{
    namespace fs = std::filesystem;
 
    fs::path tmpDir = fs::temp_directory_path() / "kicad_drc_stub_length";
    fs::create_directories( tmpDir );
 
    fs::path druPath = tmpDir / "stub_length_pad_to_die.kicad_dru";
 
    // Three-net chain whose middle net (MID_B) is a stub. The stub's routed
    // copper is a short 2 mm trace, well under the 5 mm budget. A 10 mm
    // pad-to-die length is attached to the MID_B pad, so the *total* stub
    // length (route + pad-to-die) is 12 mm and must violate the constraint.
    static const char* PAD_TO_DIE_BOARD_FILE = "net_chains/stub_length_pad_to_die.kicad_pcb";
 
    static const char* PAD_TO_DIE_DRU = R"KICAD((version 1)
 
(rule "StubBudget"
    (condition "A.NetClass == 'Default'")
    (constraint stub_length (min 0mm) (max 5mm))
)
)KICAD";
 
    {
        std::ofstream druOut( druPath );
        druOut << PAD_TO_DIE_DRU;
    }
 
    PCB_IO_KICAD_SEXPR     plugin;
    std::unique_ptr<BOARD> board = plugin.LoadBoard( KI_TEST::GetPcbnewTestDataDir() + PAD_TO_DIE_BOARD_FILE );
 
    NETINFO_ITEM* netA = board->FindNet( wxS( "/TRUNK_A" ) );
    NETINFO_ITEM* netB = board->FindNet( wxS( "/MID_B" ) );
    NETINFO_ITEM* netC = board->FindNet( wxS( "/TRUNK_C" ) );
 
    BOOST_REQUIRE( netA );
    BOOST_REQUIRE( netB );
    BOOST_REQUIRE( netC );
 
    netA->SetNetChain( wxS( "SIG" ) );
    netB->SetNetChain( wxS( "SIG" ) );
    netC->SetNetChain( wxS( "SIG" ) );
 
    PAD* padA = nullptr;
    PAD* padB = nullptr;
    PAD* padC = nullptr;
 
    for( FOOTPRINT* fp : board->Footprints() )
    {
        for( PAD* pad : fp->Pads() )
        {
            if( pad->GetNetCode() == netA->GetNetCode() )
                padA = pad;
 
            if( pad->GetNetCode() == netB->GetNetCode() )
                padB = pad;
 
            if( pad->GetNetCode() == netC->GetNetCode() )
                padC = pad;
        }
    }
 
    BOOST_REQUIRE( padA );
    BOOST_REQUIRE( padB );
    BOOST_REQUIRE( padC );
 
    // 10 mm pad-to-die on the stub net pushes the total stub length above the
    // 5 mm budget even though the routed copper is only 2 mm.
    padB->SetPadToDieLength( pcbIUScale.mmToIU( 10 ) );
 
    netA->SetTerminalPad( 0, padA );
    netC->SetTerminalPad( 1, padC );
 
    board->BuildConnectivity();
 
    BOARD_DESIGN_SETTINGS& bds = board->GetDesignSettings();
 
    auto       drcEngine = std::make_shared<DRC_ENGINE>( board.get(), &bds );
    wxFileName ruleFile( druPath.string() );
    drcEngine->InitEngine( ruleFile );
    bds.m_DRCEngine = drcEngine;
 
    bds.m_DRCSeverities[DRCE_INVALID_OUTLINE] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_UNCONNECTED_ITEMS] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_LIB_FOOTPRINT_ISSUES] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_LIB_FOOTPRINT_MISMATCH] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_DANGLING_VIA] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_DRILL_OUT_OF_RANGE] = SEVERITY::RPT_SEVERITY_IGNORE;
    bds.m_DRCSeverities[DRCE_NET_CHAIN_STUB_TOO_LONG] = SEVERITY::RPT_SEVERITY_ERROR;
 
    std::vector<wxString> stubMessages;
 
    drcEngine->SetViolationHandler(
            [&]( const std::shared_ptr<DRC_ITEM>& aItem, const VECTOR2I&, int,
                 const std::function<void( PCB_MARKER* )>& )
            {
                if( aItem->GetErrorCode() == DRCE_NET_CHAIN_STUB_TOO_LONG )
                    stubMessages.push_back( aItem->GetErrorMessage( false ) );
            } );
 
    drcEngine->RunTests( EDA_UNITS::MM, true, false );
 
    auto matchesNet = [&]( const wxString& netName )
    {
        for( const wxString& msg : stubMessages )
        {
            if( msg.Contains( wxString::Format( wxS( "'%s'" ), netName ) ) )
                return true;
        }
 
        return false;
    };
 
    BOOST_CHECK_MESSAGE( matchesNet( netB->GetNetname() ),
                         "Stub net MID_B with 2 mm route + 10 mm pad-to-die must violate "
                         "the 5 mm stub_length budget" );
 
    std::error_code ec;
    fs::remove( druPath, ec );
}
 
 
BOOST_AUTO_TEST_CASE( StubLengthAcceptsTimeDomainUnits )
{
    // Regression for H-5: stub_length used to be missing from the
    // allowsTimeDomain allow-list in DRC_RULES_PARSER, so a (max 100ps)
    // value would be rejected with "Time based units not allowed for
    // constraint type." The rule must parse cleanly and the resulting
    // constraint must be flagged as TIME_DOMAIN.
    const wxString DRU_TIME = wxS(
            "(version 1)\n"
            "(rule \"StubBudgetTime\"\n"
            "    (condition \"A.NetClass == 'Default'\")\n"
            "    (constraint stub_length (max 100ps))\n"
            ")\n" );
 
    DRC_RULES_PARSER parser( DRU_TIME, wxS( "stub_length_time_test" ) );
 
    WX_STRING_REPORTER reporter;
    std::vector<std::shared_ptr<DRC_RULE>> rules;
 
    parser.Parse( rules, &reporter );
 
    BOOST_CHECK_MESSAGE( !reporter.HasMessageOfSeverity( RPT_SEVERITY_ERROR ),
                         "stub_length with ps units must parse without error, got: "
                                 << reporter.GetMessages().ToStdString() );
 
    BOOST_REQUIRE_EQUAL( rules.size(), 1u );
 
    std::optional<DRC_CONSTRAINT> stubConstraint =
            rules.front()->FindConstraint( NET_CHAIN_STUB_LENGTH_CONSTRAINT );
 
    BOOST_REQUIRE( stubConstraint.has_value() );
    BOOST_CHECK( stubConstraint->GetOption( DRC_CONSTRAINT::OPTIONS::TIME_DOMAIN ) );
    BOOST_CHECK( !stubConstraint->GetOption( DRC_CONSTRAINT::OPTIONS::SPACE_DOMAIN ) );
    BOOST_REQUIRE( stubConstraint->GetValue().HasMax() );
    // 1 ps == 1e6 IU (internal time unit is the attosecond).
    BOOST_CHECK_EQUAL( stubConstraint->GetValue().Max(), 100 * 1000 * 1000 );
}
 
 
BOOST_AUTO_TEST_CASE( ReturnPathDoesNotCarryTimeDomainOption )
{
    // return_path takes a layer name, not a delay, so the TIME_DOMAIN option
    // must never be set on a return_path constraint regardless of stray
    // (max ...) subtrees. The return_path branch in the parser ignores any
    // unknown sub-options (it only consumes (layer ...)), so we verify the
    // resulting constraint stays in space-domain configuration.
    const wxString DRU_LAYER = wxS(
            "(version 1)\n"
            "(rule \"GoodReturnPath\"\n"
            "    (condition \"A.NetClass == 'Default'\")\n"
            "    (constraint return_path (layer \"B.Cu\"))\n"
            ")\n" );
 
    DRC_RULES_PARSER parser( DRU_LAYER, wxS( "return_path_layer_test" ) );
 
    WX_STRING_REPORTER reporter;
    std::vector<std::shared_ptr<DRC_RULE>> rules;
 
    parser.Parse( rules, &reporter );
 
    BOOST_CHECK_MESSAGE( !reporter.HasMessageOfSeverity( RPT_SEVERITY_ERROR ),
                         "return_path with a layer must parse without error, got: "
                                 << reporter.GetMessages().ToStdString() );
 
    BOOST_REQUIRE_EQUAL( rules.size(), 1u );
 
    std::optional<DRC_CONSTRAINT> rpConstraint =
            rules.front()->FindConstraint( NET_CHAIN_RETURN_PATH_CONSTRAINT );
 
    BOOST_REQUIRE( rpConstraint.has_value() );
    BOOST_CHECK( !rpConstraint->GetOption( DRC_CONSTRAINT::OPTIONS::TIME_DOMAIN ) );
}
 
 
BOOST_AUTO_TEST_SUITE_END()