diff --git a/Octokit.Reactive/Clients/ObservableAuthorizationsClient.cs b/Octokit.Reactive/Clients/ObservableAuthorizationsClient.cs index 62589d8d..df061785 100644 --- a/Octokit.Reactive/Clients/ObservableAuthorizationsClient.cs +++ b/Octokit.Reactive/Clients/ObservableAuthorizationsClient.cs @@ -28,11 +28,15 @@ namespace Octokit.Reactive.Clients public IObservable Update(long id, AuthorizationUpdate authorization) { + Ensure.ArgumentNotNull(authorization, "authorization"); + return client.Update(id, authorization).ToObservable(); } public IObservable Create(AuthorizationUpdate authorization) { + Ensure.ArgumentNotNull(authorization, "authorization"); + return client.Create(authorization).ToObservable(); } diff --git a/Octokit.Reactive/Clients/ObservableOrganizationsClient.cs b/Octokit.Reactive/Clients/ObservableOrganizationsClient.cs index a556b06f..3c81cc64 100644 --- a/Octokit.Reactive/Clients/ObservableOrganizationsClient.cs +++ b/Octokit.Reactive/Clients/ObservableOrganizationsClient.cs @@ -17,6 +17,8 @@ namespace Octokit.Reactive.Clients public IObservable Get(string org) { + Ensure.ArgumentNotNullOrEmptyString(org, "org"); + return client.Get(org).ToObservable(); } @@ -27,6 +29,8 @@ namespace Octokit.Reactive.Clients public IObservable> GetAll(string user) { + Ensure.ArgumentNotNullOrEmptyString(user, "user"); + return client.GetAll(user).ToObservable(); } } diff --git a/Octokit.Reactive/Clients/ObservableRepositoriesClient.cs b/Octokit.Reactive/Clients/ObservableRepositoriesClient.cs index 5463f789..ffa04059 100644 --- a/Octokit.Reactive/Clients/ObservableRepositoriesClient.cs +++ b/Octokit.Reactive/Clients/ObservableRepositoriesClient.cs @@ -17,6 +17,9 @@ namespace Octokit.Reactive.Clients public IObservable Get(string owner, string name) { + Ensure.ArgumentNotNullOrEmptyString(owner, "owner"); + Ensure.ArgumentNotNullOrEmptyString(name, "name"); + return client.Get(owner, name).ToObservable(); } @@ -27,16 +30,23 @@ namespace Octokit.Reactive.Clients public IObservable> GetAllForUser(string login) { + Ensure.ArgumentNotNullOrEmptyString(login, "login"); + return client.GetAllForUser(login).ToObservable(); } public IObservable> GetAllForOrg(string organization) { + Ensure.ArgumentNotNullOrEmptyString(organization, "organization"); + return client.GetAllForOrg(organization).ToObservable(); } public IObservable GetReadme(string owner, string name) { + Ensure.ArgumentNotNullOrEmptyString(owner, "owner"); + Ensure.ArgumentNotNullOrEmptyString(name, "name"); + return client.GetReadme(owner, name).ToObservable(); } } diff --git a/Octokit.Reactive/Clients/ObservableSshKeysClient.cs b/Octokit.Reactive/Clients/ObservableSshKeysClient.cs index 3ca33f80..9127daaf 100644 --- a/Octokit.Reactive/Clients/ObservableSshKeysClient.cs +++ b/Octokit.Reactive/Clients/ObservableSshKeysClient.cs @@ -23,6 +23,8 @@ namespace Octokit.Reactive.Clients public IObservable> GetAll(string user) { + Ensure.ArgumentNotNullOrEmptyString(user, "user"); + return client.GetAll(user).ToObservable(); } @@ -33,11 +35,15 @@ namespace Octokit.Reactive.Clients public IObservable Create(SshKeyUpdate key) { + Ensure.ArgumentNotNull(key, "key"); + return client.Create(key).ToObservable(); } public IObservable Update(long id, SshKeyUpdate key) { + Ensure.ArgumentNotNull(key, "key"); + return client.Update(id, key).ToObservable(); } diff --git a/Octokit.Reactive/Clients/ObservableUsersClient.cs b/Octokit.Reactive/Clients/ObservableUsersClient.cs index 3325ca2b..cefb3dcb 100644 --- a/Octokit.Reactive/Clients/ObservableUsersClient.cs +++ b/Octokit.Reactive/Clients/ObservableUsersClient.cs @@ -16,6 +16,8 @@ namespace Octokit.Reactive.Clients public IObservable Get(string login) { + Ensure.ArgumentNotNull(login, "login"); + return client.Get(login).ToObservable(); } @@ -26,6 +28,8 @@ namespace Octokit.Reactive.Clients public IObservable Update(UserUpdate user) { + Ensure.ArgumentNotNull(user, "user"); + return client.Update(user).ToObservable(); } } diff --git a/Octokit.Reactive/ObservableExtensions.cs b/Octokit.Reactive/ObservableExtensions.cs index 45a3bd13..bc2e582a 100644 --- a/Octokit.Reactive/ObservableExtensions.cs +++ b/Octokit.Reactive/ObservableExtensions.cs @@ -10,6 +10,10 @@ namespace Octokit.Reactive string owner, string name) { + Ensure.ArgumentNotNull(client, "client"); + Ensure.ArgumentNotNullOrEmptyString(owner, "owner"); + Ensure.ArgumentNotNullOrEmptyString(name, "name"); + return client.GetReadme(owner, name).SelectMany(r => r.GetHtmlContent().ToObservable()); } } diff --git a/Octokit.Reactive/Octokit.Reactive.csproj b/Octokit.Reactive/Octokit.Reactive.csproj index bd5a1969..48f692c6 100644 --- a/Octokit.Reactive/Octokit.Reactive.csproj +++ b/Octokit.Reactive/Octokit.Reactive.csproj @@ -83,6 +83,11 @@ + + + CustomDictionary.xml + +