Better text

canonical · Mar 15, 2024 · f7a7aad · f7a7aad
1 parent 81d8a8b
commit f7a7aad
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 12 deletions.
diff --git a/checkbox-ng/plainbox/impl/session/remote_assistant.py b/checkbox-ng/plainbox/impl/session/remote_assistant.py
@@ -769,16 +769,15 @@ def resume_by_id(self, session_id=None, overwrite_result_dict={}):
                 if "noreturn" in the_job.get_flag_set():
                     result_dict["outcome"] = IJobResult.OUTCOME_PASS
                     result_dict["comments"] = (
-                        "Job crashed the machine or the Checkbox agent, "
-                        "automatically marking it as passed after resuming "
-                        "the session because it has the noreturn flag"
+                        "Job rebooted the machine or the Checkbox agent. "
+                        "Resuming the session and marking it as passed "
+                        "because the job has the `noreturn` flag"
                     )
                 else:
                     result_dict["outcome"] = IJobResult.OUTCOME_CRASH
                     result_dict["comments"] = (
-                        "Job crashed the machine or the Checkbox agent, "
-                        "automatically marking it as crashed after resuming "
-                        "the session"
+                        "Job rebooted the machine or the Checkbox agent. "
+                        "Resuming the session and marking it as crashed."
                     )
 
         result_dict.update(overwrite_result_dict)

diff --git a/checkbox-ng/plainbox/impl/session/test_remote_assistant.py b/checkbox-ng/plainbox/impl/session/test_remote_assistant.py
@@ -250,9 +250,9 @@ def test_resume_by_id_with_result_no_file_noreturn(
             {
                 "outcome": IJobResult.OUTCOME_PASS,
                 "comments": (
-                    "Job crashed the machine or the Checkbox agent, "
-                    "automatically marking it as passed after resuming "
-                    "the session because it has the noreturn flag"
+                    "Job rebooted the machine or the Checkbox agent. "
+                    "Resuming the session and marking it as passed "
+                    "because the job has the `noreturn` flag"
                 ),
             }
         )
@@ -286,9 +286,8 @@ def test_resume_by_id_with_result_no_file_normal(self, mock_load_configs):
             {
                 "outcome": IJobResult.OUTCOME_CRASH,
                 "comments": (
-                    "Job crashed the machine or the Checkbox agent, "
-                    "automatically marking it as crashed after resuming "
-                    "the session"
+                    "Job rebooted the machine or the Checkbox agent. "
+                    "Resuming the session and marking it as crashed."
                 ),
             }
         )