From 9cdf01085ffb974bde24c2409d004263c71da755 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= Date: Tue, 25 Feb 2020 14:28:33 +0100 Subject: Constraints: added unit test for Child Of with bone target No functional changes. --- tests/python/bl_constraints.py | 83 +++++++++++++++++++++++++++--------------- 1 file changed, 54 insertions(+), 29 deletions(-) (limited to 'tests/python/bl_constraints.py') diff --git a/tests/python/bl_constraints.py b/tests/python/bl_constraints.py index 1b3ced9b4d7..2a48ffe35ea 100644 --- a/tests/python/bl_constraints.py +++ b/tests/python/bl_constraints.py @@ -34,16 +34,17 @@ class AbstractConstraintTests(unittest.TestCase): def setUp(self): bpy.ops.wm.open_mainfile(filepath=str(args.testdir / "constraints.blend")) - def assert_matrix(self, actual, expect, places=6, delta=None): + def assert_matrix(self, actual_matrix, expect_matrix, object_name: str, places=6, delta=None): """Asserts that the matrices almost equal.""" - self.assertEqual(len(actual), 4, 'Expected a 4x4 matrix') + self.assertEqual(len(actual_matrix), 4, 'Expected a 4x4 matrix') # TODO(Sybren): decompose the matrices and compare loc, rot, and scale separately. # That'll probably improve readability & understandability of test failures. - for row, (act_row, exp_row) in enumerate(zip(actual, expect)): - for col, (act, exp) in enumerate(zip(act_row, exp_row)): - self.assertAlmostEqual(act, exp, places=places, delta=delta, - msg=f'{act} != {exp} at element [{row}][{col}]') + for row, (act_row, exp_row) in enumerate(zip(actual_matrix, expect_matrix)): + for col, (actual, expect) in enumerate(zip(act_row, exp_row)): + self.assertAlmostEqual( + actual, expect, places=places, delta=delta, + msg=f'Matrix of object {object_name!r} failed: {actual} != {expect} at element [{row}][{col}]') def matrix(self, object_name: str) -> Matrix: """Return the evaluated world matrix.""" @@ -56,35 +57,38 @@ class AbstractConstraintTests(unittest.TestCase): def matrix_test(self, object_name: str, expect: Matrix): """Assert that the object's world matrix is as expected.""" actual = self.matrix(object_name) - self.assert_matrix(actual, expect) + self.assert_matrix(actual, expect, object_name) - def constraint_context(self, constraint_name: str) -> dict: + def constraint_context(self, constraint_name: str, owner_name: str='') -> dict: """Return a context suitable for calling constraint operators. - Assumes the owner is called "{constraint_name}.owner". + Assumes the owner is called "{constraint_name}.owner" if owner_name=''. """ - owner = bpy.context.scene.objects['Child Of.owner'] + owner = bpy.context.scene.objects[owner_name or f'{constraint_name}.owner'] constraint = owner.constraints[constraint_name] context = { **bpy.context.copy(), + 'object': owner, + 'active_object': owner, 'constraint': constraint, } return context + class ChildOfTest(AbstractConstraintTests): - def test_childof_simple_parent(self): - """Child Of: simple evaluation.""" + def test_object_simple_parent(self): + """Child Of: simple evaluation of object parent.""" initial_matrix = Matrix(( (0.5872668623924255, -0.3642929494380951, 0.29567837715148926, 1.0886117219924927), (0.31689348816871643, 0.7095895409584045, 0.05480116978287697, 2.178966999053955), (-0.21244174242019653, 0.06738340109586716, 0.8475662469863892, 3.2520291805267334), (0.0, 0.0, 0.0, 1.0), )) - self.matrix_test('Child Of.owner', initial_matrix) + self.matrix_test('Child Of.object.owner', initial_matrix) - context = self.constraint_context('Child Of') + context = self.constraint_context('Child Of', owner_name='Child Of.object.owner') bpy.ops.constraint.childof_set_inverse(context, constraint='Child Of') - self.matrix_test('Child Of.owner', Matrix(( + self.matrix_test('Child Of.object.owner', Matrix(( (0.9992385506629944, 0.019844001159071922, -0.03359175845980644, 0.10000011324882507), (-0.01744179055094719, 0.997369647026062, 0.07035345584154129, 0.1999998837709427), (0.034899525344371796, -0.06971397250890732, 0.9969563484191895, 0.3000001311302185), @@ -92,11 +96,11 @@ class ChildOfTest(AbstractConstraintTests): ))) bpy.ops.constraint.childof_clear_inverse(context, constraint='Child Of') - self.matrix_test('Child Of.owner', initial_matrix) + self.matrix_test('Child Of.object.owner', initial_matrix) - def test_childof_rotation_only(self): + def test_object_rotation_only(self): """Child Of: rotation only.""" - owner = bpy.context.scene.objects['Child Of.owner'] + owner = bpy.context.scene.objects['Child Of.object.owner'] constraint = owner.constraints['Child Of'] constraint.use_location_x = constraint.use_location_y = constraint.use_location_z = False constraint.use_scale_x = constraint.use_scale_y = constraint.use_scale_z = False @@ -107,11 +111,11 @@ class ChildOfTest(AbstractConstraintTests): (-0.31230923533439636, 0.09144517779350281, 0.9455690383911133, 0.30000001192092896), (0.0, 0.0, 0.0, 1.0), )) - self.matrix_test('Child Of.owner', initial_matrix) + self.matrix_test('Child Of.object.owner', initial_matrix) - context = self.constraint_context('Child Of') + context = self.constraint_context('Child Of', owner_name='Child Of.object.owner') bpy.ops.constraint.childof_set_inverse(context, constraint='Child Of') - self.matrix_test('Child Of.owner', Matrix(( + self.matrix_test('Child Of.object.owner', Matrix(( (0.9992386102676392, 0.019843975082039833, -0.033591702580451965, 0.10000000149011612), (-0.017441781237721443, 0.9973695874214172, 0.0703534483909607, 0.20000000298023224), (0.03489946573972702, -0.06971397250890732, 0.9969563484191895, 0.30000001192092896), @@ -119,12 +123,11 @@ class ChildOfTest(AbstractConstraintTests): ))) bpy.ops.constraint.childof_clear_inverse(context, constraint='Child Of') - self.matrix_test('Child Of.owner', initial_matrix) - + self.matrix_test('Child Of.object.owner', initial_matrix) - def test_childof_no_x_axis(self): + def test_object_no_x_axis(self): """Child Of: loc/rot/scale on only Y and Z axes.""" - owner = bpy.context.scene.objects['Child Of.owner'] + owner = bpy.context.scene.objects['Child Of.object.owner'] constraint = owner.constraints['Child Of'] constraint.use_location_x = False constraint.use_rotation_x = False @@ -136,11 +139,11 @@ class ChildOfTest(AbstractConstraintTests): (-0.31224438548088074, -0.06574578583240509, 0.8546382784843445, 3.219514846801758), (0.0, 0.0, 0.0, 1.0), )) - self.matrix_test('Child Of.owner', initial_matrix) + self.matrix_test('Child Of.object.owner', initial_matrix) - context = self.constraint_context('Child Of') + context = self.constraint_context('Child Of', owner_name='Child Of.object.owner') bpy.ops.constraint.childof_set_inverse(context, constraint='Child Of') - self.matrix_test('Child Of.owner', Matrix(( + self.matrix_test('Child Of.object.owner', Matrix(( (0.9228900671005249, 0.23250490427017212, -0.035540513694286346, 0.10000000149011612), (-0.011224273592233658, 0.9838480949401855, 0.24731633067131042, 0.21246682107448578), (0.0383986234664917, -0.3163823187351227, 0.9553266167640686, 0.27248233556747437), @@ -148,8 +151,30 @@ class ChildOfTest(AbstractConstraintTests): ))) bpy.ops.constraint.childof_clear_inverse(context, constraint='Child Of') - self.matrix_test('Child Of.owner', initial_matrix) + self.matrix_test('Child Of.object.owner', initial_matrix) + + def test_bone_simple_parent(self): + """Child Of: simple evaluation of bone parent.""" + initial_matrix = Matrix(( + (0.5133683681488037, -0.06418320536613464, -0.014910104684531689, -0.2277737855911255), + (-0.03355155512690544, 0.12542974948883057, -1.080492377281189, 2.082599639892578), + (0.019313642755150795, 1.0446348190307617, 0.1244703009724617, 2.8542778491973877), + (0.0, 0.0, 0.0, 1.0), + )) + self.matrix_test('Child Of.armature.owner', initial_matrix) + context = self.constraint_context('Child Of', owner_name='Child Of.armature.owner') + bpy.ops.constraint.childof_set_inverse(context, constraint='Child Of') + + self.matrix_test('Child Of.armature.owner', Matrix(( + (0.9992386102676392, 0.019843988120555878, -0.03359176218509674, 0.8358089923858643), + (-0.017441775649785995, 0.997369647026062, 0.0703534483909607, 1.7178752422332764), + (0.03489949554204941, -0.06971397995948792, 0.9969563484191895, -1.8132872581481934), + (0.0, 0.0, 0.0, 1.0), + ))) + + bpy.ops.constraint.childof_clear_inverse(context, constraint='Child Of') + self.matrix_test('Child Of.armature.owner', initial_matrix) def main(): global args -- cgit v1.2.3