Merge pull request #376 from microsoftgraph/enableRootSiteAccess

Fixes #374: Adds route to a Site's root structural property.

Author: MIchaelMainer

src/Microsoft.Graph/Requests/Extensions/GraphServiceSitesCollectionRequestBuilderExtension.cs

@@ -27,5 +27,16 @@ public ISiteRequestBuilder GetByPath(string siteRelativePath, string hostname)
                 string.Format("{0}/{1}:{2}", this.RequestUrl, hostname, siteRelativePath),
                 this.Client);
         }
+
+        /// <summary>
+        /// Gets a request builder for accessing a sites.
+        /// </summary>
+        public ISiteRequestBuilder Root
+        {
+            get
+            {
+                return new SiteRequestBuilder(this.AppendSegmentToRequestUrl("root"), this.Client);
+            }
+        }
     }
 }

src/Microsoft.Graph/Requests/Extensions/IGraphServiceSitesCollectionRequestBuilderExtension.cs

@@ -15,5 +15,10 @@ public partial interface IGraphServiceSitesCollectionRequestBuilder
         /// </summary>
         /// <returns>The <see cref="ISiteRequestBuilder"/>.</returns>
         ISiteRequestBuilder GetByPath(string siteRelativePath, string hostname);
+
+        /// <summary>
+        /// Gets a request builder for accessing a site's root. This is how we can provide a request builder for structural properties.
+        /// </summary>
+        ISiteRequestBuilder Root { get; }
     }
 }

tests/Microsoft.Graph.Test/Requests/Functional/OneNoteTests.cs

@@ -17,9 +17,7 @@ public class OneNoteTests : GraphTestBase
         private Notebook testNotebook;
         private static string firstSectionID;
 
-        [ClassInitialize]
-        public static void GetTestSectionId(TestContext c)
-        {
+        public OneNoteTests() : base() {
             // Get a page of OneNote sections.
             IOnenoteSectionsCollectionPage sectionPage = graphClient.Me
                                                                     .Onenote
@@ -31,7 +29,7 @@ public static void GetTestSectionId(TestContext c)
             // Get a handle to the first section.
             firstSectionID = sectionPage[0].Id;
         }
-
+        
         public async void TestPageCleanUp()
         {
             await graphClient.Me.Onenote.Pages[testPage.Id].Request().DeleteAsync();
@@ -731,5 +729,26 @@ public async Async.Task OneNoteAddPageWithMultipart()
                 Assert.Fail("Error code: {0}", e.Message);
             }
         }
+
+        /// <summary>
+        /// Test the custom 'Root' partial request builder and accessing Onenote notebook collection.
+        /// </summary>
+        /// <returns></returns>
+        [TestMethod]
+        public async Async.Task It_accesses_a_sites_OneNote_notebooks()
+        {
+            try
+            {
+                Site site = await graphClient.Sites.Root.Request().GetAsync();
+                Assert.IsNotNull(site);
+
+                IOnenoteNotebooksCollectionPage notebooks = await graphClient.Sites[site.Id].Onenote.Notebooks.Request().GetAsync();
+                Assert.IsNotNull(notebooks);
+            }
+            catch (Exception)
+            {
+                Assert.Fail("An unexpected exception was thrown. This test case failed.");
+            }
+        }
     }
 }

tests/Microsoft.Graph.Test/Requests/Functional/SharePointTests.cs

@@ -8,11 +8,11 @@ namespace Microsoft.Graph.Test.Requests.Functional
 {
     [Ignore]
     [TestClass]
-    public class SharePointTests : GraphTestBase
+    public class Given_a_valid_SharePoint_Site : GraphTestBase
     {
         // Test search a SharePoint site.
         [TestMethod]
-        public async Async.Task SharePointSearchSites()
+        public async Async.Task It_searches_the_SharePoint_Site_and_returns_results()
         {
             try
             {
@@ -34,9 +34,9 @@ public async Async.Task SharePointSearchSites()
             }
         }
 
-        // Test accessing the default document libraries for a SharePoint site.
+        // Test accessing the document libraries for a SharePoint site.
         [TestMethod]
-        public async Async.Task SharePointGetDocumentLibraries()
+        public async Async.Task It_gets_the_sites_drives()
         {
             try
             {
@@ -64,7 +64,7 @@ public async Async.Task SharePointGetDocumentLibraries()
 
         // Test accessing the non-default document library on a SharePoint site.
         [TestMethod]
-        public async Async.Task SharePointGetNonDefaultDocumentLibraries()
+        public async Async.Task It_gets_the_sites_drives_root_children()
         {
             try
             {
@@ -110,7 +110,7 @@ public async Async.Task SharePointGetNonDefaultDocumentLibraries()
         /// 
         [Ignore] // Need reset test data  in demo tenant
         [TestMethod]
-        public async Async.Task SharePointGetSiteWithPath()
+        public async Async.Task It_gets_a_site_by_path()
         {
             try
             {
@@ -156,16 +156,35 @@ public async Async.Task SharePointGetSiteWithPath()
                 Assert.Fail("Something happened, check out a trace. Error code: {0}", e.Error.Code);
             }
         }
-        
+
+        /// <summary>
+        /// Test the custom 'Root' partial request builder.
+        /// </summary>
+        /// <returns></returns>
+        [TestMethod]
+        public async Async.Task It_gets_the_root_site()
+        {
+            try
+            {
+                Site site = await graphClient.Sites.Root.Request().GetAsync();
+                Assert.IsNotNull(site);
+            }
+            catch (Exception)
+            {
+                Assert.Fail("An unexpected exception was thrown. This test case failed.");
+            }
+        }
+
         /// <summary>
         /// Test to get information about a SharePoint site by its URL.
         /// </summary>
         [TestMethod]
-        public async Async.Task SharePointAccessSiteByUrl()
+        public async Async.Task It_gets_a_site_by_URL()
         {
             try
             {
                 Site site = await graphClient.Shares[UrlToSharingToken("https://m365x462896.sharepoint.com/sites/portals2")].Site.Request().GetAsync();
+
                 Assert.IsNotNull(site);
             }
             catch (Microsoft.Graph.ServiceException e)