comments are not preserved in some cases (user-interact)

[Zygmunt Krynicki]

}vagrant@vagrant-ubuntu-trusty-32:~/src/plainbox$ stubbox run -i 2013.com.canonical.plainbox::representative/plugin/user-interact -f json -p with-io-log,with-comments
===============================[ Analyzing Jobs ]===============================
=============================[ Session Statistics ]=============================
This session is about 0.00% complete
Estimated duration cannot be determined for automated jobs.
Estimated duration is 60.00 for manual jobs.
Size of the desired job list: 1
Size of the effective execution plan: 1
===========================[ Running Selected Jobs ]============================
--------------[ Running job 1 / 1. Estimated time left: 0:00:30 ]---------------
-----------------------[ Job with plugin=user-interact ]------------------------
ID: 2013.com.canonical.plainbox::representative/plugin/user-interact
Category: 2013.com.canonical.plainbox::plugin-representative
Job with plugin=user-interact
Pick an action
    => press ENTER to continue
  c => add a comment
  s => skip this job
  q => save the session and quit
[csq]: c
Please enter your comments:
before
Job with plugin=user-interact
Pick an action
    => press ENTER to continue
  c => add a comment
  s => skip this job
  q => save the session and quit
[csq]:
... 8< -------------------------------------------------------------------------
(interaction)
------------------------------------------------------------------------- >8 ---
Outcome: job passed
==================================[ Results ]===================================
{
    "result_map": {
        "2013.com.canonical.plainbox::representative/plugin/user-interact": {
            "summary": "Job with plugin=user-interact",
            "category_id": "2013.com.canonical.plainbox::plugin-representative",
            "outcome": "pass",
            "execution_duration": 4.600465774536133,
            "comments": null,
            "io_log": [
                [
                    0.005313,
                    "stdout",
                    "KGludGVyYWN0aW9uKQo="
                ]
            ]
        }
    }
}vagrant@vagrant-ubuntu-trusty-32:~/src/plainbox$

[Sylvain Pineau (sylvain-pineau)]

11:57 <@zyga> spineau: that bug is more complex
11:57 <@zyga> spineau: +1 on the merge but there's a different problem there too
11:57 <@zyga> spineau: you can Ctrl-C at so many places after entering input and before we have a result
11:57 <@zyga> spineau: that we still lose stuff
11:57 <@zyga> spineau: the only way to solve that would to instantly keep a partial "under construction" result in the session somewhere
11:58 <@spineau> zyga: I see