feat: make status independent from uncaught error

`test_status` should not be based on the presence of uncaught exception. 
Instead, each recorder should determine its own method to define it. 
Also, if a track has not be explicitely ended, the test should be 
considered to have stop abruptly which amount to failure.

components/trace/appmap/index.mjs

@@ -77,7 +77,7 @@ export const compileTrace = (configuration, messages) => {
   const sources = [];
   const errors = [];
   const events = [];
-  let status = 0;
+  let status = 1;
   for (const message of messages) {
     const { type } = message;
     if (type === "start") {

components/trace/appmap/index.test.mjs

@@ -173,7 +173,7 @@ assertDeepEqual(
         recorder: { name: "process" },
         recording: undefined,
         git: undefined,
-        test_status: "failed",
+        test_status: "succeeded",
         exception: {
           class: "name",
           message: "message",

components/trace/appmap/metadata.mjs

@@ -74,10 +74,7 @@ const makeMapName = (map_name, file_name, main) => {
   return null;
 };
 
-const makeTestStatus = (errors, status) => {
-  const { length } = errors;
-  return length === 0 && status === 0 ? "succeeded" : "failed";
-};
+const makeTestStatus = (status) => (status === 0 ? "succeeded" : "failed");
 
 const makeRecorder = (recorder) => {
   assert(
@@ -133,7 +130,7 @@ export const compileMetadata = (
   recorder: makeRecorder(recorder),
   recording: makeRecording(recording) ?? undefined,
   git: makeGit(repository) ?? undefined,
-  test_status: makeTestStatus(errors, status) ?? undefined,
+  test_status: makeTestStatus(status) ?? undefined,
   exception: makeException(errors) ?? undefined,
 });
 /* c8 ignore stop */

components/trace/appmap/metadata.test.mjs

@@ -82,7 +82,7 @@ assertDeepEqual(
         message: "error-message",
       },
     ],
-    0,
+    1,
   ),
   {
     ...default_meta_data,