update Outlook types and tests

Author: vgrem

generator/import_metadata.py

@@ -26,13 +26,13 @@ def export_to_file(path, content):
     "--endpoint",
     dest="endpoint",
     help="Import metadata endpoint",
-    default="sharepoint",
+    default="graph",
 )
 parser.add_argument(
     "-p",
     "--path",
     dest="path",
-    default="./metadata/SharePoint.xml",
+    default="./metadata/Graph.xml",
     help="Import metadata endpoint",
 )
 

generator/metadata/Graph.xml

@@ -40138,6 +40138,10 @@ within the time frame of their original request."/>
             <Parameter Name="bindingParameter" Type="Collection(graph.riskyUser)"/>
             <Parameter Name="userIds" Type="Collection(Edm.String)" Unicode="false"/>
          </Action>
+         <Action Name="confirmSafe" IsBound="true">
+            <Parameter Name="bindingParameter" Type="Collection(graph.riskyUser)"/>
+            <Parameter Name="userIds" Type="Collection(Edm.String)" Unicode="false"/>
+         </Action>
          <Action Name="reprocess" IsBound="true">
             <Parameter Name="bindingParameter" Type="graph.accessPackageAssignmentRequest"/>
          </Action>
@@ -41381,6 +41385,7 @@ within the time frame of their original request."/>
          <Action Name="clockIn" IsBound="true">
             <Parameter Name="bindingParameter" Type="Collection(graph.timeCard)"/>
             <Parameter Name="isAtApprovedLocation" Type="Edm.Boolean"/>
+            <Parameter Name="onBehalfOfUserId" Type="Edm.String" Unicode="false"/>
             <Parameter Name="notes" Type="graph.itemBody"/>
             <ReturnType Type="graph.timeCard"/>
          </Action>

generator/metadata/SharePoint.xml

@@ -3,5 +3,17 @@
 
 
 class MailFolderCollection(DeltaCollection[MailFolder]):
+    """Mail folder's collection"""
+
     def __init__(self, context, resource_path=None):
         super(MailFolderCollection, self).__init__(context, MailFolder, resource_path)
+
+    def add(self, display_name, is_hidden=False):
+        # type: (str, bool) -> MailFolder
+        """
+        Use this API to create a new mail folder in the root folder of the user's mailbox.
+
+        If you intend a new folder to be hidden, you must set the isHidden property to true on creation.
+        """
+        props = {"displayName": display_name, "isHidden": is_hidden}
+        return super(MailFolderCollection, self).add(**props)

office365/outlook/mail/folders/collection.py

@@ -16,6 +16,9 @@ class MailFolder(Entity):
     """A mail folder in a user's mailbox, such as Inbox and Drafts. Mail folders can contain messages,
     other Outlook items, and child mail folders."""
 
+    def __str__(self):
+        return self.display_name or self.entity_type_name
+
     def copy(self, destination_id):
         """
         Copy a mailfolder and its contents to another mailfolder.
@@ -62,6 +65,12 @@ def _mark_all_items_as_unread(col):
         self.messages.get_all(page_loaded=_mark_all_items_as_unread)
         return self
 
+    def permanent_delete(self):
+        """Permanently delete a mail folder and remove its items from the user's mailbox."""
+        qry = ServiceOperationQuery(self, "permanentDelete")
+        self.context.add_query(qry)
+        return self
+
     @property
     def child_folder_count(self):
         # type: () -> Optional[int]

office365/outlook/mail/folders/folder.py

@@ -1,4 +1,7 @@
+from typing import Optional
+
 from office365.outlook.mail.folders.folder import MailFolder
+from office365.runtime.types.collections import StringCollection
 
 
 class MailSearchFolder(MailFolder):
