forked from openlp/openlp
Fix bug and add new test or bibles
This commit is contained in:
parent
f3e77b39d7
commit
d253ef5852
@ -591,7 +591,8 @@ class SlideController(DisplayController):
|
|||||||
"""
|
"""
|
||||||
request = self.sender().text()
|
request = self.sender().text()
|
||||||
slide_no = self.slideList[request]
|
slide_no = self.slideList[request]
|
||||||
self.__updatePreviewSelection(slide_no)
|
width = self.main_window.controlSplitter.sizes()[self.split]
|
||||||
|
self.preview_widget.replace_service_item(self.service_item, width, slide_no)
|
||||||
self.slideSelected()
|
self.slideSelected()
|
||||||
|
|
||||||
def receive_spin_delay(self):
|
def receive_spin_delay(self):
|
||||||
@ -757,10 +758,8 @@ class SlideController(DisplayController):
|
|||||||
self.display.audio_player.play()
|
self.display.audio_player.play()
|
||||||
self.setAudioItemsVisibility(True)
|
self.setAudioItemsVisibility(True)
|
||||||
row = 0
|
row = 0
|
||||||
text = []
|
|
||||||
width = self.main_window.controlSplitter.sizes()[self.split]
|
width = self.main_window.controlSplitter.sizes()[self.split]
|
||||||
for framenumber, frame in enumerate(self.service_item.get_frames()):
|
for framenumber, frame in enumerate(self.service_item.get_frames()):
|
||||||
slideHeight = 0
|
|
||||||
if self.service_item.is_text():
|
if self.service_item.is_text():
|
||||||
if frame[u'verseTag']:
|
if frame[u'verseTag']:
|
||||||
# These tags are already translated.
|
# These tags are already translated.
|
||||||
@ -776,7 +775,6 @@ class SlideController(DisplayController):
|
|||||||
row += 1
|
row += 1
|
||||||
self.slideList[unicode(row)] = row - 1
|
self.slideList[unicode(row)] = row - 1
|
||||||
else:
|
else:
|
||||||
slideHeight = width * (1 / self.ratio)
|
|
||||||
row += 1
|
row += 1
|
||||||
self.slideList[unicode(row)] = row - 1
|
self.slideList[unicode(row)] = row - 1
|
||||||
# If current slide set background to image
|
# If current slide set background to image
|
||||||
@ -867,7 +865,7 @@ class SlideController(DisplayController):
|
|||||||
Settings().remove(self.main_window.general_settings_section + u'/screen blank')
|
Settings().remove(self.main_window.general_settings_section + u'/screen blank')
|
||||||
self.blankPlugin()
|
self.blankPlugin()
|
||||||
self.updatePreview()
|
self.updatePreview()
|
||||||
self.onToggleLoop()
|
self.on_toggle_loop()
|
||||||
|
|
||||||
def onThemeDisplay(self, checked=None):
|
def onThemeDisplay(self, checked=None):
|
||||||
"""
|
"""
|
||||||
@ -886,7 +884,7 @@ class SlideController(DisplayController):
|
|||||||
Settings().remove(self.main_window.general_settings_section + u'/screen blank')
|
Settings().remove(self.main_window.general_settings_section + u'/screen blank')
|
||||||
self.blankPlugin()
|
self.blankPlugin()
|
||||||
self.updatePreview()
|
self.updatePreview()
|
||||||
self.onToggleLoop()
|
self.on_toggle_loop()
|
||||||
|
|
||||||
def onHideDisplay(self, checked=None):
|
def onHideDisplay(self, checked=None):
|
||||||
"""
|
"""
|
||||||
@ -905,7 +903,7 @@ class SlideController(DisplayController):
|
|||||||
Settings().remove(self.main_window.general_settings_section + u'/screen blank')
|
Settings().remove(self.main_window.general_settings_section + u'/screen blank')
|
||||||
self.hidePlugin(checked)
|
self.hidePlugin(checked)
|
||||||
self.updatePreview()
|
self.updatePreview()
|
||||||
self.onToggleLoop()
|
self.on_toggle_loop()
|
||||||
|
|
||||||
def blankPlugin(self):
|
def blankPlugin(self):
|
||||||
"""
|
"""
|
||||||
@ -1073,7 +1071,7 @@ class SlideController(DisplayController):
|
|||||||
self.preview_widget.change_slide(row)
|
self.preview_widget.change_slide(row)
|
||||||
self.slideSelected()
|
self.slideSelected()
|
||||||
|
|
||||||
def onToggleLoop(self):
|
def on_toggle_loop(self):
|
||||||
"""
|
"""
|
||||||
Toggles the loop state.
|
Toggles the loop state.
|
||||||
"""
|
"""
|
||||||
@ -1117,7 +1115,7 @@ class SlideController(DisplayController):
|
|||||||
else:
|
else:
|
||||||
self.play_slides_loop.setIcon(build_icon(u':/media/media_time.png'))
|
self.play_slides_loop.setIcon(build_icon(u':/media/media_time.png'))
|
||||||
self.play_slides_loop.setText(UiStrings().PlaySlidesInLoop)
|
self.play_slides_loop.setText(UiStrings().PlaySlidesInLoop)
|
||||||
self.onToggleLoop()
|
self.on_toggle_loop()
|
||||||
|
|
||||||
def onPlaySlidesOnce(self, checked=None):
|
def onPlaySlidesOnce(self, checked=None):
|
||||||
"""
|
"""
|
||||||
@ -1138,7 +1136,7 @@ class SlideController(DisplayController):
|
|||||||
else:
|
else:
|
||||||
self.play_slides_once.setIcon(build_icon(u':/media/media_time'))
|
self.play_slides_once.setIcon(build_icon(u':/media/media_time'))
|
||||||
self.play_slides_once.setText(UiStrings().PlaySlidesToEnd)
|
self.play_slides_once.setText(UiStrings().PlaySlidesToEnd)
|
||||||
self.onToggleLoop()
|
self.on_toggle_loop()
|
||||||
|
|
||||||
def setAudioItemsVisibility(self, visible):
|
def setAudioItemsVisibility(self, visible):
|
||||||
"""
|
"""
|
||||||
|
1
tests/functional/openlp_plugins/bibles/__init__.py
Normal file
1
tests/functional/openlp_plugins/bibles/__init__.py
Normal file
@ -0,0 +1 @@
|
|||||||
|
__author__ = 'tim'
|
33
tests/functional/openlp_plugins/bibles/test_lib_http.py
Normal file
33
tests/functional/openlp_plugins/bibles/test_lib_http.py
Normal file
@ -0,0 +1,33 @@
|
|||||||
|
"""
|
||||||
|
Package to test the openlp.plugin.bible.lib.https package.
|
||||||
|
"""
|
||||||
|
|
||||||
|
from unittest import TestCase
|
||||||
|
from mock import MagicMock, patch
|
||||||
|
|
||||||
|
from openlp.core.lib import Registry
|
||||||
|
from openlp.plugins.bibles.lib.http import BGExtract
|
||||||
|
|
||||||
|
|
||||||
|
class TestBibleHTTP(TestCase):
|
||||||
|
|
||||||
|
def setUp(self):
|
||||||
|
"""
|
||||||
|
Set up the Registry
|
||||||
|
"""
|
||||||
|
Registry.create()
|
||||||
|
Registry().register(u'service_list', MagicMock())
|
||||||
|
Registry().register(u'application', MagicMock())
|
||||||
|
|
||||||
|
def bible_gateway_extract_test(self):
|
||||||
|
"""
|
||||||
|
Test the Bible Gateway retrieval of book list for NIV
|
||||||
|
"""
|
||||||
|
# GIVEN: A new Bible Gateway extraction class
|
||||||
|
handler = BGExtract()
|
||||||
|
|
||||||
|
# WHEN: The Books list is called
|
||||||
|
books = handler.get_books_from_http(u'NIV')
|
||||||
|
|
||||||
|
# THEN: We should get back a valid service item
|
||||||
|
assert len(books) == 66, u'The bible should not have had its lenght changed'
|
Loading…
Reference in New Issue
Block a user