fix: add entity_type_name returning None to OneNote Section and Notebook — prevent @odata.type rejection by OneNote API

Author: vgrem

office365/onenote/notebooks/notebook.py

@@ -39,3 +39,7 @@ def section_groups(self):
                 ResourcePath("sectionGroups", self.resource_path),
             ),
         )
+
+    @property
+    def entity_type_name(self):
+        return None  # type: ignore

office365/onenote/sections/section.py

@@ -99,3 +99,7 @@ def parent_section_group(self) -> SectionGroup:
             "parentSectionGroup",
             SectionGroup(self.context, ResourcePath("parentSectionGroup", self.resource_path)),
         )
+
+    @property
+    def entity_type_name(self):
+        return None  # type: ignore

tests/onenote/test_page.py

@@ -60,10 +60,10 @@ def test_02_page_has_expected_properties(self):
         if not page:
             self.skipTest("No page created from previous test")
 
-        self.assertIsNotNone(page.get_property("title"))
-        self.assertIsNotNone(page.get_property("createdDateTime"))
+        self.assertIsNotNone(page.title)
+        self.assertIsNotNone(page.created_datetime)
         # links should contain parentNotebook and parentSection URLs
-        self.assertIsNotNone(page.get_property("links"))
+        self.assertIsNotNone(page.links)
 
     @requires_delegated(
         "Notes.Read",
@@ -94,7 +94,7 @@ def test_04_page_title_includes_html_title(self):
         if not page:
             self.skipTest("No page created from previous test")
 
-        title = page.get_property("title")
+        title = page.title
         if title:
             self.assertIsInstance(title, str)
             self.assertGreater(len(title), 0)

tests/onenote/test_section.py

@@ -50,13 +50,13 @@ def test_02_create_notebook_and_section(self):
         notebook_name = create_unique_name("SDK Test Notebook")
         notebook = self.client.me.onenote.notebooks.add(notebook_name).execute_query()
         self.assertIsNotNone(notebook.resource_path)
-        self.assertEqual(notebook.get_property("displayName"), notebook_name)
+        self.assertEqual(notebook.display_name, notebook_name)
         TestSection.target_notebook = notebook
 
         section_name = create_unique_name("SDK Test Section")
         section = notebook.sections.add(displayName=section_name).execute_query()
         self.assertIsNotNone(section.resource_path)
-        self.assertEqual(section.get_property("displayName"), section_name)
+        self.assertEqual(section.display_name, section_name)
         TestSection.target_section = section
 
     @requires_delegated(
@@ -72,7 +72,7 @@ def test_03_update_section_name(self):
 
         new_name = create_unique_name("SDK Updated Section")
         section.set_property("displayName", new_name).update().execute_query()
-        self.assertEqual(section.get_property("displayName"), new_name)
+        self.assertEqual(section.display_name, new_name)
 
     @requires_delegated(
         "Notes.Read",
@@ -87,9 +87,9 @@ def test_04_section_has_expected_properties(self):
         if not section:
             self.skipTest("No section created from previous test")
 
-        self.assertIsNotNone(section.get_property("displayName"))
-        self.assertIsNotNone(section.get_property("createdDateTime"))
-        self.assertIsNotNone(section.get_property("isDefault"))
+        self.assertIsNotNone(section.display_name)
+        self.assertIsNotNone(section.created_datetime)
+        self.assertIsNotNone(section.is_default)
 
     @requires_delegated(
         "Notes.Read",