test_altium_parser_utils.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 <qa_utils/wx_utils/unit_test_utils.h>
#include <boost/test/data/test_case.hpp>
 
#include <common/io/altium/altium_parser_utils.h>
#include <common/io/altium/altium_props_utils.h>
 
struct ALTIUM_PARSER_UTILS_FIXTURE
{
    ALTIUM_PARSER_UTILS_FIXTURE() {}
};
 

BOOST_FIXTURE_TEST_SUITE( AltiumParserUtils, ALTIUM_PARSER_UTILS_FIXTURE )
 
 
struct SPECIAL_STRINGS_TO_KICAD
{
    wxString                     input;
    wxString                     exp_result;
    std::map<wxString, wxString> override;
};
 
std::ostream & operator<<(std::ostream & strm, SPECIAL_STRINGS_TO_KICAD const & data) {
    return strm << "[" << data.input << " -> " << data.exp_result << "]";
}

static const std::vector<SPECIAL_STRINGS_TO_KICAD> sch_special_string_to_kicad_property = {
    // Empty
    { "", "", {} },
    // No Special Strings
    { "A", "A", {} },
    { "  A", "  A", {} },
    { "A  ", "A  ", {} },
    { "A=B", "A=B", {} },
    { "A=B+C", "A=B+C", {} },
    { "A\nB", "A\nB", {} },
    { "A\tB", "A\tB", {} },
    { "This is a long text with spaces", "This is a long text with spaces", {} },
    // Text format (underscore,...), TODO: add
    // Escaping, TODO: add
    { "+", "+", {} },
    { "'", "'", {} },
    { "'A'", "'A'", {} },
    { "A+B", "A+B", {} },
    { "A=B", "A=B", {} },
    { "$", "$", {} },
    { "{", "{", {} },
    { "}", "}", {} },
    { "${A}", "${A}", {} }, // TODO: correct substitution
    // Simple special strings
    { "=A", "${A}", {} },
    { "=A", "${C}", { { "A", "C" } } },
    { "=A_B", "${A_B}", {} },
    // Combined special strings
    { "=A+B", "${A}${B}", {} },
    { "=A+B", "${A}${B}", {} },
    { "=A+B", "${C}${B}", { { "A", "C" } } },
    { "=A+B", "${C}${D}", { { "A", "C" }, { "B", "D" } } },
    // Case-insensitive special strings
    { "=A", "${C}", { { "A", "C" } } },
    { "=a", "${C}", { { "A", "C" } } },
    { "=AB", "${C}", { { "AB", "C" } } },
    { "=aB", "${C}", { { "AB", "C" } } },
    { "=Ab", "${C}", { { "AB", "C" } } },
    { "=ab", "${C}", { { "AB", "C" } } },
    // Special strings with text
    { "='A'", "A", {} },
    { "='This is a long text with spaces'", "This is a long text with spaces", {} },
    { "='='", "=", {} },
    { "='+'", "+", {} },
    { "='$'", "$", {} },
    { "='{'", "{", {} },
    { "='}'", "}", {} },
    { "='${A}'", "${A}", {} }, // TODO: correct substitution
    { "='A'+'B'", "AB", {} },
    { "='A'+' '", "A ", {} },
    { "=' '+'B'", " B", {} },
    { "='A'+B", "A${B}", {} },
    { "='A'+\"B\"", "A${B}", {} },
    { "='A' + \"B\"", "A${B}", {} },
    { "=\"A\"+'B'", "${A}B", {} },
    { "=\"A\" + 'B'", "${A}B", {} },
    { "=A+'B'", "${A}B", {} },
    { "=A+' '+B", "${A} ${B}", {} },
    { "='A'+B+'C'+D", "A${B}C${D}", {} },
    // Some special cases we do not know yet how to handle correctly. But we should not crash ;)
    { "=+", "", {} },
    { "=++", "", {} },
    { "=+++", "", {} },
    { "=B+", "${B}", {} },
    { "=+B", "${B}", {} },
    { "=B++", "${B}", {} },
    { "=+B+", "${B}", {} },
    { "=++B", "${B}", {} },
    { " =", " =", {} },
    { "= ", "", {} },
    { "= A", "${A}", {} },
    { "=A ", "${A}", {} },
    { "='A'B", "A", {} },
    { "=A'B'", "B", {} },
    { "=A'B", "B", {} },
    { "=A+ 'B'", "${A}B", {} },
};
 

