Comment 0 for bug 1732348

Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

As of version 17.10, MediaInfo's XML output has changed, causing tests to fail (and no doubt OpenLP as well). Unfortunately while the changelog says that you can still use the old XML, the command line utility itself does not seem to support that option (I can't figure out how to give it the option).

Changelog: https://mediaarea.net/MediaInfo/ChangeLog

Test output:

======================================================================
ERROR: test_media_length (tests.interfaces.openlp_core.ui.media.vendor.test_mediainfoWrapper.TestMediainfoWrapper)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/raoul/Projects/OpenLP/OpenLP/settings-upgrade/tests/interfaces/openlp_core/ui/media/vendor/test_mediainfoWrapper.py", line 48, in test_media_length
    self.assertEqual(results.tracks[0].duration, test_data[1], 'The correct duration is returned for ' +
  File "/home/raoul/Projects/OpenLP/OpenLP/settings-upgrade/openlp/core/ui/media/vendor/mediainfoWrapper.py", line 130, in tracks
    self._populate_tracks()
  File "/home/raoul/Projects/OpenLP/OpenLP/settings-upgrade/openlp/core/ui/media/vendor/mediainfoWrapper.py", line 122, in _populate_tracks
    for xml_track in self.xml_dom.Mediainfo.File.find_all("track"):
AttributeError: 'NoneType' object has no attribute 'File'

----------------------------------------------------------------------