raoul/openlp
1
0
Fork 0
forked from openlp/openlp
Code Issues Pull Requests Projects Releases Wiki Activity

[bug 1410694] Fixing typo, also add additional error.

Browse Source 
Add the OS Name and version to the version file request headers

bzr-revno: 2476
Fixes: https://launchpad.net/bugs/1410694
This commit is contained in:
Philip Ridout 2015-01-16 22:40:35 +02:00 committed by Raoul Snyman
commit bd48100e55
2 changed files with 17 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
openlp/core/utils/__init__.py
View File

@ -34,6 +34,7 @@ from distutils.version import LooseVersion
import logging import logging
import locale import locale
import os import os
import platform
import re import re
import time import time
from shutil import which from shutil import which
@ -251,14 +252,15 @@ def check_latest_version(current_version):
req = urllib.request.Request('http://www.openlp.org/files/dev_version.txt') req = urllib.request.Request('http://www.openlp.org/files/dev_version.txt')
else: else:
req = urllib.request.Request('http://www.openlp.org/files/version.txt') req = urllib.request.Request('http://www.openlp.org/files/version.txt')
req.add_header('User-Agent', 'OpenLP/%s' % current_version['full']) req.add_header('User-Agent', 'OpenLP/%s %s/%s; ' % (current_version['full'], platform.system(),
platform.release()))
remote_version = None remote_version = None
retries = 0 retries = 0
while True: while True:
try: try:
remote_version = str(urllib.request.urlopen(req, None, remote_version = str(urllib.request.urlopen(req, None,
timeout=CONNECTION_TIMEOUT).read().decode()).strip() timeout=CONNECTION_TIMEOUT).read().decode()).strip()
except ConnectionException: except (urllib.error.URLError, ConnectionError):
if retries > CONNECTION_RETRIES: if retries > CONNECTION_RETRIES:
log.exception('Failed to download the latest OpenLP version file') log.exception('Failed to download the latest OpenLP version file')
else: else:

13
tests/functional/openlp_core_utils/test_utils.py
View File

@ -236,6 +236,19 @@ class TestUtils(TestCase):
# THEN: the resolve method is called with the correct argument # THEN: the resolve method is called with the correct argument
mock_resolver.resolve.assert_called_with('uno:pipe,name=openlp_pipe;urp;StarOffice.ComponentContext') mock_resolver.resolve.assert_called_with('uno:pipe,name=openlp_pipe;urp;StarOffice.ComponentContext')
def get_uno_instance_socket_test(self):
"""
Test that when the UNO connection type is other than "pipe" the resolver is given the "socket" URI
"""
# GIVEN: A mock resolver object and UNO_CONNECTION_TYPE is "socket"
mock_resolver = MagicMock()
# WHEN: get_uno_instance() is called
get_uno_instance(mock_resolver, 'socket')
# THEN: the resolve method is called with the correct argument
mock_resolver.resolve.assert_called_with('uno:socket,host=localhost,port=2002;urp;StarOffice.ComponentContext')
def get_user_agent_linux_test(self): def get_user_agent_linux_test(self):
""" """
Test that getting a user agent on Linux returns a user agent suitable for Linux Test that getting a user agent on Linux returns a user agent suitable for Linux