luke
/
OpenSTA
mirror of https://github.com/The-OpenROAD-Project/OpenSTA.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

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 <mkim@precisioninno.com>
This commit is contained in:
Minju Kim 2026-08-14 19:27:20 +09:00
parent d765822226
commit 509913b139
2 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
search/test/cpp/TestSearchStaInit.cc
View File

@ -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());

6
search/test/cpp/TestSearchStaInitB.cc
View File

@ -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);