Refactor semantic error handling and add tests for validation

[AsafMah]

Fixes #557

[Copilot]

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

[github-actions]

Test Results

    5 files  ± 0      5 suites  ±0   16m 33s ⏱️ + 2m 45s
  296 tests + 2    262 ✅ + 2   34 💤 ±0  0 ❌ ±0 
1 480 runs  +10  1 310 ✅ +10  170 💤 ±0  0 ❌ ±0 

Results for commit 2c06bc7. ± Comparison against base commit 6bbba52.

♻️ This comment has been updated with latest results.

[Copilot]

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (4)

azure-kusto-data/tests/test_e2e_data.py:390

• The test_semantic_error method should cover more cases, such as different types of semantic errors or different responses.

def test_semantic_error(self):

azure-kusto-data/tests/test_e2e_data.py:397

• The test_semantic_error_async method should cover more cases, such as different types of semantic errors or different responses.

async def test_semantic_error_async(self):

azure-kusto-data/azure/kusto/data/exceptions.py:30

• Ensure that the string 'Semantic error:' is consistently replaced with the constant SEMANTIC_ERROR_STRING throughout the codebase.

SEMANTIC_ERROR_STRING = "Semantic error:"

azure-kusto-data/azure/kusto/data/exceptions.py:46

• Add a test case to ensure that self.message_text is correctly set and used in the is_semantic_error method.

self.message_text = messages if isinstance(messages, str) else "\n\n".join(repr(m) for m in messages)