BOOST_DATA_TEST_CASE( AltiumSchSpecialStringsToKiCadVariablesProperties,
                      boost::unit_test::data::make(sch_special_string_to_kicad_property),
                      data )
{
    wxString result = AltiumSchSpecialStringsToKiCadVariables( data.input, data.override );
 
    // These are all valid
    BOOST_CHECK_EQUAL( result, data.exp_result );
}
 

static const std::vector<SPECIAL_STRINGS_TO_KICAD> pcb_special_string_to_kicad_property = {
    // Empty
    { "", "", {} },
    // No Special Strings
    { "A", "A", {} },
    { "  A", "  A", {} },
    { "A  ", "A  ", {} },
    { "A=B", "A=B", {} },
    { "A=B+C", "A=B+C", {} },
    { "A\nB", "A\nB", {} },
    { "A\tB", "A\tB", {} },
    { "This is a long text with spaces", "This is a long text with spaces", {} },
    // Text format (underscore,...), TODO: add
    // Escaping, TODO: add
    { "'", "'", {} },
    { "'A'", "'A'", {} },
    { "$", "$", {} },
    { "{", "{", {} },
    { "}", "}", {} },
    { "${A}", "${A}", {} }, // TODO: correct substitution
    // Simple special strings starting with dot
    { ".A", "${A}", {} },
    { ".A", "${C}", { { "A", "C" } } },
    { ".A_B", "${A_B}", {} },
    // Concatenated special strings
    { "'.A'", "${A}", {} },
    { "'.A''.B'", "${A}${B}", {} },
    { "'.A''.B'", "${C}${B}", { { "A", "C" } } },
    { "'.A''.B'", "${CC}${D}", { { "A", "CC" }, { "B", "D" } } },
    { "A='.A', B='.B'", "A=${A}, B=${B}", {} },
    // Case-insensitive special strings
    { ".A", "${C}", { { "A", "C" } } },
    { ".a", "${C}", { { "A", "C" } } },
    { ".AB", "${C}", { { "AB", "C" } } },
    { ".aB", "${C}", { { "AB", "C" } } },
    { ".Ab", "${C}", { { "AB", "C" } } },
    { ".ab", "${C}", { { "AB", "C" } } },
    // Some special cases we do not know yet how to handle correctly. But we should not crash ;)
    { "''", "''", {} },
    { " .", " .", {} },
    { ". ", "${ }", {} },
    { ". A", "${ A}", {} },
    { ".A ", "${A }", {} },
    { " .A", " .A", {} },
    { "...", "${..}", {} },
};
 

BOOST_DATA_TEST_CASE( AltiumPcbSpecialStringsToKiCadStringsProperties,
                      boost::unit_test::data::make(pcb_special_string_to_kicad_property),
                      data )
{
    wxString result = AltiumPcbSpecialStringsToKiCadStrings( data.input, data.override );
 
    // These are all valid
    BOOST_CHECK_EQUAL( result, data.exp_result );
}
 
struct PIN_DESIGNATOR_CASE
{
    wxString input;
    wxString expected;
};
 
std::ostream& operator<<( std::ostream& strm, PIN_DESIGNATOR_CASE const& data )
{
    return strm << "[" << data.input << " -> " << data.expected << "]";
}
 
