2.6.2 release

Author: vgrem

office365/onedrive/listitems/collection.py

@@ -8,7 +8,7 @@ class ListItemCollection(EntityCollection[ListItem]):
 
     def __init__(self, context, resource_path):
         super(ListItemCollection, self).__init__(context, ListItem, resource_path)
-        self._honor_nonindexed = False
+        self._honor_nonindexed = True
 
     def honor_nonindexed(self, value):
         # type: (bool) -> "ListItemCollection"
@@ -32,3 +32,7 @@ def _construct_request(request):
     def get_by_name(self, name):
         """Retrieve a list item by name"""
         return self.single("fields/FileLeafRef eq '{0}'".format(name))
+
+    def get_by_path(self, path):
+        """Retrieve a list item by path"""
+        return self.single("fields/FileRef eq '{0}'".format(path))

office365/onedrive/sites/site.py

@@ -26,7 +26,7 @@ class Site(BaseItem):
     """The site resource provides metadata and relationships for a SharePoint site."""
 
     def __str__(self):
-        return self.name
+        return self.name or self.entity_type_name
 
     def __repr__(self):
         return self.name or self.web_url or self.id or self.entity_type_name

office365/sharepoint/fields/builtin_field_id.py

@@ -10,6 +10,14 @@ class SPBuiltInFieldId:
     FSObjType = "FSObjType"
     """Identifies a field that contains information about the file system type"""
 
+    FileDirRef = "FileDirRef"
+    """Identifies a field that contains information about the file directory for the specified
+    SharePoint Foundation object."""
+
+    FileLeafRef = "FileLeafRef"
+    """Identifies a field that contains information about the server-relative URL for the file node that is
+    associated with the specified SharePoint Foundation object."""
+
     FileType = "FileType"
     """Identifies a field that contains information about the file type for version history of the specified
     SharePoint Foundation library picture object."""

office365/sharepoint/userprofiles/people_manager.py

@@ -148,12 +148,13 @@ def stop_following_tag(self, value):
         return self
 
     def get_user_profile_properties(self, user_or_name):
+        # type: (str|User) -> ClientResult[dict]
         """
         Gets the specified user profile properties for the specified user.
 
         :param str or User user_or_name: User or Login name of the specified user.
         """
-        return_type = ClientResult(self.context, dict())
+        return_type = ClientResult(self.context, {})
 
         def _user_resolved(account_name):
             # type: (str) -> None

setup.py

@@ -9,7 +9,7 @@
 
 setup(
     name="Office365-REST-Python-Client",
-    version="2.6.1",
+    version="2.6.2",
     author="Vadim Gremyachev",
     author_email="[email protected]",
     maintainer="Konrad Gądek, Domenico Di Nicola",

tests/onedrive/test_list.py

@@ -9,6 +9,7 @@ class TestList(GraphTestCase):
 
     target_list = None  # type: List
     target_column = None  # type: ColumnDefinition
+    list_name = create_unique_name("Documents")
 
     @classmethod
     def setUpClass(cls):
@@ -18,25 +19,33 @@ def setUpClass(cls):
     def tearDownClass(cls):
         pass
 
-    def test1_get_list(self):
-        target_list = self.client.sites.root.lists["Documents"].get().execute_query()
+    def test1_create_list(self):
+        result = self.client.sites.root.lists.add(
+            self.list_name, "documentLibrary"
+        ).execute_query()
+        self.__class__.target_list = result
+
+    def test2_get_list(self):
+        target_list = self.client.sites.root.lists[self.list_name].get().execute_query()
         self.assertIsNotNone(target_list.resource_path)
-        self.__class__.target_list = target_list
 
-    def test2_get_list_items(self):
+    def test3_get_list_items(self):
         items = self.target_list.items.get().execute_query()
         self.assertIsNotNone(items.resource_path)
 
-    def test3_get_list_columns(self):
+    def test4_get_list_columns(self):
         columns = self.target_list.columns.get().execute_query()
         self.assertIsNotNone(columns.resource_path)
 
-    def test4_create_list_column(self):
+    def test5_create_list_column(self):
         column_name = create_unique_name("Text")
         text_column = self.target_list.columns.add_text(column_name).execute_query()
         self.assertIsNotNone(text_column.resource_path)
         self.__class__.target_column = text_column
 
-    def test5_delete_list_column(self):
+    def test6_delete_list_column(self):
         column_to_del = self.__class__.target_column
         column_to_del.delete_object().execute_query()
+
+    def test7_delete_list(self):
+        self.__class__.target_list.delete_object().execute_query()

tests/onedrive/test_listitem.py

@@ -0,0 +1,31 @@
+from office365.onedrive.listitems.list_item import ListItem
+from office365.onedrive.lists.list import List
+from tests import test_team_site_url
+from tests.graph_case import GraphTestCase
+
+
+class TestListItem(GraphTestCase):
+    """"""
+
+    target_list = None  # type: List
+    target_item = None  # type: ListItem
+
+    @classmethod
+    def setUpClass(cls):
+        super(TestListItem, cls).setUpClass()
+        cls.test_site = cls.client.sites.get_by_url(test_team_site_url)
+        cls.pages_list = cls.test_site.lists.get_by_name("Site Pages")
+
+    @classmethod
+    def tearDownClass(cls):
+        pass
+
+    def test_1_get_item_by_name(self):
+        result = self.pages_list.items.get_by_name("Home.aspx").execute_query()
+        self.assertIsNotNone(result.resource_path)
+        self.__class__.target_item = result
+
+    def test_3_get_item_values(self):
+        item = self.__class__.target_item
+        result = item.fields.get().execute_query()
+        self.assertIsNotNone(result.resource_path)

tests/onedrive/test_sitepage.py

@@ -31,11 +31,7 @@ def test2_get_site_page(self):
     def test3_checkin_site_page(self):
         page_name = self.__class__.target_page.name
         pages_list = self.test_site.lists.get_by_name("Site Pages")
-        list_item = (
-            pages_list.items.honor_nonindexed(True)
-            .get_by_name(page_name)
-            .execute_query()
-        )
+        list_item = pages_list.items.get_by_name(page_name).execute_query()
         list_item.drive_item.checkin("Initial version").execute_query()
         self.assertIsNotNone(list_item.resource_path)
 
@@ -74,12 +70,3 @@ def test_10_delete_site_page(self):
     def test_11_get_site_page_list(self):
         result = self.test_site.lists.get_by_name("Site Pages").get().execute_query()
         self.assertIsNotNone(result.resource_path)
-
-    def test_12_get_site_page_item_by_name(self):
-        pages_list = self.test_site.lists.get_by_name("Site Pages")
-        result = (
-            pages_list.items.honor_nonindexed(True)
-            .get_by_name("Home.aspx")
-            .execute_query()
-        )
-        self.assertIsNotNone(result.resource_path)