Skip to content

Commit e0e29f3

Browse files
Abel Milashclaude
andcommitted
Address Copilot review comments: fix diacritic test input and add Cobertura tool to ADO coverage task
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent c208e5d commit e0e29f3

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

.azdo/ci-pr.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,5 +79,6 @@ extends:
7979
- task: PublishCodeCoverageResults@2
8080
condition: succeededOrFailed()
8181
inputs:
82+
codeCoverageTool: 'Cobertura'
8283
summaryFileLocation: '**/coverage.xml'
8384
displayName: 'Publish code coverage'

tests/unit/data/test_odata_internal.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1492,7 +1492,7 @@ def test_normalize_non_string_returns_empty_string(self):
14921492

14931493
def test_normalize_diacritics(self):
14941494
"""_normalize_picklist_label strips diacritics and lowercases."""
1495-
self.assertEqual(self.od._normalize_picklist_label("Actif"), "actif")
1495+
self.assertEqual(self.od._normalize_picklist_label("Åctíf"), "actif")
14961496

14971497
def test_empty_inputs_return_none(self):
14981498
"""_optionset_map returns None for empty table/attr inputs."""

0 commit comments

Comments
 (0)