@@ -9,3 +12,30 @@ class MailSearchFolder(MailFolder):
     to appear in Outlook, Outlook for the web, or Outlook Live, the folder must be created in the
     WellKnownFolderName.SearchFolders folder.
     """
+
+    @property
+    def filter_query(self):
+        # type: () -> Optional[str]
+        """The OData query to filter the messages."""
+        return self.properties.get("filterQuery", None)
+
+    @property
+    def include_nested_folders(self):
+        # type: () -> Optional[bool]
+        """Indicates how the mailbox folder hierarchy should be traversed in the search. true means that a deep
+        search should be done to include child folders in the hierarchy of each folder explicitly specified in
+        sourceFolderIds. false means a shallow search of only each of the folders explicitly specified
+        in sourceFolderIds."""
+        return self.properties.get("includeNestedFolders", None)
+
+    @property
+    def is_supported(self):
+        # type: () -> Optional[bool]
+        """Indicates whether a search folder is editable using REST APIs."""
+        return self.properties.get("isSupported", None)
+
+    @property
+    def source_folder_ids(self):
+        # type: () -> StringCollection
+        """The mailbox folders that should be mined."""
+        return self.properties.get("sourceFolderIds", StringCollection())

office365/outlook/mail/folders/search.py

@@ -34,7 +34,7 @@ def test3_ensure_created(self):
         result = self.__class__.business.get().execute_query_retry()
         self.assertIsNotNone(result.resource_path)
 
-    #def test4_get_staff_availability(self):
+    # def test4_get_staff_availability(self):
     #    result = self.__class__.business.get_staff_availability().execute_query()
     #    self.assertIsNotNone(result.resource_path)
 

tests/booking/test_business.py

@@ -16,7 +16,7 @@ def setUpClass(cls):
     def tearDownClass(cls):
         pass
 
-    #def test1_create_peer_to_peer_call(self):
+    # def test1_create_peer_to_peer_call(self):
     #    result = self.client.communications.calls.create("https://bot.mediadev8.com/callback").execute_query()
     #    self.assertIsNotNone(result.resource_path)
 

tests/communications/test_callrecords.py

@@ -30,16 +30,12 @@ def wrapper(self, *args, **kwargs):
             if not client:
                 self.skipTest("No client available for permission check")
 
-            try:
-                permissions = _get_cached_permissions(client, test_client_id)
+            permissions = _get_cached_permissions(client, test_client_id)
 
-                if not any(role.value == app_role for role in permissions):
-                    self.skipTest(f"Required app permission '{app_role}' not granted")
+            if not any(role.value == app_role for role in permissions):
+                self.skipTest(f"Required app permission '{app_role}' not granted")
 
-                return test_method(self, *args, **kwargs)
-
-            except Exception as e:
-                self.skipTest(f"Permission check failed: {str(e)}")
+            return test_method(self, *args, **kwargs)
 
         return wrapper
 
@@ -68,19 +64,15 @@ def wrapper(self, *args, **kwargs):
             if not client:
                 self.skipTest("No client available for permission check")
 
-            try:
-                # Get permissions from cache or API
-                granted_scopes = _get_cached_delegated_permissions(client, test_client_id)
-
-                if not any(scope in granted_scopes for scope in scopes):
-                    self.skipTest(
-                        f"Required delegated permission '{', '.join(scopes)}' not granted"
-                    )
+            # Get permissions from cache or API
+            granted_scopes = _get_cached_delegated_permissions(client, test_client_id)
 
-                return test_method(self, *args, **kwargs)
+            if not any(scope in granted_scopes for scope in scopes):
+                self.skipTest(
+                    f"Required delegated permission '{', '.join(scopes)}' not granted"
+                )
 
-            except Exception as e:
-                self.skipTest(f"Permission check failed: {str(e)}")
+            return test_method(self, *args, **kwargs)
 
         return wrapper  # type: ignore[return-value]
 

tests/decorators.py

@@ -1,5 +1,6 @@
 from office365.onenote.notebooks.notebook import Notebook
 from tests import create_unique_name
+from tests.decorators import requires_delegated_permission
 from tests.graph_case import GraphTestCase
 
 
@@ -14,6 +15,9 @@ def setUpClass(cls):
     def tearDownClass(cls):
         pass
 
+    @requires_delegated_permission(
+        "Notes.Create", "Notes.ReadWrite", "Notes.ReadWrite.All"
+    )
     def test1_create_notebook(self):
         notebook_name = create_unique_name("My Private notebook")
         new_notebook = self.client.me.onenote.notebooks.add(
@@ -22,14 +26,27 @@ def test1_create_notebook(self):
         self.assertIsNotNone(new_notebook.resource_path)
         self.__class__.target_notebook = new_notebook
 
+    @requires_delegated_permission("Notes.Create")
     def test2_list_notebooks(self):
         my_notebooks = self.client.me.onenote.notebooks.get().execute_query()
         self.assertIsNotNone(my_notebooks.resource_path)
 
+    @requires_delegated_permission(
+        "Notes.Create",
+        "Notes.Read",
+        "Notes.Read.All",
+        "Notes.ReadWrite",
+        "Notes.ReadWrite.All",
+    )
     def test3_get_recent_notebooks(self):
         result = self.client.me.onenote.notebooks.get_recent_notebooks().execute_query()
         self.assertIsNotNone(result.value)
 
+    @requires_delegated_permission(
+        "Notes.Create",
+        "Notes.ReadWrite",
+        "Notes.ReadWrite.All",
+    )
     def test4_create_section(self):
         name = create_unique_name("Section name")
         new_section = self.__class__.target_notebook.sections.add(