Skip to content

Commit 3d397aa

Browse files
nshyDifferentialOrange
authored andcommitted
test: use coverage instead of equality to test error payload
We are going to add missing 'user' payload field for ACCESS_DENIED error which will break current tests. Let rewrite tests to allow adding new payload fields for this error. Need for tarantool/tarantool#9108
1 parent a12f6e7 commit 3d397aa

File tree

2 files changed

+8
-4
lines changed

2 files changed

+8
-4
lines changed

test/suites/test_dml.py

+3-3
Original file line numberDiff line numberDiff line change
@@ -419,13 +419,13 @@ def test_16_extra_error_info_fields(self):
419419
"Create access to function 'forbidden_function' is denied for user 'test'")
420420
self.assertEqual(exc.extra_info.errno, 0)
421421
self.assertEqual(exc.extra_info.errcode, 42)
422-
self.assertEqual(
423-
exc.extra_info.fields,
422+
self.assertGreaterEqual(
423+
exc.extra_info.fields.items(),
424424
{
425425
'object_type': 'function',
426426
'object_name': 'forbidden_function',
427427
'access_type': 'Create'
428-
})
428+
}.items())
429429
self.assertEqual(exc.extra_info.prev, None)
430430
else:
431431
self.fail('Expected error')

test/suites/test_error_ext.py

+5-1
Original file line numberDiff line numberDiff line change
@@ -327,7 +327,11 @@ def test_tarantool_decode(self):
327327
self.assertEqual(err.message, expected_err.message)
328328
self.assertEqual(err.errno, expected_err.errno)
329329
self.assertEqual(err.errcode, expected_err.errcode)
330-
self.assertEqual(err.fields, expected_err.fields)
330+
if expected_err.fields is not None:
331+
self.assertGreaterEqual(err.fields.items(),
332+
expected_err.fields.items())
333+
else:
334+
self.assertEqual(err.fields, None)
331335

332336
err = err.prev
333337
expected_err = expected_err.prev

0 commit comments

Comments
 (0)