haskal
/
kicad
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove some video nets from order testing. 

Conflicting nets are disambiguated with "_n" suffixes, but the order
of these is not deterministic.
This commit is contained in:
Jeff Young 2022-01-02 14:56:15 +00:00
parent 0365c6bf4b
commit 88ee2e269f
1 changed files with 7 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
qa/eeschema/test_netlists.cpp
View File

@ -197,6 +197,13 @@ void TEST_NETLISTS_FIXTURE::compareNetlists()
const COMPONENT_NET& goldenNet = goldenComp->GetNet( net ); const COMPONENT_NET& goldenNet = goldenComp->GetNet( net );
const COMPONENT_NET& testNet = refComp->GetNet( net ); const COMPONENT_NET& testNet = refComp->GetNet( net );
// The video test has a bunch of unconnected RESERVED pins which cause duplicate
// auto-generated netnames. The connectivity algo disambiguates these with "_n"
// suffixes, but since the algorithm is multi-threaded which ones get which suffix
// is not deterministic. So skip these.
if( testNet.GetPinName().Contains( "RESERVED" ) )
continue;
// The two nets at the same index should be identical // The two nets at the same index should be identical
BOOST_REQUIRE_EQUAL( goldenNet.GetNetName(), testNet.GetNetName() ); BOOST_REQUIRE_EQUAL( goldenNet.GetNetName(), testNet.GetNetName() );
BOOST_REQUIRE_EQUAL( goldenNet.GetPinName(), testNet.GetPinName() ); BOOST_REQUIRE_EQUAL( goldenNet.GetPinName(), testNet.GetPinName() );