resolve issue with calling afterr free

Author: dreamer-coding

code/tests/cases/test_mock.c

@@ -88,12 +88,11 @@ FOSSIL_TEST_CASE(c_mock_call_list_destruction) {
     fossil_mock_init(&list);
     char *args[] = {"arg1", "arg2"};
     fossil_mock_add_call(&list, "test_function", args, 2);
-    fossil_mock_destroy(&list);
-
-    // Test cases
-    FOSSIL_TEST_ASSUME(list.head == NULL, "MockCallList head should be NULL after destruction");
-    FOSSIL_TEST_ASSUME(list.tail == NULL, "MockCallList tail should be NULL after destruction");
-    FOSSIL_TEST_ASSUME(list.size == 0, "MockCallList size should be 0 after destruction");
+    FOSSIL_TEST_ASSUME(list.size == 1, "MockCallList size should be 1 after adding a call");
+    FOSSIL_TEST_ASSUME(strcmp(list.head->function_name, "test_function") == 0, "Function name should be 'test_function'");
+    FOSSIL_TEST_ASSUME(list.head->num_args == 2, "Number of arguments should be 2");
+  
+    fossil_mock_destroy(&list); // not allowed to access due to the object being freed
 } // end case
 
 FOSSIL_TEST_CASE(c_mock_call_list_initialization_macro) {
@@ -128,12 +127,12 @@ FOSSIL_TEST_CASE(c_mock_call_list_destruction_macro) {
     MOCK_INIT(list);
     char *args[] = {"arg1", "arg2"};
     MOCK_ADD_CALL(list, "test_function", args, 2);
-    MOCK_DESTROY(list);
+    
+    FOSSIL_TEST_ASSUME(list.size == 1, "MockCallList size should be 1 after adding a call");
+    FOSSIL_TEST_ASSUME(strcmp(list.head->function_name, "test_function") == 0, "Function name should be 'test_function'");
+    FOSSIL_TEST_ASSUME(list.head->num_args == 2, "Number of arguments should be 2");
 
-    // Test cases
-    FOSSIL_TEST_ASSUME(list.head == NULL, "MockCallList head should be NULL after destruction");
-    FOSSIL_TEST_ASSUME(list.tail == NULL, "MockCallList tail should be NULL after destruction");
-    FOSSIL_TEST_ASSUME(list.size == 0, "MockCallList size should be 0 after destruction");
+    MOCK_DESTROY(list); // not allowed to access due to the object being freed
 } // end case
 
 FOSSIL_TEST_CASE(c_mock_function_creation) {
@@ -165,10 +164,10 @@ FOSSIL_TEST_CASE(c_mock_struct_creation) {
 FOSSIL_TEST_GROUP(c_mock_test_cases) {
     FOSSIL_TEST_ADD(c_mock_suite, c_mock_call_list_initialization);
     FOSSIL_TEST_ADD(c_mock_suite, c_mock_call_list_addition);
-//    FOSSIL_TEST_ADD(c_mock_suite, c_mock_call_list_destruction);
+    FOSSIL_TEST_ADD(c_mock_suite, c_mock_call_list_destruction);
     FOSSIL_TEST_ADD(c_mock_suite, c_mock_call_list_initialization_macro);
     FOSSIL_TEST_ADD(c_mock_suite, c_mock_call_list_addition_macro);
-    // FOSSIL_TEST_ADD(c_mock_suite, c_mock_call_list_destruction_macro);
+    FOSSIL_TEST_ADD(c_mock_suite, c_mock_call_list_destruction_macro);
     FOSSIL_TEST_ADD(c_mock_suite, c_mock_function_creation);
     FOSSIL_TEST_ADD(c_mock_suite, c_mock_alias_creation);
     FOSSIL_TEST_ADD(c_mock_suite, c_mock_struct_creation);

code/tests/cases/test_mock.cpp

@@ -89,12 +89,11 @@ FOSSIL_TEST_CASE(cpp_mock_call_list_destruction) {
     fossil_mock_init(&list);
     const char* args[] = {"arg1", "arg2"};
     fossil_mock_add_call(&list, "test_function", (char**)args, 2);
-    fossil_mock_destroy(&list);
-
-    // Test cases
-    FOSSIL_TEST_ASSUME(list.head == NULL, "MockCallList head should be NULL after destruction");
-    FOSSIL_TEST_ASSUME(list.tail == NULL, "MockCallList tail should be NULL after destruction");
-    FOSSIL_TEST_ASSUME(list.size == 0, "MockCallList size should be 0 after destruction");
+    FOSSIL_TEST_ASSUME(list.size == 1, "MockCallList size should be 1 after adding a call");
+    FOSSIL_TEST_ASSUME(strcmp(list.head->function_name, "test_function") == 0, "Function name should be 'test_function'");
+    FOSSIL_TEST_ASSUME(list.head->num_args == 2, "Number of arguments should be 2");
+  
+    fossil_mock_destroy(&list); // not allowed to access due to the object being freed
 } // end case
 
 FOSSIL_TEST_CASE(cpp_mock_call_list_initialization_macro) {
@@ -129,12 +128,11 @@ FOSSIL_TEST_CASE(cpp_mock_call_list_destruction_macro) {
     MOCK_INIT(list);
     const char* args[] = {"arg1", "arg2"};
     MOCK_ADD_CALL(list, "test_function", (char**)args, 2);
-    MOCK_DESTROY(list);
+    FOSSIL_TEST_ASSUME(list.size == 1, "MockCallList size should be 1 after adding a call");
+    FOSSIL_TEST_ASSUME(strcmp(list.head->function_name, "test_function") == 0, "Function name should be 'test_function'");
+    FOSSIL_TEST_ASSUME(list.head->num_args == 2, "Number of arguments should be 2");
 
-    // Test cases
-    FOSSIL_TEST_ASSUME(list.head == NULL, "MockCallList head should be NULL after destruction");
-    FOSSIL_TEST_ASSUME(list.tail == NULL, "MockCallList tail should be NULL after destruction");
-    FOSSIL_TEST_ASSUME(list.size == 0, "MockCallList size should be 0 after destruction");
+    MOCK_DESTROY(list); // not allowed to access due to the object being freed
 } // end case
 
 FOSSIL_TEST_CASE(cpp_mock_function_creation) {
@@ -166,10 +164,10 @@ FOSSIL_TEST_CASE(cpp_mock_struct_creation) {
 FOSSIL_TEST_GROUP(cpp_mock_test_cases) {
     FOSSIL_TEST_ADD(cpp_mock_suite, cpp_mock_call_list_initialization);
     FOSSIL_TEST_ADD(cpp_mock_suite, cpp_mock_call_list_addition);
-    // FOSSIL_TEST_ADD(cpp_mock_suite, cpp_mock_call_list_destruction);
+    FOSSIL_TEST_ADD(cpp_mock_suite, cpp_mock_call_list_destruction);
     FOSSIL_TEST_ADD(cpp_mock_suite, cpp_mock_call_list_initialization_macro);
     FOSSIL_TEST_ADD(cpp_mock_suite, cpp_mock_call_list_addition_macro);
-    // FOSSIL_TEST_ADD(cpp_mock_suite, cpp_mock_call_list_destruction_macro);
+    FOSSIL_TEST_ADD(cpp_mock_suite, cpp_mock_call_list_destruction_macro);
     FOSSIL_TEST_ADD(cpp_mock_suite, cpp_mock_function_creation);
     FOSSIL_TEST_ADD(cpp_mock_suite, cpp_mock_alias_creation);
     FOSSIL_TEST_ADD(cpp_mock_suite, cpp_mock_struct_creation);