Document code to clean up

Author: hoijnet

terminusdb_client/tests/test_woql_query_builder.py

@@ -192,14 +192,27 @@ def test_vlist_with_mixed_items(self):
         for item in result:
             assert isinstance(item, dict)
 
+    @pytest.mark.skip(reason="This method is deprecated and dead code - it is never called anywhere in the codebase.")
     def test_data_value_list_with_mixed_items(self):
-        """Test _data_value_list with mixed item types."""
+        """Test _data_value_list with mixed item types.
+        
+        DEPRECATED: This method is dead code - it is never called anywhere in the
+        codebase. The similar method _value_list() is used instead throughout the
+        client. This test and the method will be removed in a future release.
+        
+        There was an issue that is now fixed; however, test is disabled due to
+        deprecation. To be cleaned up once confirmed.
+        """
         query = WOQLQuery()
         items = ["string", "42", True, None]
-        # BLOCKED: Bug in line 355 of woql_query.py - it calls clean_data_value instead of _clean_data_value
-        # This test will fail until the bug is fixed in the main codebase
-        with pytest.raises(AttributeError):
-            query._data_value_list(items)
+        
+        result = query._data_value_list(items)
+        
+        assert isinstance(result, list)
+        assert len(result) == 4
+        for item in result:
+            assert isinstance(item, dict)
+            assert "@type" in item
 
     def test_clean_subject_with_dict(self):
         """Test _clean_subject with dictionary input."""