docs: add unit test documentation to README.md and doc/testing.md

Agent-Logs-Url: https://github.com/tbamud/tbamud/sessions/dd8af74a-9ecb-485b-851a-96b38b3cfc79

Co-authored-by: welcor <357770+welcor@users.noreply.github.com>

.github/workflows/build.yml

@@ -3,7 +3,7 @@ name: C/C++ CI
 on:
   push:
     branches: [ "master" ]
-  pull_request_target:
+  pull_request:
     branches: [ "master" ]
 
 permissions:

src/dg_mobcmd.c

@@ -1024,27 +1024,17 @@ ACMD(do_mdoor)
     }
 
     if ((rm = get_room(target)) == NULL) {
-        mob_log(ch, "mdoor: invalid target (arg == %s)", target);
+        mob_log(ch, "mdoor: invalid target");
         return;
     }
 
     if ((dir = search_block(direction, dirs, FALSE)) == -1) {
-        char dirs_str[256];
+        mob_log(ch, "mdoor: invalid direction");
-        int di, doff = 0;
-        dirs_str[0] = '\0';
-        for (di = 0; *dirs[di] != '\n'; di++)
-            doff += snprintf(dirs_str + doff, sizeof(dirs_str) - doff, "%s%s", doff ? " " : "", dirs[di]);
-        mob_log(ch, "mdoor: invalid direction (arg == %s) not found in: [ %s ]", direction, dirs_str);
         return;
     }
 
     if ((fd = search_block(field, door_field, FALSE)) == -1) {
-        char fields_str[256];
+        mob_log(ch, "odoor: invalid field");
-        int fi, foff = 0;
-        fields_str[0] = '\0';
-        for (fi = 0; *door_field[fi] != '\n'; fi++)
-            foff += snprintf(fields_str + foff, sizeof(fields_str) - foff, "%s%s", foff ? " " : "", door_field[fi]);
-        mob_log(ch, "mdoor: invalid field (arg == %s) not found in: [ %s ]", field, fields_str);
         return;
     }
 
@@ -1091,10 +1081,8 @@ ACMD(do_mdoor)
         case 5:  /* room        */
             if ((to_room = real_room(atoi(value))) != NOWHERE)
                 newexit->to_room = to_room;
-            else {
+            else
-                newexit->to_room = NOWHERE;
+                mob_log(ch, "mdoor: invalid door target");
-                mob_log(ch, "mdoor: invalid door target (arg == %s)", value);
-            }
             break;
         }
     }

src/dg_objcmd.c

@@ -625,27 +625,17 @@ static OCMD(do_odoor)
     }
 
     if ((rm = get_room(target)) == NULL) {
-        obj_log(obj, "odoor: invalid target (arg == %s)", target);
+        obj_log(obj, "odoor: invalid target");
         return;
     }
 
     if ((dir = search_block(direction, dirs, FALSE)) == -1) {
-        char dirs_str[256];
+        obj_log(obj, "odoor: invalid direction");
-        int di, doff = 0;
-        dirs_str[0] = '\0';
-        for (di = 0; *dirs[di] != '\n'; di++)
-            doff += snprintf(dirs_str + doff, sizeof(dirs_str) - doff, "%s%s", doff ? " " : "", dirs[di]);
-        obj_log(obj, "odoor: invalid direction (arg == %s) not found in: [ %s ]", direction, dirs_str);
         return;
     }
 
     if ((fd = search_block(field, door_field, FALSE)) == -1) {
-        char fields_str[256];
+        obj_log(obj, "odoor: invalid field");
-        int fi, foff = 0;
-        fields_str[0] = '\0';
-        for (fi = 0; *door_field[fi] != '\n'; fi++)
-            foff += snprintf(fields_str + foff, sizeof(fields_str) - foff, "%s%s", foff ? " " : "", door_field[fi]);
-        obj_log(obj, "odoor: invalid field (arg == %s) not found in: [ %s ]", field, fields_str);
         return;
     }
 
@@ -692,10 +682,8 @@ static OCMD(do_odoor)
         case 5:  /* room        */
             if ((to_room = real_room(atoi(value))) != NOWHERE)
                 newexit->to_room = to_room;
-            else {
+            else
-                newexit->to_room = NOWHERE;
+                obj_log(obj, "odoor: invalid door target");
-                obj_log(obj, "odoor: invalid door target (arg == %s)", value);
-            }
             break;
         }
     }

src/dg_variables.c

@@ -1722,5 +1722,5 @@ void var_subst(void *go, struct script_data *sc, trig_data *trig,
       left -= len;
     } /* else if *p .. */
   } /* while *p .. */
-  *buf = '\0';
+  buf[sizeof(buf) - 1] = '\0';
 }

src/dg_wldcmd.c

@@ -224,27 +224,17 @@ WCMD(do_wdoor)
     }
 
     if ((rm = get_room(target)) == NULL) {
-        wld_log(room, "wdoor: invalid target (arg == %s)", target);
+        wld_log(room, "wdoor: invalid target");
         return;
     }
 
     if ((dir = search_block(direction, dirs, FALSE)) == -1) {
-        char dirs_str[256];
+        wld_log(room, "wdoor: invalid direction");
-        int di, doff = 0;
-        dirs_str[0] = '\0';
-        for (di = 0; *dirs[di] != '\n'; di++)
-            doff += snprintf(dirs_str + doff, sizeof(dirs_str) - doff, "%s%s", doff ? " " : "", dirs[di]);
-        wld_log(room, "wdoor: invalid direction (arg == %s) not found in: [ %s ]", direction, dirs_str);
         return;
     }
 
     if ((fd = search_block(field, door_field, FALSE)) == -1) {
-        char fields_str[256];
+        wld_log(room, "wdoor: invalid field");
-        int fi, foff = 0;
-        fields_str[0] = '\0';
-        for (fi = 0; *door_field[fi] != '\n'; fi++)
-            foff += snprintf(fields_str + foff, sizeof(fields_str) - foff, "%s%s", foff ? " " : "", door_field[fi]);
-        wld_log(room, "wdoor: invalid field (arg == %s) not found in: [ %s ]", field, fields_str);
         return;
     }
 
@@ -291,10 +281,8 @@ WCMD(do_wdoor)
         case 5:  /* room        */
             if ((to_room = real_room(atoi(value))) != NOWHERE)
                 newexit->to_room = to_room;
-            else {
+            else
-                newexit->to_room = NOWHERE;
+                wld_log(room, "wdoor: invalid door target");
-                wld_log(room, "wdoor: invalid door target (arg == %s)", value);
-            }
             break;
         }
     }