Changed parameters to the Update method for organizations.

This commit is contained in:
Viktor Elofsson
2014-07-01 09:33:00 +02:00
parent 767685344b
commit 42e68d4c4a
3 changed files with 13 additions and 11 deletions
@@ -87,7 +87,7 @@ namespace Octokit.Tests.Clients
var client = Substitute.For<IApiConnection>();
var orgs = new OrganizationsClient(client);
orgs.Update(new OrganizationUpdate { Name = "initrode" });
orgs.Update("initrode", new OrganizationUpdate());
client.Received().Patch<Organization>(Arg.Is<Uri>(u => u.ToString() == "orgs/initrode"), Args.OrganizationUpdate);
}
+4 -3
View File
@@ -51,11 +51,12 @@ namespace Octokit
Task<IReadOnlyList<Organization>> GetAll(string user);
/// <summary>
/// Update the specified <see cref="OrganizationUpdate"/>.
/// Update the specified organization with data from <see cref="OrganizationUpdate"/>.
/// </summary>
/// <param name="organization"></param>
/// <param name="organizationName">The name of the organization to update.</param>
/// <param name="updateRequest"></param>
/// <exception cref="AuthorizationException">Thrown if the client is not authenticated.</exception>
/// <returns>A <see cref="Organization"/></returns>
Task<Organization> Update(OrganizationUpdate organization);
Task<Organization> Update(string organizationName, OrganizationUpdate updateRequest);
}
}
+8 -7
View File
@@ -71,19 +71,20 @@ namespace Octokit
}
/// <summary>
/// Update the specified <see cref="OrganizationUpdate"/>.
/// Update the specified organization with data from <see cref="OrganizationUpdate"/>.
/// </summary>
/// <param name="organization"></param>
/// <param name="organizationName">The name of the organization to update.</param>
/// <param name="updateRequest"></param>
/// <exception cref="AuthorizationException">Thrown if the client is not authenticated.</exception>
/// <returns>A <see cref="Organization"/></returns>
public Task<Organization> Update(OrganizationUpdate organization)
public Task<Organization> Update(string organizationName, OrganizationUpdate updateRequest)
{
Ensure.ArgumentNotNull(organization, "organization");
Ensure.ArgumentNotNullOrEmptyString(organization.Name, "organization");
Ensure.ArgumentNotNullOrEmptyString(organizationName, "organizationName");
Ensure.ArgumentNotNull(updateRequest, "updateRequest");
var updateUri = new Uri("orgs/" + organization.Name, UriKind.Relative);
var updateUri = new Uri("orgs/" + organizationName, UriKind.Relative);
return ApiConnection.Patch<Organization>(updateUri, organization);
return ApiConnection.Patch<Organization>(updateUri, updateRequest);
}
}
}