Fix manifest comparisons in new collection tests
authorBrett Smith <brett.smith@curii.com>
Mon, 28 Nov 2022 20:25:03 +0000 (15:25 -0500)
committerBrett Smith <brett.smith@curii.com>
Mon, 28 Nov 2022 20:25:03 +0000 (15:25 -0500)
commit032ecb480134f6138346865fba7732a836b809c6
treebd00ee6f588d773497ac015d8b198f80080a56c1
parent2b7d05cdf4e054024607b859bd6fb41e04855bfa
Fix manifest comparisons in new collection tests

The previous version of these tests was comparing bound method objects
directly, rather than comparing the manifest texts returned by calling
them. Apparently in some setups this comparison returns True, but on my
development box it returns False, causing test failures. We haven't
fully diagnosed what causes the difference, but it seems clear enough
that comparing the method objects isn't the intent of these tests, but
rather they should compare the returned manifests. This commit updates
them to do so. No issue #.

Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>
sdk/python/tests/test_collections.py