kindred/create
1
1
Fork 0
Code Issues 53 Pull Requests Actions Packages Projects 9 Releases 2 Wiki Activity

Part/Toponaming: Transfer WireJoiner

Browse Source 
* Added test for WireJoiner::Modified()
 * Fixed typo in test for WireJoiner::getResultWires()

Signed-off-by: CalligaroV <vincenzo.calligaro@gmail.com>
This commit is contained in:
CalligaroV
2024-03-14 11:27:41 +01:00
parent 66c08664a5
commit c70f34b1f4
1 changed files with 45 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

46
tests/src/Mod/Part/App/WireJoiner.cpp
View File

@@ -739,7 +739,7 @@ TEST_F(WireJoinerTest, getResultWires)
// Act
wjNoResultWires.addShape(edgesNoResultWires);
// wjNoOpenWires.Build() is called by wjNoResultWires.getResultWires()
// wjNoResultWires.Build() is called by wjNoResultWires.getResultWires()
wjNoResultWires.getResultWires(wireNoResultWires);
wjNoOp.addShape(edgesNoOp);
@@ -809,4 +809,48 @@ TEST_F(WireJoinerTest, Build)
}
#endif
TEST_F(WireJoinerTest, Modified)
{
// Arrange
// Create various edges that will be used for the WireJoiner objects tests
auto edge1 {BRepBuilderAPI_MakeEdge(gp_Pnt(-0.1, 0.0, 0.0), gp_Pnt(1.1, 0.0, 0.0)).Edge()};
auto edge2 {BRepBuilderAPI_MakeEdge(gp_Pnt(1.0, -0.1, 0.0), gp_Pnt(1.0, 1.1, 0.0)).Edge()};
auto edge3 {BRepBuilderAPI_MakeEdge(gp_Pnt(1.1, 1.1, 0.0), gp_Pnt(-0.1, -0.1, 0.0)).Edge()};
// A vector of edges used as argument for wjModified.addShape()
std::vector<TopoDS_Shape> edges {edge1, edge2, edge3};
// A WireJoiner object that will create no closed wires
auto wjModified {WireJoiner()};
// Act
wjModified.addShape(edges);
wjModified.Build();
// Assert
// In this case, every edge added with wjModified.addShape() modifies other shapes 3 times
// edge1 modifies the edges with vertexes:
// (0.0, 0.0, 0.0) - (1.0, 1.0, 0.0)
// (-0.1, -0.1, 0.0) - (0.0, 0.0, 0.0)
// (1.0, -0.1, 0.0) - (1.0, 0.0, 0.0)
EXPECT_EQ(wjModified.Modified(edge1).Size(), 3);
// edge2 modifies the edges with vertexes:
// (1.0, 1.0, 0.0) - (1.1, 1.1, 0.0)
// (0.0, 0.0, 0.0) - (1.0, 0.0, 0.0)
// (1.0, 0.0, 0.0) - (1.1, 0.0, 0.0)
EXPECT_EQ(wjModified.Modified(edge2).Size(), 3);
// edge3 modifies the edges with vertexes:
// (1.0, 0.0, 0.0) - (1.0, 1.0, 0.0)
// (1.0, 1.0, 0.0) - (1.0, 1.1, 0.0)
// (-0.1, 0.0, 0.0) - (0.0, 0.0, 0.0)
EXPECT_EQ(wjModified.Modified(edge3).Size(), 3);
}
// NOLINTEND(readability-magic-numbers,cppcoreguidelines-avoid-magic-numbers)