Skip to content

Commit 62aaa89

Browse files
Abel Milashclaude
andcommitted
Add tests for display_name validation error paths in _create_table and _build_create_entity
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent bcc7b75 commit 62aaa89

1 file changed

Lines changed: 33 additions & 0 deletions

File tree

tests/unit/data/test_odata_internal.py

Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1840,6 +1840,24 @@ def test_display_name_defaults_to_schema_name(self):
18401840
label_value = post_json["DisplayName"]["LocalizedLabels"][0]["Label"]
18411841
self.assertEqual(label_value, "new_TestTable")
18421842

1843+
def test_display_name_empty_string_raises(self):
1844+
"""_create_table raises TypeError when display_name is an empty string."""
1845+
self._setup_for_create()
1846+
with self.assertRaises(TypeError):
1847+
self.od._create_table("new_TestTable", {}, display_name="")
1848+
1849+
def test_display_name_whitespace_raises(self):
1850+
"""_create_table raises TypeError when display_name is whitespace only."""
1851+
self._setup_for_create()
1852+
with self.assertRaises(TypeError):
1853+
self.od._create_table("new_TestTable", {}, display_name=" ")
1854+
1855+
def test_display_name_non_string_raises(self):
1856+
"""_create_table raises TypeError when display_name is not a string."""
1857+
self._setup_for_create()
1858+
with self.assertRaises(TypeError):
1859+
self.od._create_table("new_TestTable", {}, display_name=123)
1860+
18431861

18441862
class TestCreateColumns(unittest.TestCase):
18451863
"""Unit tests for _ODataClient._create_columns."""
@@ -2854,6 +2872,21 @@ def test_display_collection_name_derived_from_display_name(self):
28542872
body = self._body(display_name="Test Table")
28552873
self.assertEqual(body["DisplayCollectionName"]["LocalizedLabels"][0]["Label"], "Test Tables")
28562874

2875+
def test_display_name_empty_string_raises(self):
2876+
"""_build_create_entity raises TypeError when display_name is an empty string."""
2877+
with self.assertRaises(TypeError):
2878+
self.od._build_create_entity("new_TestTable", {}, display_name="")
2879+
2880+
def test_display_name_whitespace_raises(self):
2881+
"""_build_create_entity raises TypeError when display_name is whitespace only."""
2882+
with self.assertRaises(TypeError):
2883+
self.od._build_create_entity("new_TestTable", {}, display_name=" ")
2884+
2885+
def test_display_name_non_string_raises(self):
2886+
"""_build_create_entity raises TypeError when display_name is not a string."""
2887+
with self.assertRaises(TypeError):
2888+
self.od._build_create_entity("new_TestTable", {}, display_name=123)
2889+
28572890

28582891
if __name__ == "__main__":
28592892
unittest.main()

0 commit comments

Comments
 (0)