openlp-core team mailing list archive

[Raoul Snyman <raoul@xxxxxxxxxxx>]

Public bug reported:

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 "OpenLP/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 "OpenLP/openlp/core/ui/media/vendor/mediainfoWrapper.py", line 130, in tracks
    self._populate_tracks()
  File "OpenLP/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'

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

** Affects: openlp
     Importance: High
         Status: New

** Description changed:

  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"):
+   File "OpenLP/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 "OpenLP/openlp/core/ui/media/vendor/mediainfoWrapper.py", line 130, in tracks
+     self._populate_tracks()
+   File "OpenLP/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'
  
  ----------------------------------------------------------------------

-- 
You received this bug notification because you are a member of OpenLP
Core, which is subscribed to OpenLP.
https://bugs.launchpad.net/bugs/1732348

Title:
  MediaInfo's XML output has changed

Status in OpenLP:
  New

Bug description:
  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 "OpenLP/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 "OpenLP/openlp/core/ui/media/vendor/mediainfoWrapper.py", line 130, in tracks
      self._populate_tracks()
    File "OpenLP/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'

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

To manage notifications about this bug go to:
https://bugs.launchpad.net/openlp/+bug/1732348/+subscriptions