QA: Mark failing SHAPE_POLY_SET tests as expected failures

This allows the tests to remain sensitive to other errors.

When SHAPE_POLY_SET is fixed, these expected failures should be
removed.
This commit is contained in:
John Beard 2019-01-30 09:45:05 +00:00 committed by jean-pierre charras
parent 872f0eb44c
commit 8e75f6029c
1 changed files with 7 additions and 3 deletions

View File

@ -77,7 +77,7 @@ struct CollisionFixture
/**
* Declares the CollisionFixture as the boost test suite fixture.
*/
BOOST_FIXTURE_TEST_SUITE( Collision, CollisionFixture )
BOOST_FIXTURE_TEST_SUITE( SPSCollision, CollisionFixture )
/**
* Simple dummy test to check that HasHoles() definition is right
@ -113,11 +113,13 @@ BOOST_AUTO_TEST_CASE( PointOnEdge )
BOOST_CHECK( !common.holeyPolySet.PointOnEdge( VECTOR2I( 200, 200 ) ) );
}
#ifdef HAVE_EXPECTED_FAILURES
/**
* This test checks that the function Contains, whose behaviour has been updated to also manage
* holey polygons, does the right work.
*/
BOOST_AUTO_TEST_CASE( pointInPolygonSet )
BOOST_AUTO_TEST_CASE( pointInPolygonSet, *boost::unit_test::expected_failures( 1 ) )
{
// Check that the set contains the points that collide with it
for( const VECTOR2I& point : collidingPoints )
@ -135,7 +137,7 @@ BOOST_AUTO_TEST_CASE( pointInPolygonSet )
/**
* This test checks the behaviour of the Collide (with a point) method.
*/
BOOST_AUTO_TEST_CASE( Collide )
BOOST_AUTO_TEST_CASE( Collide, *boost::unit_test::expected_failures( 1 ) )
{
// When clearance = 0, the behaviour should be the same as with Contains
@ -160,6 +162,8 @@ BOOST_AUTO_TEST_CASE( Collide )
BOOST_CHECK( common.holeyPolySet.Collide( VECTOR2I( 11, 11 ), 5 ) );
}
#endif
/**
* This test checks the behaviour of the CollideVertex method, testing whether the collision with
* vertices is well detected