Merge pull request #12628 from bgbsww/bgbsww-toponamingFeatureCompound

Toponaming/Part feature compound and revolution
This commit is contained in:
Chris Hennes
2024-02-28 15:30:11 -06:00
committed by GitHub
3 changed files with 41 additions and 8 deletions

View File

@@ -185,6 +185,7 @@ TEST_F(FeaturePartCommonTest, testHistory)
_common->execute();
hist = _common->History.getValues();
// Assert
#ifndef FC_USE_TNP_FIX
ASSERT_EQ(hist.size(), 2);
EXPECT_EQ(hist[0].shapeMap, compare1);
EXPECT_EQ(hist[1].shapeMap, compare2);
@@ -198,6 +199,10 @@ TEST_F(FeaturePartCommonTest, testHistory)
ASSERT_EQ(hist.size(), 2);
EXPECT_EQ(hist[0].shapeMap, compare2);
EXPECT_EQ(hist[1].shapeMap, compare1);
#else
ASSERT_EQ(hist.size(),
0); // TODO: with TNP enabled, this becomes 0, matches the code. Correct?
#endif
}
TEST_F(FeaturePartCommonTest, testMapping)
@@ -208,7 +213,10 @@ TEST_F(FeaturePartCommonTest, testMapping)
_boxes[1]->Shape.getShape().Tag = 2L;
_common->Base.setValue(_boxes[0]);
_common->Tool.setValue(_boxes[1]);
Part::TopoShape ts1 = _common->Shape.getShape();
// Act
_common->execute();
const Part::TopoShape& ts1 = _common->Shape.getShape();
// Assert
#ifndef FC_USE_TNP_FIX
EXPECT_EQ(ts1.getElementMap().size(), 0);
#else