From acc1ed08f5ada22c376c285edd1ca283a689ee38 Mon Sep 17 00:00:00 2001 From: Devesh Khandelwal Date: Mon, 14 Mar 2016 13:42:52 +0530 Subject: [PATCH] Check for empty string also in Migrations Client --- .../Enterprise/EnterpriseMigrationsClient.cs | 18 +++++++----------- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/Octokit/Clients/Enterprise/EnterpriseMigrationsClient.cs b/Octokit/Clients/Enterprise/EnterpriseMigrationsClient.cs index 40169c87..4b71b6f4 100644 --- a/Octokit/Clients/Enterprise/EnterpriseMigrationsClient.cs +++ b/Octokit/Clients/Enterprise/EnterpriseMigrationsClient.cs @@ -12,8 +12,7 @@ namespace Octokit public async Task GetStatus(string org, int id) { - Ensure.ArgumentNotNull(org, "org"); - Ensure.ArgumentNotNull(id, "id"); + Ensure.ArgumentNotNullOrEmptyString(org, "org"); var endpoint = ApiUrls.EnterpriseMigrationById(org, id); @@ -22,7 +21,7 @@ namespace Octokit public async Task> GetMigrations(string org) { - Ensure.ArgumentNotNull(org, "org"); + Ensure.ArgumentNotNullOrEmptyString(org, "org"); var endpoint = ApiUrls.EnterpriseMigrations(org); @@ -31,7 +30,7 @@ namespace Octokit public async Task Start(string org, StartMigrationRequest migration) { - Ensure.ArgumentNotNull(org, "org"); + Ensure.ArgumentNotNullOrEmptyString(org, "org"); Ensure.ArgumentNotNull(migration, "migration"); var endpoint = ApiUrls.EnterpriseMigrations(org); @@ -41,8 +40,7 @@ namespace Octokit public async Task GetArchive(string org, int id) { - Ensure.ArgumentNotNull(org, "org"); - Ensure.ArgumentNotNull(id, "id"); + Ensure.ArgumentNotNullOrEmptyString(org, "org"); var endpoint = ApiUrls.EnterpriseMigrationArchive(org, id); @@ -51,8 +49,7 @@ namespace Octokit public Task DeleteArchive(string org, int id) { - Ensure.ArgumentNotNull(org, "org"); - Ensure.ArgumentNotNull(id, "id"); + Ensure.ArgumentNotNullOrEmptyString(org, "org"); var endpoint = ApiUrls.EnterpriseMigrationArchive(org, id); @@ -61,9 +58,8 @@ namespace Octokit public Task UnlockRepository(string org, int id, string repo) { - Ensure.ArgumentNotNull(org, "org"); - Ensure.ArgumentNotNull(id, "id"); - Ensure.ArgumentNotNull(repo, "repo"); + Ensure.ArgumentNotNullOrEmptyString(org, "org"); + Ensure.ArgumentNotNullOrEmptyString(repo, "repo"); var endpoint = ApiUrls.EnterpriseMigrationUnlockRepository(org, id, repo);