Fix e2e tests: switch to new_ prefix, address PR review comments

- Switch all table/column/relationship names from test_ to new_ prefix
  (follows Dataverse convention for custom entity publisher prefix)
- Use PascalCase nav prop for @odata.bind and \ (case-sensitive)
- Use lowercase attribute logical name for _xxx_value in \/\
- Replace time.sleep() with _wait_for_lookup_ready() polling helper
- Replace cascade delete time.sleep() with retry polling for 404
- Add pytestmark = pytest.mark.e2e and register marker in pyproject.toml
- Close DataverseClient in fixture teardown
- Read DATAVERSE_URL lazily for late-set env vars

Validated: all 30 .scratch/ e2e tests pass (25 original + 5 fixed)
Validated: all 11 repo e2e tests collected, 398 unit tests pass

Author: Saurabh Badenkal

examples/basic/functional_testing.py

@@ -432,7 +432,8 @@ def test_relationships(client: DataverseClient) -> None:
 
     try:
         # --- Cleanup any leftover resources from previous run ---
-        print("Cleaning up previous relationship test resources...")
+        print("Checking for leftover relationship test resources...")
+        found_leftovers = False
         for rel_name in [
             "test_RelParent_RelChild",
             "contact_test_relchild_test_ManagerId",
@@ -441,18 +442,45 @@ def test_relationships(client: DataverseClient) -> None:
             try:
                 rel = client.tables.get_relationship(rel_name)
                 if rel:
-                    client.tables.delete_relationship(rel.relationship_id)
-                    print(f"   (Cleaned up relationship: {rel_name})")
+                    found_leftovers = True
+                    break
             except Exception:
                 pass
 
-        for tbl in [rel_child_schema, rel_parent_schema, rel_m2m_schema]:
-            try:
-                if client.tables.get(tbl):
-                    client.tables.delete(tbl)
-                    print(f"   (Cleaned up table: {tbl})")
-            except Exception:
-                pass
+        if not found_leftovers:
+            for tbl in [rel_child_schema, rel_parent_schema, rel_m2m_schema]:
+                try:
+                    if client.tables.get(tbl):
+                        found_leftovers = True
+                        break
+                except Exception:
+                    pass
+
+        if found_leftovers:
+            cleanup_ok = input("Found leftover test resources. Clean up? (y/N): ").strip().lower() in ["y", "yes"]
+            if cleanup_ok:
+                for rel_name in [
+                    "test_RelParent_RelChild",
+                    "contact_test_relchild_test_ManagerId",
+                    "test_relchild_relproject",
+                ]:
+                    try:
+                        rel = client.tables.get_relationship(rel_name)
+                        if rel:
+                            client.tables.delete_relationship(rel.relationship_id)
+                            print(f"   (Cleaned up relationship: {rel_name})")
+                    except Exception:
+                        pass
+
+                for tbl in [rel_child_schema, rel_parent_schema, rel_m2m_schema]:
+                    try:
+                        if client.tables.get(tbl):
+                            client.tables.delete(tbl)
+                            print(f"   (Cleaned up table: {tbl})")
+                    except Exception:
+                        pass
+            else:
+                print("Skipping cleanup -- resources may conflict with new test run.")
 
         # --- Create parent and child tables ---
         print("\nCreating relationship test tables...")

pyproject.toml

@@ -93,5 +93,8 @@ select = [
 
 [tool.pytest.ini_options]
 testpaths = ["tests/unit"]
+markers = [
+    "e2e: end-to-end tests requiring a live Dataverse environment (DATAVERSE_URL)",
+]
 # e2e tests require a live Dataverse environment:
 #   DATAVERSE_URL=https://yourorg.crm.dynamics.com pytest tests/e2e/ -v -s