plainbox:sessions:legacy: switch tests to test compat implementation

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
plainbox:provider: PEP8 fixes

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
plainbox:session:legacy: fix session_dir property

The property should not raise AttributeError before open() is called.
It should return None instead.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
plainbox: drop original implementation of suspend/resume

This very large patch drops all kinds of code through the plainbox tree.
The code either implements suspend, mostly via
_get_persistence_subset(), or resume, via from_json(). There are a
number of support classes that got removed (SessionStateEncoder, a bunch
of classes for io logs). Obviously a large amount of tests are dropped
as well.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
plainbox:service: fix RunningJob id generator

Each DBus object needs to have an unique identifier path. This cannot
be reliably obtained using random numbers as collisions are still possible.
This patch switches to python object identity to generate DBus
object paths as those are uniquer during the lifetime of the object.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
plainbox:service: make metadata settable

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
plainbox:service: remove unused RunningJob.SetComments() method

This method is never used by the current GUI and is not implemented anyway

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
plainbox:service: rename some internal methods

This patch renames publish_objects to publish_related_objects
and publish_children to publish_managed_objects. It also alters the
docstrings so that the difference between the two is more evident.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
plainbox: add notes about API issues in service and higlevel modules

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
plainbox:applogic: add WhiteList.__repr__()

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
plainbox:session:jobs: modify on_result_changed(old, new) signal

The on_result_changed() signal, which is fired when the result associated
with a job state is changed, now keeps track of the new and old result
object.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
"automatic merge by tarmac [r=roadmr][bug=][author=zkrynicki]"
"automatic merge by tarmac [r=roadmr][bug=][author=zkrynicki]"
plainbox:service: expose IJobResult.execution_duration over DBus

This fixes bug #1221393

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
"automatic merge by tarmac [r=roadmr][bug=1221393][author=zkrynicki]"
plainbox:providers:stubbox: fix the location of the stubbox provider

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
"automatic merge by tarmac [r=zkrynicki][bug=][author=zkrynicki]"
Launchpad automatic translations update.
plainbox:session:legacy: fix typo

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
plainbox:session:legacy: drop unused imports

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>