Rename to _read so _write can be separate.

test/unit/Makefile

@@ -7,7 +7,7 @@ TEST_LDFLAGS=-lcunit -coverage
 
 all : test
 
-mosq_test : test.o datatypes.o utf8.o memory_mosq.o packet_datatypes.o utf8_mosq.o
+mosq_test : test.o datatype_read.o utf8.o memory_mosq.o packet_datatypes.o utf8_mosq.o
 	$(CROSS_COMPILE)$(CC) -o $@ $^ ${TEST_LDFLAGS}
 
 memory_mosq.o : ../../lib/memory_mosq.c
@@ -23,7 +23,7 @@ test : mosq_test
 	./mosq_test
 
 clean : 
-	-rm -f test *.o
+	-rm -rf mosq_test *.o *.gcda *.gcno coverage.info out/
 
 coverage :
 	lcov --capture --directory . --output-file coverage.info

test/unit/datatype_read.c

@@ -632,11 +632,11 @@ static void TEST_string_read_mqtt_1_5_4_3(void)
  * TEST SUITE SETUP
  * ======================================================================== */
 
-int init_datatype_tests(void)
+int init_datatype_read_tests(void)
 {
 	CU_pSuite test_suite = NULL;
 
-	test_suite = CU_add_suite("datatypes", NULL, NULL);
+	test_suite = CU_add_suite("Datatype read", NULL, NULL);
 	if(!test_suite){
 		printf("Error adding CUnit test suite.\n");
 		return 1;

test/unit/test.c

@@ -3,7 +3,7 @@
 #include <CUnit/CUnit.h>
 #include <CUnit/Basic.h>
 
-int init_datatype_tests(void);
+int init_datatype_read_tests(void);
 int init_utf8_tests(void);
 
 int main(int argc, char *argv[])
@@ -19,7 +19,7 @@ int main(int argc, char *argv[])
         return 1;
     }
 
-    if(init_datatype_tests()){
+    if(init_datatype_read_tests()){
         CU_cleanup_registry();
         return 1;
     }