// Verifies GitLab issue 23709 handling: Altium encodes pins tied to multiple physical
// pads as a comma-separated designator string; KiCad expects stacked-pin bracket notation.
static const std::vector<PIN_DESIGNATOR_CASE> pin_designator_cases = {
    // Single-designator inputs return after whitespace trim only
    { wxT( "" ), wxT( "" ) },
    { wxT( "1" ), wxT( "1" ) },
    { wxT( "A1" ), wxT( "A1" ) },
    { wxT( "GND" ), wxT( "GND" ) },
    { wxT( " 1 " ), wxT( "1" ) },
    { wxT( "  A1 " ), wxT( "A1" ) },
 
    // Basic multi-designator conversion
    { wxT( "1,2" ), wxT( "[1,2]" ) },
    { wxT( "1,2,3" ), wxT( "[1,2,3]" ) },
    { wxT( "A1,A2,A3" ), wxT( "[A1,A2,A3]" ) },
 
    // Whitespace around tokens is trimmed
    { wxT( "1, 2, 3" ), wxT( "[1,2,3]" ) },
    { wxT( " 1 , 2 , 3 " ), wxT( "[1,2,3]" ) },
 
    // Stray empty tokens (trailing comma, duplicate comma) are dropped
    { wxT( "1,2," ), wxT( "[1,2]" ) },
    { wxT( "1,,2" ), wxT( "[1,2]" ) },
 
    // Degenerate inputs collapse back to a single bare token or to the original
    { wxT( "1," ), wxT( "1" ) },
    { wxT( "," ), wxT( "," ) },
};
 
 
BOOST_DATA_TEST_CASE( AltiumPinDesignatorToKiCadProperties,
                      boost::unit_test::data::make( pin_designator_cases ),
                      data )
{
    BOOST_CHECK_EQUAL( AltiumPinDesignatorToKiCad( data.input ), data.expected );
}
 
 
// A value too large for an int is clamped, which turns a bogus dimension into a coordinate at
// the edge of the board.  Callers that place geometry need to know the difference
BOOST_AUTO_TEST_CASE( ConvertToKicadUnitFlagsClampedValues )
{
    bool outOfRange = true;
 
    BOOST_CHECK_EQUAL( ALTIUM_PROPS_UTILS::ConvertToKicadUnit( 10000., &outOfRange ), 25400 );
    BOOST_CHECK( !outOfRange );
 
    BOOST_CHECK_EQUAL( ALTIUM_PROPS_UTILS::ConvertToKicadUnit( -10000., &outOfRange ), -25400 );
    BOOST_CHECK( !outOfRange );
 
    // -111339.9634mil, the vertex TIDA-00204_revE3_PCB.PcbDoc carries in a copper polygon
    const int32_t clamped = ALTIUM_PROPS_UTILS::ConvertToKicadUnit( -1113399634., &outOfRange );
 
    BOOST_CHECK( outOfRange );
    BOOST_CHECK_LT( clamped, -2000000000 );
 
    BOOST_CHECK_EQUAL( ALTIUM_PROPS_UTILS::ConvertToKicadUnit( 1113399634., &outOfRange ),
                       -clamped );
    BOOST_CHECK( outOfRange );
 
    // The flag is optional and the clamp still happens without it
    BOOST_CHECK_EQUAL( ALTIUM_PROPS_UTILS::ConvertToKicadUnit( -1113399634. ), clamped );
}
 
 
BOOST_AUTO_TEST_CASE( ReadKicadUnitFlagsClampedValues )
{
    const std::map<wxString, wxString> props = {
        { wxT( "SANE" ), wxT( "1000mil" ) },
        { wxT( "HUGE" ), wxT( "-111339.9634mil" ) },
        { wxT( "JUNK" ), wxT( "not a number" ) },
    };
 
    bool outOfRange = true;
 
    BOOST_CHECK_EQUAL( ALTIUM_PROPS_UTILS::ReadKicadUnit( props, wxT( "SANE" ), wxT( "0mil" ),
                                                          &outOfRange ),
                       25400000 );
    BOOST_CHECK( !outOfRange );
 
    ALTIUM_PROPS_UTILS::ReadKicadUnit( props, wxT( "HUGE" ), wxT( "0mil" ), &outOfRange );
    BOOST_CHECK( outOfRange );
 
    // An unparseable or absent property yields zero, which is in range rather than clamped
    BOOST_CHECK_EQUAL( ALTIUM_PROPS_UTILS::ReadKicadUnit( props, wxT( "JUNK" ), wxT( "0mil" ),
                                                          &outOfRange ),
                       0 );
    BOOST_CHECK( !outOfRange );
 
    BOOST_CHECK_EQUAL( ALTIUM_PROPS_UTILS::ReadKicadUnit( props, wxT( "ABSENT" ), wxT( "0mil" ),
                                                          &outOfRange ),
                       0 );
    BOOST_CHECK( !outOfRange );
}
 
BOOST_AUTO_TEST_SUITE_END()