3DPrinting/PrusaSlicer-NonPlainar
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Reenabled some of the previously disabled tests after the computation of bridging perimeters was added. One test was disabled for Arachne because Arachne is causing issues with seam alignment.

Browse Source
This commit is contained in:
Lukáš Hejl 2022-06-09 10:53:43 +02:00
parent 3d9b13a5e3
commit ce038fd7f0
2 changed files with 1 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
t/perimeters.t
View File

@ -257,8 +257,6 @@ use Slic3r::Test;
$config->set('bridge_fan_speed', [ 100 ]); $config->set('bridge_fan_speed', [ 100 ]);
$config->set('bridge_flow_ratio', 33); # arbitrary value $config->set('bridge_flow_ratio', 33); # arbitrary value
$config->set('overhangs', 1); $config->set('overhangs', 1);
# FIXME Lukas H.: For now, this unit test is disabled for Arachne because of an issue with detecting overhang when Arachne is enabled.
$config->set('perimeter_generator', 'classic');
my $print = Slic3r::Test::init_print('overhang', config => $config); my $print = Slic3r::Test::init_print('overhang', config => $config);
my %layer_speeds = (); # print Z => [ speeds ] my %layer_speeds = (); # print Z => [ speeds ]
my $fan_speed = 0; my $fan_speed = 0;
@ -397,8 +395,6 @@ use Slic3r::Test;
$config->set('overhangs', 1); $config->set('overhangs', 1);
$config->set('cooling', [ 0 ]); # to prevent speeds from being altered $config->set('cooling', [ 0 ]); # to prevent speeds from being altered
$config->set('first_layer_speed', '100%'); # to prevent speeds from being altered $config->set('first_layer_speed', '100%'); # to prevent speeds from being altered
# FIXME Lukas H.: For now, this unit test is disabled for Arachne because of an issue with detecting overhang when Arachne is enabled.
$config->set('perimeter_generator', 'classic');
my $test = sub { my $test = sub {
my ($print) = @_; my ($print) = @_;

5
tests/fff_print/test_print.cpp
View File

@ -20,10 +20,7 @@ SCENARIO("PrintObject: Perimeter generation", "[PrintObject]") {
} }
THEN("Every layer in region 0 has 1 island of perimeters") { THEN("Every layer in region 0 has 1 island of perimeters") {
for (const Layer *layer : object.layers()) for (const Layer *layer : object.layers())
if (object.config().perimeter_generator == PerimeterGeneratorType::Arachne) REQUIRE(layer->regions().front()->perimeters.entities.size() == 1);
REQUIRE(layer->regions().front()->perimeters.entities.size() == 3);
else
REQUIRE(layer->regions().front()->perimeters.entities.size() == 1);
} }
THEN("Every layer in region 0 has 3 paths in its perimeters list.") { THEN("Every layer in region 0 has 3 paths in its perimeters list.") {
for (const Layer *layer : object.layers()) for (const Layer *layer : object.layers())