#872 fix & tests updates

Author: vgrem

examples/onedrive/files/get_by_abs_url.py

@@ -10,10 +10,15 @@
     test_username,
 )
 
-file_abs_url = "{0}/Shared Documents/Financial Sample.xlsx".format(test_team_site_url)
+file_abs_url = "{0}/Shared Documents/Financial Sample.csv".format(test_team_site_url)
 
 client = GraphClient.with_username_and_password(
     test_tenant, test_client_id, test_username, test_password
 )
+
+# csv_file = client.me.drive.root.upload_file("../../data/Financial Sample.csv").execute_query()
+
 file_item = client.shares.by_url(file_abs_url).drive_item.get().execute_query()
-print(file_item.web_url)
+
+result = file_item.get_content().execute_query()
+print(result.value)

office365/onedrive/driveitems/driveItem.py

@@ -913,8 +913,6 @@ def subscriptions(self):
 
     def set_property(self, name, value, persist_changes=True):
         super(DriveItem, self).set_property(name, value, persist_changes)
-        if name == "parentReference":
-            pass
         return self
 
     def get_property(self, name, default_value=None):

office365/onedrive/shares/drive_item.py

@@ -8,7 +8,6 @@
 from office365.onedrive.permissions.permission import Permission
 from office365.onedrive.sites.site import Site
 from office365.runtime.paths.resource_path import ResourcePath
-from office365.runtime.paths.v4.entity import EntityPath
 
 
 class SharedDriveItem(BaseItem):
@@ -48,7 +47,7 @@ def drive_item(self):
         """Used to access the underlying driveItem"""
         return self.properties.get(
             "driveItem",
-            DriveItem(self.context, EntityPath("driveItem", self.resource_path)),
+            DriveItem(self.context, ResourcePath("driveItem", self.resource_path)),
         )
 
     @property

tests/onedrive/test_file.py

@@ -111,7 +111,7 @@ def test_14_copy_file(self):
     #    self.assertIsNotNone(result.value)
 
     def test_15_get_activities_by_interval(self):
-        end_time = datetime.utcnow()
+        end_time = datetime.now()
         start_time = end_time - timedelta(days=14)
         result = self.__class__.target_file.get_activities_by_interval(
             start_time, end_time, "day"

tests/onedrive/test_shares.py

@@ -0,0 +1,27 @@
+import os
+
+from tests import test_team_site_url
+from tests.graph_case import GraphTestCase
+
+
+class TestShares(GraphTestCase):
+    """Shares API specific test case"""
+
+    @classmethod
+    def setUpClass(cls):
+        super(TestShares, cls).setUpClass()
+        path = "{0}/../data/Financial Sample.xlsx".format(os.path.dirname(__file__))
+        cls.file_item = cls.client.sites.get_by_url(test_team_site_url).drive.root.upload_file(path).execute_query()
+        assert cls.file_item.resource_path is not None
+
+    @classmethod
+    def tearDownClass(cls):
+        cls.file_item.delete_object().execute_query_retry()
+
+    def test1_get_file_by_abs_url(self):
+        file_abs_url = "{0}/Shared Documents/Financial Sample.xlsx".format(
+            test_team_site_url
+        )
+        result = self.client.shares.by_url(file_abs_url).drive_item.get().execute_query()
+        self.assertIsNotNone(result.resource_path)
+        self.assertTrue(result.is_file)