Merge branch 'user/tpellissier/logical_name_prefix' of https://github.com/microsoft/PowerPlatform-DataverseClient-Python into user/tpellissier/logical_name_prefix

Author: tpellissier

examples/basic/quickstart.py

@@ -551,7 +551,7 @@ def run_paging_demo(label: str, *, top: Optional[int], page_size: Optional[int])
 
 # 6) Column metadata helpers: column create/delete
 print("Column metadata helpers (create/delete column):")
-scratch_column = f"scratch_{int(time.time())}"
+scratch_column = f"new_scratch_{int(time.time())}"
 column_payload = {scratch_column: "string"}
 try:
 	log_call(f"client.create_column('{logical}', {repr(column_payload)})")

src/dataverse_sdk/client.py

@@ -403,7 +403,7 @@ def create_table(
         """
         Create a simple custom table with specified columns.
 
-        :param logical_name: Table logical name with publisher prefix (e.g. ``"new_orders"``).
+        :param logical_name: Table logical name with publisher prefix (e.g. ``"new_sampleitem"``).
             Both table and column names must include the publisher prefix (default is ``"new_"``).
         :type logical_name: str
         :param schema: Dictionary mapping column logical names (with prefix) to their types.

src/dataverse_sdk/data/odata.py

@@ -711,7 +711,7 @@ def _get_entity_metadata(self, logical_name: str) -> TableMetadata:
         return metadata
 
     def _logical_to_schema_name(self, logical_name: str) -> str:
-        """Convert logical name (new_sampleitem) to SchemaName (new_SampleItem) for CREATE operations.
+        """Convert logical name (new_sampleitem) to SchemaName (new_Sampleitem) for CREATE operations.
         
         Use this ONLY when creating new entities where we control the SchemaName.
         For existing entities, use _get_entity_schema_name() to get the actual SchemaName from server.

tests/unit/data/test_naming_normalization.py

@@ -13,8 +13,7 @@
 from tests.unit.test_helpers import (
     TestableClient,
     MD_ACCOUNT,
-    MD_SAMPLE_ITEM,
-    make_entity_create_headers
+    MD_SAMPLE_ITEM
 )
 
 # ============================================================================
@@ -331,15 +330,14 @@ def test_get_entity_by_logical_normalization():
 
 def test_create_table_normalizes_logical_name():
     """Test that _create_table normalizes the logical name."""
-    guid = "11111111-2222-3333-4444-555555555555"
     responses = [
         (200, {}, {}),  # POST to create entity
         (200, {}, {"value": [MD_ENTITY_BY_LOGICAL]}),  # GET entity by logical with Consistency: Strong
     ]
     c = TestableClient(responses)
 
     # Create with mixed case - _create_table takes schema dict, not individual params
-    result = c._create_table(
+    c._create_table(
         logical_name="NEW_SAMPLEITEM",
         schema={"new_field1": "string"},  # Dict of column_name -> type
         solution_unique_name=None
@@ -518,7 +516,6 @@ def test_create_with_one_casing_crud_with_another():
     )
 
     # 3. GET entity set with different casings - first call populates cache
-    calls_before = len(c._http.calls)
     entity_set1 = c._entity_set_from_logical("New_Product")
     calls_after_first = len(c._http.calls)
 

tests/unit/test_helpers.py

@@ -73,7 +73,7 @@ def __init__(self, responses, org_url="https://org.example", config=None):
         super().__init__(DummyAuth(), org_url, config)
         self._http = DummyHTTPClient(responses)
 
-    def _convert_labels_to_ints(self, logical_name, record):
+    def _convert_labels_to_ints(self, logical_name, record):  # pragma: no cover - test shim
         """Test shim - no-op conversion for simplicity."""
         return record