Fix unit tests: add missing break; and better messages.

Author: richardash1981

src/modbus.c

@@ -685,6 +685,7 @@ static int check_confirmation(modbus_t *ctx, uint8_t *req, uint8_t *rsp, int rsp
         case MODBUS_FC_WRITE_FILE_RECORD:
             req_nb_value = (req[offset + 7] << 8) + req[offset + 8];
             rsp_nb_value = (rsp[offset + 7] << 8) + rsp[offset + 8];
+            break;
         case MODBUS_FC_WRITE_SINGLE_COIL:
         case MODBUS_FC_WRITE_SINGLE_REGISTER:
             /* address in request and response must be equal */

tests/unit-test-client.c

@@ -335,14 +335,14 @@ int main(int argc, char *argv[])
     rc = modbus_write_file_record(ctx, UT_FILE, UT_RECORD_ADDRESS,
                                   UT_REGISTERS_NB, UT_REGISTERS_TAB);
     printf("3/6 modbus_write_file_record: ");
-    ASSERT_TRUE(rc == UT_REGISTERS_NB, "FAILED (nb points %d)\n", rc);
+    ASSERT_TRUE(rc == UT_REGISTERS_NB, "FAILED (nb records written %d)\n", rc);
     /* Write four more records to file 1 on server, starting at record 5,
      * i.e. after previous records */
     nb_points = UT_RECORD_ADDRESS + UT_REGISTERS_NB;
     rc = modbus_write_file_record(ctx, UT_FILE, nb_points,
                                   UT_FILE_RECORD_NB, UT_FILE_TAB);
     printf("4/6 modbus_write_file_record: ");
-    ASSERT_TRUE(rc == UT_FILE_RECORD_NB, "FAILED (nb points %d)\n", rc);
+    ASSERT_TRUE(rc == UT_FILE_RECORD_NB, "FAILED (nb records written %d)\n", rc);
 
     /* Read back all 7 records and make sure the values are correct.
      * Allocate and initialize the memory to store the registers as we need more.