[tracker-miners/sam/extract-tests-fix: 1/2] Revert "tests: Drop assert requiring perfect multivalued resource matches"



commit 803f9bbd5a8aa2697d20e4407bec5fc5b4042fd0
Author: Sam Thursfield <sam afuera me uk>
Date:   Sat Jun 27 15:14:25 2020 +0200

    Revert "tests: Drop assert requiring perfect multivalued resource matches"
    
    This reverts commit df314b5af77d69eefb7d51e5b5613b6e922d114e.

 tests/functional-tests/fixtures.py | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)
---
diff --git a/tests/functional-tests/fixtures.py b/tests/functional-tests/fixtures.py
index 96da81a29..f6efe7adb 100644
--- a/tests/functional-tests/fixtures.py
+++ b/tests/functional-tests/fixtures.py
@@ -358,12 +358,8 @@ class TrackerExtractTestCase(ut.TestCase):
                         raise AssertionError("Expected a list property for %s, but got a %s: %s" % (
                             prop, type(result[prop]).__name__, result[prop]))
 
-                    # FIXME: depending on the gstreamer version we link to, we may extract different
-                    # nfo:hasHash and tracker:hasExternalReference. Those tags should probably be
-                    # handled in separate tests that we can disable entirely with build-time checks.
-                    # This check should be restored once we can thoroughly compare outputs again.
-                    #self.assertEqual(len(expected_value), len(result[prop]),
-                    #                    error_wrong_length % (prop, filename, spec_filename))
+                    self.assertEqual(len(expected_value), len(result[prop]),
+                                        error_wrong_length % (prop, filename, spec_filename))
 
                     for i in range(0, len(expected_value)):
                         if isinstance(expected_value[i], dict):


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]