From 509913b1398b36eda23caa1f1f380167465dceee Mon Sep 17 00:00:00 2001 From: Minju Kim Date: Fri, 14 Aug 2026 19:27:20 +0900 Subject: [PATCH] search/test: adapt PathEnd gtests to set_path_margin API Upstream e9c075ab (BIGFEATURE: Add `set_path_margin` command, parallaxsw#469) added a `PathMargin *path_margin` parameter to the PathEndCheck, PathEndOutputDelay and PathEndGatedClock constructors, between `mcp` and the argument that followed it. Our per-module gtests are not in the upstream tree, so the upstream change did not touch them and the merge left them calling the old arity. Pass nullptr for the new parameter; these cases only exercise type()/typeName()/copy(), so no margin is needed. No behavioral change - test-only API adaptation. Signed-off-by: Minju Kim --- search/test/cpp/TestSearchStaInit.cc | 6 +++--- search/test/cpp/TestSearchStaInitB.cc | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/search/test/cpp/TestSearchStaInit.cc b/search/test/cpp/TestSearchStaInit.cc index 83436500..5d853df6 100644 --- a/search/test/cpp/TestSearchStaInit.cc +++ b/search/test/cpp/TestSearchStaInit.cc @@ -3229,7 +3229,7 @@ TEST_F(StaInitTest, PathEndCheckConstruct) { Path *data_path = new Path(); Path *clk_path = new Path(); PathEndCheck *pe = new PathEndCheck(data_path, nullptr, nullptr, - clk_path, nullptr, sta_); + clk_path, nullptr, nullptr, sta_); EXPECT_EQ(pe->type(), PathEnd::Type::check); EXPECT_STREQ(pe->typeName(), "check"); EXPECT_TRUE(pe->isCheck()); @@ -3250,7 +3250,7 @@ TEST_F(StaInitTest, PathEndOutputDelayConstruct) { Path *data_path = new Path(); Path *clk_path = new Path(); PathEndOutputDelay *pe = new PathEndOutputDelay(nullptr, data_path, - clk_path, nullptr, sta_); + clk_path, nullptr, nullptr, sta_); EXPECT_EQ(pe->type(), PathEnd::Type::output_delay); EXPECT_STREQ(pe->typeName(), "output_delay"); EXPECT_TRUE(pe->isOutputDelay()); @@ -3266,7 +3266,7 @@ TEST_F(StaInitTest, PathEndGatedClockConstruct) { Path *clk_path = new Path(); PathEndGatedClock *pe = new PathEndGatedClock(data_path, clk_path, TimingRole::setup(), - nullptr, 0.0f, sta_); + nullptr, nullptr, 0.0f, sta_); EXPECT_EQ(pe->type(), PathEnd::Type::gated_clk); EXPECT_STREQ(pe->typeName(), "gated_clk"); EXPECT_TRUE(pe->isGatedClock()); diff --git a/search/test/cpp/TestSearchStaInitB.cc b/search/test/cpp/TestSearchStaInitB.cc index 2e044a73..63fdfe22 100644 --- a/search/test/cpp/TestSearchStaInitB.cc +++ b/search/test/cpp/TestSearchStaInitB.cc @@ -655,7 +655,7 @@ TEST_F(StaInitTest, PathEndUnconstrainedExceptPathCmp) { TEST_F(StaInitTest, PathEndCheckConstruct2) { Path *p = new Path(); Path *clk = new Path(); - PathEndCheck pe(p, nullptr, nullptr, clk, nullptr, sta_); + PathEndCheck pe(p, nullptr, nullptr, clk, nullptr, nullptr, sta_); EXPECT_EQ(pe.type(), PathEnd::Type::check); EXPECT_TRUE(pe.isCheck()); EXPECT_FALSE(pe.isLatchCheck()); @@ -665,7 +665,7 @@ TEST_F(StaInitTest, PathEndCheckConstruct2) { TEST_F(StaInitTest, PathEndCheckGetters) { Path *p = new Path(); Path *clk = new Path(); - PathEndCheck pe(p, nullptr, nullptr, clk, nullptr, sta_); + PathEndCheck pe(p, nullptr, nullptr, clk, nullptr, nullptr, sta_); EXPECT_EQ(pe.checkArc(), nullptr); EXPECT_EQ(pe.multiCyclePath(), nullptr); } @@ -673,7 +673,7 @@ TEST_F(StaInitTest, PathEndCheckGetters) { TEST_F(StaInitTest, PathEndCheckCopy) { Path *p = new Path(); Path *clk = new Path(); - PathEndCheck pe(p, nullptr, nullptr, clk, nullptr, sta_); + PathEndCheck pe(p, nullptr, nullptr, clk, nullptr, nullptr, sta_); PathEnd *copy = pe.copy(); EXPECT_NE(copy, nullptr); EXPECT_EQ(copy->type(), PathEnd::Type::check);