gaming/lutris
gaming/lutris
1
0
Fork 0
mirror of https://github.com/lutris/lutris synced 2024-09-04 23:13:35 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix tests for new installer structure

Browse source
This commit is contained in:
Mathieu Comandon 2017-04-13 19:33:12 -07:00
parent bf77341827
commit f0d6563067
2 changed files with 13 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
lutris/installer/interpreter.py
View file

@ -71,12 +71,12 @@ class ScriptInterpreter(CommandsMixin):
self.files = self.script.get('files', [])
self.slug = installer['slug']
self.year = installer['year']
self.year = installer.get('year')
self.name = installer['name']
self.runner = installer['runner']
self.game_name = installer['name']
self.game_slug = installer['game_slug']
self.steamid = installer['steamid']
self.steamid = installer.get('steamid')
self.requires = self.script.get('requires')
self.extends = self.script.get('extends')

21
tests/test_installer.py
View file

@ -5,7 +5,7 @@ from lutris.installer.errors import ScriptingError
class MockInterpreter(ScriptInterpreter):
"""A script interpreter mock."""
script = {'runner': 'linux'}
runner = 'linux'
def is_valid(self):
return True
@ -13,20 +13,21 @@ class MockInterpreter(ScriptInterpreter):
class TestScriptInterpreter(TestCase):
def test_script_with_correct_values_is_valid(self):
script = {
'runner': 'wine',
'installer': [],
'name': 'baz',
'game_slug': 'baz',
installer = {
'runner': 'linux',
'script': {'exe': 'doom'},
'name': 'Doom',
'slug': 'doom',
'game_slug': 'doom',
}
interpreter = ScriptInterpreter(script, None)
interpreter = ScriptInterpreter(installer, None)
self.assertFalse(interpreter.errors)
self.assertTrue(interpreter.is_valid())
def test_move_requires_src_and_dst(self):
script = {
'foo': 'bar',
'installer': [],
'script': [],
'name': 'missing_runner',
'game_slug': 'missing-runner'
}
@ -35,14 +36,14 @@ class TestScriptInterpreter(TestCase):
interpreter._get_move_paths({})
def test_get_command_returns_a_method(self):
interpreter = MockInterpreter({}, None)
interpreter = MockInterpreter({'script': []}, None)
command, params = interpreter._map_command({'move': 'whatever'})
self.assertIn("bound method CommandsMixin.move", str(command))
self.assertEqual(params, "whatever")
def test_get_command_doesnt_return_private_methods(self):
""" """
interpreter = MockInterpreter({}, None)
interpreter = MockInterpreter({'script': []}, None)
with self.assertRaises(ScriptingError) as ex:
command, params = interpreter._map_command(
{'_substitute': 'foo'}