From bad34c4d8fd13b644df7b86a3d939059ed6b986f Mon Sep 17 00:00:00 2001 From: Mike Welsh Date: Sat, 25 Apr 2020 02:04:37 -0700 Subject: [PATCH] tests: Fix tests for regressions caused by #498 Changes to the action queue caused actions queued during other actions to run too late. These regressions weren't caught by the tests because many of the goto tests ran for more frames than was necessary, allowing the late actions to still run. --- core/tests/regression_tests.rs | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/core/tests/regression_tests.rs b/core/tests/regression_tests.rs index 189e4ac0e..ffc542de9 100644 --- a/core/tests/regression_tests.rs +++ b/core/tests/regression_tests.rs @@ -71,19 +71,19 @@ swf_tests! { (execution_order3, "avm1/execution_order3", 5), (single_frame, "avm1/single_frame", 2), (looping, "avm1/looping", 6), - (goto_advance1, "avm1/goto_advance1", 10), - (goto_advance2, "avm1/goto_advance2", 10), - (goto_both_ways1, "avm1/goto_both_ways1", 10), - (goto_both_ways2, "avm1/goto_both_ways2", 10), - (goto_frame, "avm1/goto_frame", 8), - (goto_frame2, "avm1/goto_frame2", 20), - (goto_label, "avm1/goto_label", 8), + (goto_advance1, "avm1/goto_advance1", 2), + (goto_advance2, "avm1/goto_advance2", 2), + (goto_both_ways1, "avm1/goto_both_ways1", 2), + (goto_both_ways2, "avm1/goto_both_ways2", 3), + (goto_frame, "avm1/goto_frame", 3), + (goto_frame2, "avm1/goto_frame2", 5), + (goto_label, "avm1/goto_label", 4), (goto_methods, "avm1/goto_methods", 1), - (goto_rewind1, "avm1/goto_rewind1", 10), - (goto_rewind2, "avm1/goto_rewind2", 10), - (goto_rewind3, "avm1/goto_rewind3", 10), + (goto_rewind1, "avm1/goto_rewind1", 4), + (goto_rewind2, "avm1/goto_rewind2", 5), + (goto_rewind3, "avm1/goto_rewind3", 2), (goto_execution_order, "avm1/goto_execution_order", 3), - (goto_execution_order2, "avm1/goto_execution_order2", 3), + (goto_execution_order2, "avm1/goto_execution_order2", 2), (greaterthan_swf5, "avm1/greaterthan_swf5", 1), (greaterthan_swf8, "avm1/greaterthan_swf8", 1), (strictly_equals, "avm1/strictly_equals", 1),