Add unit tests for EnterpriseMigrationsClient

Octokit.Tests/Clients/Enterprise/EnterpriseMigrationClientTests.cs

@@ -0,0 +1,186 @@
+using System;
+using System.Collections.Generic;
+using NSubstitute;
+using Xunit;
+
+namespace Octokit.Tests.Clients
+{
+    public class EnterpriseMigrationClientTests
+    {
+        public class TheGetMethod
+        {
+            [Fact]
+            public void RequestsCorrectUrl()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new EnterpriseMigrationsClient(connection);
+
+                client.GetStatus("fake", 69);
+
+                connection.Received().Get<Migration>(
+                    Arg.Is<Uri>(u => u.ToString() == "orgs/fake/migrations/69"));
+            }
+
+            [Fact]
+            public async void EnsuresNonNullAndNonEmptyArguments()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new EnterpriseMigrationsClient(connection);
+
+                await Assert.ThrowsAsync<ArgumentNullException>(() => client.GetStatus(null, 69));
+                await Assert.ThrowsAsync<ArgumentException>(() => client.GetStatus("", 69));
+            }
+        }
+
+        public class TheGetAllMethod
+        {
+            [Fact]
+            public void RequestsCorrectUrl()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new EnterpriseMigrationsClient(connection);
+
+                client.GetMigrations("fake");
+
+                connection.Received().Get<List<Migration>>(
+                    Arg.Is<Uri>(u => u.ToString() == "orgs/fake/migrations"));
+            }
+
+            [Fact]
+            public async void EnsuresNonNullAndNonEmptyArguments()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new EnterpriseMigrationsClient(connection);
+
+                await Assert.ThrowsAsync<ArgumentNullException>(() => client.GetMigrations(null));
+                await Assert.ThrowsAsync<ArgumentException>(() => client.GetMigrations(""));
+            }
+        }
+
+        public class TheStartNewMethod
+        {
+            [Fact]
+            public void RequestsCorrectUrl()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new EnterpriseMigrationsClient(connection);
+                var migrationRequest = new StartMigrationRequest(new List<string> { "fake/repo" });
+
+                client.Start("fake", migrationRequest);
+
+                connection.Received().Post<Migration>(
+                    Arg.Is<Uri>(u => u.ToString() == "orgs/fake/migrations"),
+                    migrationRequest);
+            }
+
+            [Fact]
+            public async void EnsuresNonNullAndNonEmptyArguments()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new EnterpriseMigrationsClient(connection);
+                var migrationRequest = new StartMigrationRequest(new List<string> { "fake/repo" });
+
+                await Assert.ThrowsAsync<ArgumentNullException>(
+                    () => client.Start(null, migrationRequest));
+                await Assert.ThrowsAsync<ArgumentException>(
+                    () => client.Start("", migrationRequest));
+                await Assert.ThrowsAsync<ArgumentNullException>(
+                    () => client.Start("fake", null));
+            }
+
+            [Fact]
+            public void PassesRequestBody()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new EnterpriseMigrationsClient(connection);
+                var migrationRequest = new StartMigrationRequest(new List<string> { "fake/repo" });
+
+                client.Start("fake", migrationRequest);
+
+                connection.Received().Post<Migration>(
+                    Arg.Any<Uri>(),
+                    Arg.Is<StartMigrationRequest>(m =>
+                        m.Repositories.Equals(migrationRequest.Repositories) &&
+                        m.LockRepositories == migrationRequest.LockRepositories &&
+                        m.ExcludeAttachments == migrationRequest.ExcludeAttachments));
+            }
+        }
+
+        public class TheGetArchiveMethod
+        {
+            [Fact]
+            public void RequestsCorrectUrl()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new EnterpriseMigrationsClient(connection);
+
+                client.GetArchive("fake", 69);
+
+                connection.Received().Get<string>(
+                    Arg.Is<Uri>(u => u.ToString() == "orgs/fake/migrations/69/archive"));
+            }
+
+            [Fact]
+            public async void EnsuresNonNullAndNonEmptyArguments()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new EnterpriseMigrationsClient(connection);
+
+                await Assert.ThrowsAsync<ArgumentNullException>(() => client.GetArchive(null, 69));
+                await Assert.ThrowsAsync<ArgumentException>(() => client.GetArchive("", 69));
+            }
+        }
+
+        public class TheDeleteArchiveMethod
+        {
+            [Fact]
+            public void RequestsCorrectUrl()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new EnterpriseMigrationsClient(connection);
+
+                client.DeleteArchive("fake", 69);
+
+                connection.Received().Delete(
+                    Arg.Is<Uri>(u => u.ToString() == "orgs/fake/migrations/69/archive"));
+            }
+
+            [Fact]
+            public async void EnsuresNonNullAndNonEmptyArguments()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new EnterpriseMigrationsClient(connection);
+
+                await Assert.ThrowsAsync<ArgumentNullException>(() => client.DeleteArchive(null, 69));
+                await Assert.ThrowsAsync<ArgumentException>(() => client.DeleteArchive("", 69));
+            }
+        }
+
+        public class TheUnlockRepositoryMethod
+        {
+            [Fact]
+            public void RequestsCorrectUrl()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new EnterpriseMigrationsClient(connection);
+
+                client.UnlockRepository("fake", 69, "repo");
+
+                connection.Received().Delete(
+                    Arg.Is<Uri>(u => u.ToString() == "orgs/fake/migrations/69/repos/repo/lock"));
+            }
+
+            [Fact]
+            public async void EnsuresNonNullAndNonEmptyArguments()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new EnterpriseMigrationsClient(connection);
+
+                await Assert.ThrowsAsync<ArgumentNullException>(() => client.UnlockRepository(null, 69, "repo"));
+                await Assert.ThrowsAsync<ArgumentException>(() => client.UnlockRepository("", 69, "repo"));
+                await Assert.ThrowsAsync<ArgumentNullException>(() => client.UnlockRepository("fake", 69, null));
+                await Assert.ThrowsAsync<ArgumentException>(() => client.UnlockRepository("fake", 69, ""));
+            }
+        }
+    }
+}

