Fixing unit tests for supports. After "dont_support_bridges" was fixed

for perimeters, no support was rightfully generated for a preset
using a default "dont_support_bridges" = off.
This commit is contained in:
Vojtech Bubnik 2020-12-11 17:57:29 +01:00
parent 8bfb2a4eef
commit e78ad6ffa9

View File

@ -23,6 +23,7 @@ SCENARIO("SupportMaterial: support_layers_z and contact_distance", "[SupportMate
// Box h = 20mm, hole bottom at 5mm, hole height 10mm (top edge at 15mm). // Box h = 20mm, hole bottom at 5mm, hole height 10mm (top edge at 15mm).
TriangleMesh mesh = Slic3r::Test::mesh(Slic3r::Test::TestMesh::cube_with_hole); TriangleMesh mesh = Slic3r::Test::mesh(Slic3r::Test::TestMesh::cube_with_hole);
mesh.rotate_x(float(M_PI / 2)); mesh.rotate_x(float(M_PI / 2));
// mesh.write_binary("d:\\temp\\cube_with_hole.stl");
auto check = [](Slic3r::Print &print, bool &first_support_layer_height_ok, bool &layer_height_minimum_ok, bool &layer_height_maximum_ok, bool &top_spacing_ok) auto check = [](Slic3r::Print &print, bool &first_support_layer_height_ok, bool &layer_height_minimum_ok, bool &layer_height_maximum_ok, bool &top_spacing_ok)
{ {
@ -76,7 +77,8 @@ SCENARIO("SupportMaterial: support_layers_z and contact_distance", "[SupportMate
{ "support_material", 1 }, { "support_material", 1 },
{ "layer_height", 0.2 }, { "layer_height", 0.2 },
{ "first_layer_height", 0.4 }, { "first_layer_height", 0.4 },
}); { "dont_support_bridges", false },
});
bool a, b, c, d; bool a, b, c, d;
check(print, a, b, c, d); check(print, a, b, c, d);
THEN("First layer height is honored") { REQUIRE(a == true); } THEN("First layer height is honored") { REQUIRE(a == true); }
@ -90,7 +92,8 @@ SCENARIO("SupportMaterial: support_layers_z and contact_distance", "[SupportMate
{ "support_material", 1 }, { "support_material", 1 },
{ "layer_height", 0.2 }, { "layer_height", 0.2 },
{ "first_layer_height", 0.3 }, { "first_layer_height", 0.3 },
}); { "dont_support_bridges", false },
});
bool a, b, c, d; bool a, b, c, d;
check(print, a, b, c, d); check(print, a, b, c, d);
THEN("First layer height is honored") { REQUIRE(a == true); } THEN("First layer height is honored") { REQUIRE(a == true); }
@ -104,7 +107,8 @@ SCENARIO("SupportMaterial: support_layers_z and contact_distance", "[SupportMate
{ "support_material", 1 }, { "support_material", 1 },
{ "layer_height", 0.2 }, { "layer_height", 0.2 },
{ "first_layer_height", 0.3 }, { "first_layer_height", 0.3 },
}); { "dont_support_bridges", false },
});
bool a, b, c, d; bool a, b, c, d;
check(print, a, b, c, d); check(print, a, b, c, d);
THEN("First layer height is honored") { REQUIRE(a == true); } THEN("First layer height is honored") { REQUIRE(a == true); }