Code quality improvements: PEP 8 compliance and cleanup

- Rename single-letter test mock classes to descriptive names (PEP 8 compliance):
  * T -> MockToken (5 test files)
  * R -> MockResponse (1 test file)
- Remove unused 'import time' statements from example files
- Improves code readability and follows Python naming conventions
- All 48 tests continue to pass

Author: suyask-msft

examples/advanced/complete_walkthrough.py

@@ -423,7 +423,6 @@ def _status_value_for_index(idx: int, use_french: bool):
 # 4) Query records via SQL (?sql parameter))
 print("Query (SQL via ?sql query parameter):")
 try:
-	import time
 	pause("Execute SQL Query")
 
 	def _run_query():

examples/advanced/pandas_integration.py

@@ -201,8 +201,6 @@ def backoff_retry(op, *, delays=(0, 2, 5, 10, 20), retry_http_statuses=(400, 403
 # 4) Query records via SQL (Web API ?sql=)
 print("(Pandas) Query (SQL via Web API ?sql=):")
 try:
-	import time
-
 	def _run_query():
 		id_key = f"{logical}id"
 		cols = f"{id_key}, {attr_prefix}_code, {attr_prefix}_amount, {attr_prefix}_when"

tests/unit/core/test_http_errors.py

@@ -8,8 +8,8 @@
 
 class DummyAuth:
     def acquire_token(self, scope):
-        class T: access_token = "x"
-        return T()
+        class MockToken: access_token = "x"
+        return MockToken()
 
 class DummyHTTP:
     def __init__(self, responses):
@@ -18,9 +18,9 @@ def request(self, method, url, **kwargs):
         if not self._responses:
             raise AssertionError("No more responses")
         status, headers, body = self._responses.pop(0)
-        class R:
+        class MockResponse:
             pass
-        r = R()
+        r = MockResponse()
         r.status_code = status
         r.headers = headers
         if isinstance(body, dict):

tests/unit/data/test_enum_optionset_payload.py

@@ -8,9 +8,9 @@
 
 class DummyAuth:
     def acquire_token(self, scope):  # pragma: no cover - simple stub
-        class T:
+        class MockToken:
             access_token = "token"
-        return T()
+        return MockToken()
 
 class DummyConfig:
     """Minimal config stub providing attributes ODataClient.__init__ expects."""

tests/unit/data/test_logical_crud.py

@@ -8,8 +8,8 @@
 
 class DummyAuth:
     def acquire_token(self, scope):
-        class T: access_token = "x"
-        return T()
+        class MockToken: access_token = "x"
+        return MockToken()
 
 class DummyHTTPClient:
     def __init__(self, responses):

tests/unit/data/test_metadata_retry_logic.py

@@ -10,9 +10,9 @@
 
 class DummyAuth:
     def acquire_token(self, scope):
-        class T: 
+        class MockToken: 
             access_token = "test_token"
-        return T()
+        return MockToken()
 
 
 class TestMetadataRetryLogic:

tests/unit/data/test_sql_parse.py

@@ -6,8 +6,8 @@
 
 class DummyAuth:
     def acquire_token(self, scope):
-        class T: access_token = "x"  # no real token needed for parsing tests
-        return T()
+        class MockToken: access_token = "x"  # no real token needed for parsing tests
+        return MockToken()
 
 def _client():
     return ODataClient(DummyAuth(), "https://org.example", None)