Octokit.Tests/Models/MigrationTests.cs

@@ -137,4 +137,32 @@ namespace Octokit.Tests.Models
             Assert.Equal(1296269, _migration.Repositories[0].Id);
         }
     }
-}
+
+    public class StartMigrationTests
+    {
+        const string migrationRequestJson = @"{
+  ""repositories"": [
+    ""octocat/Hello-World""
+  ],
+  ""lock_repositories"": true
+}";
+
+        private static readonly StartMigrationRequest migrationRequest = new StartMigrationRequest(new List<string>
+        {
+            "octocat/Hello-World"
+        }, true);
+
+        [Fact]
+        public void CanBeDeserialized()
+        {
+            var serializer = new SimpleJsonSerializer();
+
+            var _migrationReuqest = serializer.Deserialize<StartMigrationRequest>(migrationRequestJson);
+
+            Assert.Equal("octokit/Hello-World", _migrationReuqest.Repositories[0]);
+            Assert.Equal(1, _migrationReuqest.Repositories.Count);
+            Assert.Equal(true, _migrationReuqest.LockRepositories);
+        }
+
+    }
+}

Octokit.Tests/Octokit.Tests.csproj

@@ -87,6 +87,7 @@
     <Compile Include="Authentication\CredentialsTests.cs" />
     <Compile Include="Clients\Enterprise\EnterpriseAdminStatsClientTests.cs" />
     <Compile Include="Clients\Enterprise\EnterpriseLdapClientTests.cs" />
+    <Compile Include="Clients\Enterprise\EnterpriseMigrationClientTests.cs" />
     <Compile Include="Clients\Enterprise\EnterpriseOrganizationClientTests.cs" />
     <Compile Include="Clients\Enterprise\EnterpriseLicenseClientTests.cs" />
     <Compile Include="Clients\Enterprise\EnterpriseSearchIndexingClientTests.cs" />