Add PreviousFileName to the PullRequestFile model (#1770)

* Add PreviousFileName to the PullRequestFile model

* fix white space issue
This commit is contained in:
Ryan Clare
2018-02-23 14:47:56 -06:00
committed by Ryan Gribble
parent c4774ffaf0
commit e430a9ef14
3 changed files with 14 additions and 11 deletions
@@ -870,10 +870,10 @@ public class PullRequestsClientTests : IDisposable
{
var expectedFiles = new List<PullRequestFile>
{
new PullRequestFile(null, "Octokit.Tests.Integration/Clients/ReferencesClientTests.cs", null, 8, 3, 11, null, null, null, null),
new PullRequestFile(null, "Octokit/Clients/ApiPagination.cs", null, 21, 6, 27, null, null, null, null),
new PullRequestFile(null, "Octokit/Helpers/IApiPagination.cs", null, 1, 1, 2, null, null, null, null),
new PullRequestFile(null, "Octokit/Http/ApiConnection.cs", null, 1, 1, 2, null, null, null, null)
new PullRequestFile(null, "Octokit.Tests.Integration/Clients/ReferencesClientTests.cs", null, 8, 3, 11, null, null, null, null, null),
new PullRequestFile(null, "Octokit/Clients/ApiPagination.cs", null, 21, 6, 27, null, null, null, null, null),
new PullRequestFile(null, "Octokit/Helpers/IApiPagination.cs", null, 1, 1, 2, null, null, null, null, null),
new PullRequestFile(null, "Octokit/Http/ApiConnection.cs", null, 1, 1, 2, null, null, null, null, null)
};
var result = await _fixture.Files("octokit", "octokit.net", 288);
@@ -894,10 +894,10 @@ public class PullRequestsClientTests : IDisposable
{
var expectedFiles = new List<PullRequestFile>
{
new PullRequestFile(null, "Octokit.Tests.Integration/Clients/ReferencesClientTests.cs", null, 8, 3, 11, null, null, null, null),
new PullRequestFile(null, "Octokit/Clients/ApiPagination.cs", null, 21, 6, 27, null, null, null, null),
new PullRequestFile(null, "Octokit/Helpers/IApiPagination.cs", null, 1, 1, 2, null, null, null, null),
new PullRequestFile(null, "Octokit/Http/ApiConnection.cs", null, 1, 1, 2, null, null, null, null)
new PullRequestFile(null, "Octokit.Tests.Integration/Clients/ReferencesClientTests.cs", null, 8, 3, 11, null, null, null, null, null),
new PullRequestFile(null, "Octokit/Clients/ApiPagination.cs", null, 21, 6, 27, null, null, null, null, null),
new PullRequestFile(null, "Octokit/Helpers/IApiPagination.cs", null, 1, 1, 2, null, null, null, null, null),
new PullRequestFile(null, "Octokit/Http/ApiConnection.cs", null, 1, 1, 2, null, null, null, null, null)
};
var result = await _fixture.Files(7528679, 288);
@@ -693,7 +693,7 @@ namespace Octokit.Tests.Reactive
[Fact]
public async Task FetchesAllFilesForPullRequest()
{
var file = new PullRequestFile(null, null, null, 0, 0, 0, null, null, null, null);
var file = new PullRequestFile(null, null, null, 0, 0, 0, null, null, null, null, null);
var expectedUrl = "repos/fake/repo/pulls/42/files";
var gitHubClient = Substitute.For<IGitHubClient>();
var connection = Substitute.For<IConnection>();
@@ -717,7 +717,7 @@ namespace Octokit.Tests.Reactive
[Fact]
public async Task FetchesAllFilesForPullRequestWithRepositoryId()
{
var file = new PullRequestFile(null, null, null, 0, 0, 0, null, null, null, null);
var file = new PullRequestFile(null, null, null, 0, 0, 0, null, null, null, null, null);
var expectedUrl = "repositories/1/pulls/42/files";
var gitHubClient = Substitute.For<IGitHubClient>();
var connection = Substitute.For<IConnection>();
+4 -1
View File
@@ -9,7 +9,7 @@ namespace Octokit
{
public PullRequestFile() { }
public PullRequestFile(string sha, string fileName, string status, int additions, int deletions, int changes, string blobUrl, string rawUrl, string contentsUrl, string patch)
public PullRequestFile(string sha, string fileName, string status, int additions, int deletions, int changes, string blobUrl, string rawUrl, string contentsUrl, string patch, string previousFileName)
{
Sha = sha;
FileName = fileName;
@@ -21,6 +21,7 @@ namespace Octokit
RawUrl = rawUrl;
ContentsUrl = contentsUrl;
Patch = patch;
PreviousFileName = previousFileName;
}
public string Sha { get; protected set; }
@@ -34,6 +35,8 @@ namespace Octokit
public string RawUrl { get; protected set; }
public string ContentsUrl { get; protected set; }
public string Patch { get; protected set; }
[Parameter(Key = "previous_filename")]
public string PreviousFileName { get; protected set; }
internal string DebuggerDisplay
{