Commit b2703b2d authored by Jérome Perrin's avatar Jérome Perrin

interfaces.application.IApplication: update loadSettings signature with actual usage

Over the years (for example commits 89d754f2, 58a72658, a6fc7dfe, etc),
the signature of the implementation loadSettings evolved and the
interface became outdated.

This updates the interface with current implementation, so that we can
use zope.interface.verify.verifyClass in the tests
parent 78d15e09
...@@ -89,7 +89,7 @@ class OpenOffice(Application): ...@@ -89,7 +89,7 @@ class OpenOffice(Application):
def loadSettings(self, hostname, port, path_run_dir, def loadSettings(self, hostname, port, path_run_dir,
office_binary_path, uno_path, default_language, office_binary_path, uno_path, default_language,
environment_dict=None, **kw): environment_dict=None, **kw):
"""Method to load the configuratio to control one OpenOffice Instance """Method to load the configuration to control one OpenOffice Instance
Keyword arguments: Keyword arguments:
office_path -- Full Path of the OOo executable. office_path -- Full Path of the OOo executable.
e.g office_binary_path='/opt/openoffice.org3/program' e.g office_binary_path='/opt/openoffice.org3/program'
......
...@@ -45,10 +45,15 @@ class IApplication(Interface): ...@@ -45,10 +45,15 @@ class IApplication(Interface):
def pid(): def pid():
"""Get the process pid""" """Get the process pid"""
def loadSettings(hostname, def loadSettings(
hostname,
port, port,
working_path, working_path,
office_binary_path): office_binary_path,
uno_path,
default_language,
environment_dict=None,
):
"""Load configuration to control OOo Instances""" """Load configuration to control OOo Instances"""
def status(): def status():
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment