add GraphQL Node ID's to response models

Octokit.Tests/Models/MigrationTests.cs

@@ -110,20 +110,6 @@ public class MigrationTests
               ]
             }";
 
-        private static readonly Migration migration = new Migration(
-            id: 79,
-            guid: "0b989ba4-242f-11e5-81e1-c7b6966d2516",
-            state: Migration.MigrationState.Exported,
-            lockRepositories: true,
-            excludeAttachments: false,
-            url: "https://api.github.com/orgs/octo-org/migrations/79",
-            createdAt: "2015-07-06T15:33:38-07:00",
-            updatedAt: "2015-07-06T15:33:38-07:00",
-            repositories: new List<Repository>
-            {
-                new Repository(1296269)
-            });
-
         [Fact]
         public void CanBeDeserialized()
         {

Octokit.Tests/Reactive/ObservablePullRequestsClientTests.cs

@@ -629,7 +629,7 @@ public class TheCommitsMethod
             [Fact]
             public async Task FetchesAllCommitsForPullRequest()
             {
-                var commit = new PullRequestCommit(null, null, null, null, null, Enumerable.Empty<GitReference>(), null, null);
+                var commit = new PullRequestCommit("123ABC", null, null, null, null, null, Enumerable.Empty<GitReference>(), null, null);
                 var expectedUrl = "repos/fake/repo/pulls/42/commits";
                 var gitHubClient = Substitute.For<IGitHubClient>();
                 var connection = Substitute.For<IConnection>();
@@ -653,7 +653,7 @@ public async Task FetchesAllCommitsForPullRequest()
             [Fact]
             public async Task FetchesAllCommitsForPullRequestWithRepositoryId()
             {
-                var commit = new PullRequestCommit(null, null, null, null, null, Enumerable.Empty<GitReference>(), null, null);
+                var commit = new PullRequestCommit("123ABC", null, null, null, null, null, Enumerable.Empty<GitReference>(), null, null);
                 var expectedUrl = "repositories/1/pulls/42/commits";
                 var gitHubClient = Substitute.For<IGitHubClient>();
                 var connection = Substitute.For<IConnection>();

Octokit/Models/Common/Committer.cs

@@ -29,6 +29,26 @@ public Committer(string name, string email, DateTimeOffset date)
             Date = date;
         }
 
+        /// <summary>
+        /// Initializes a new instance of the <see cref="Committer"/> class.
+        /// </summary>
+        /// <param name="nodeId">The GraphQL Node Id</param>
+        /// <param name="name">The full name of the author or committer.</param>
+        /// <param name="email">The email.</param>
+        /// <param name="date">The date.</param>
+        public Committer(string nodeId, string name, string email, DateTimeOffset date)
+        {
+            NodeId = nodeId;
+            Name = name;
+            Email = email;
+            Date = date;
+        }
+
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// Gets the name of the author or committer.
         /// </summary>

Octokit/Models/Response/Account.cs

@@ -9,7 +9,7 @@ public abstract class Account
     {
         protected Account() { }
 
-        protected Account(string avatarUrl, string bio, string blog, int collaborators, string company, DateTimeOffset createdAt, int diskUsage, string email, int followers, int following, bool? hireable, string htmlUrl, int totalPrivateRepos, int id, string location, string login, string name, int ownedPrivateRepos, Plan plan, int privateGists, int publicGists, int publicRepos, AccountType type, string url)
+        protected Account(string avatarUrl, string bio, string blog, int collaborators, string company, DateTimeOffset createdAt, int diskUsage, string email, int followers, int following, bool? hireable, string htmlUrl, int totalPrivateRepos, int id, string location, string login, string name, string nodeId, int ownedPrivateRepos, Plan plan, int privateGists, int publicGists, int publicRepos, AccountType type, string url)
         {
             AvatarUrl = avatarUrl;
             Bio = bio;
@@ -28,6 +28,7 @@ protected Account(string avatarUrl, string bio, string blog, int collaborators,
             Location = location;
             Login = login;
             Name = name;
+            NodeId = nodeId;
             OwnedPrivateRepos = ownedPrivateRepos;
             Plan = plan;
             PrivateGists = privateGists;
@@ -103,6 +104,11 @@ protected Account(string avatarUrl, string bio, string blog, int collaborators,
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The account's geographic location.
         /// </summary>

Octokit/Models/Response/Author.cs

@@ -9,10 +9,11 @@ public class Author
     {
         public Author() { }
 
-        public Author(string login, int id, string avatarUrl, string url, string htmlUrl, string followersUrl, string followingUrl, string gistsUrl, string type, string starredUrl, string subscriptionsUrl, string organizationsUrl, string reposUrl, string eventsUrl, string receivedEventsUrl, bool siteAdmin)
+        public Author(string login, int id, string nodeId, string avatarUrl, string url, string htmlUrl, string followersUrl, string followingUrl, string gistsUrl, string type, string starredUrl, string subscriptionsUrl, string organizationsUrl, string reposUrl, string eventsUrl, string receivedEventsUrl, bool siteAdmin)
         {
             Login = login;
             Id = id;
+            NodeId = nodeId;
             AvatarUrl = avatarUrl;
             Url = url;
             HtmlUrl = htmlUrl;
@@ -33,6 +34,11 @@ public Author(string login, int id, string avatarUrl, string url, string htmlUrl
 
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         public string AvatarUrl { get; protected set; }
 
         public string Url { get; protected set; }

Octokit/Models/Response/Blob.cs

@@ -9,14 +9,20 @@ public class Blob
     {
         public Blob() { }
 
-        public Blob(string content, EncodingType encoding, string sha, int size)
+        public Blob(string nodeId, string content, EncodingType encoding, string sha, int size)
         {
+            NodeId = nodeId;
             Content = content;
             Encoding = encoding;
             Sha = sha;
             Size = size;
         }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The content of the blob.
         /// </summary>

Octokit/Models/Response/Commit.cs

@@ -10,8 +10,8 @@ public class Commit : GitReference
     {
         public Commit() { }
 
-        public Commit(string url, string label, string @ref, string sha, User user, Repository repository, string message, Committer author, Committer committer, GitReference tree, IEnumerable<GitReference> parents, int commentCount, Verification verification)
-            : base(url, label, @ref, sha, user, repository)
+        public Commit(string nodeId, string url, string label, string @ref, string sha, User user, Repository repository, string message, Committer author, Committer committer, GitReference tree, IEnumerable<GitReference> parents, int commentCount, Verification verification)
+            : base(nodeId, url, label, @ref, sha, user, repository)
         {
             Ensure.ArgumentNotNull(parents, nameof(parents));
 

Octokit/Models/Response/CommitComment.cs

@@ -9,9 +9,10 @@ public class CommitComment
     {
         public CommitComment() { }
 
-        public CommitComment(int id, string url, string htmlUrl, string body, string path, int position, int? line, string commitId, User user, DateTimeOffset createdAt, DateTimeOffset? updatedAt, ReactionSummary reactions)
+        public CommitComment(int id, string nodeId, string url, string htmlUrl, string body, string path, int position, int? line, string commitId, User user, DateTimeOffset createdAt, DateTimeOffset? updatedAt, ReactionSummary reactions)
         {
             Id = id;
+            NodeId = nodeId;
             Url = url;
             HtmlUrl = htmlUrl;
             Body = body;
@@ -30,6 +31,11 @@ public CommitComment(int id, string url, string htmlUrl, string body, string pat
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The URL for this repository comment.
         /// </summary>

Octokit/Models/Response/CommitStatus.cs

@@ -10,7 +10,7 @@ public class CommitStatus
     {
         public CommitStatus() { }
 
-        public CommitStatus(DateTimeOffset createdAt, DateTimeOffset updatedAt, CommitState state, string targetUrl, string description, string context, long id, string url, User creator)
+        public CommitStatus(DateTimeOffset createdAt, DateTimeOffset updatedAt, CommitState state, string targetUrl, string description, string context, long id, string nodeId, string url, User creator)
         {
             CreatedAt = createdAt;
             UpdatedAt = updatedAt;
@@ -19,6 +19,7 @@ public CommitStatus(DateTimeOffset createdAt, DateTimeOffset updatedAt, CommitSt
             Description = description;
             Context = context;
             Id = id;
+            NodeId = nodeId;
             Url = url;
             Creator = creator;
         }
@@ -59,6 +60,11 @@ public CommitStatus(DateTimeOffset createdAt, DateTimeOffset updatedAt, CommitSt
         /// </summary>
         public long Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The URL of the status.
         /// </summary>

Octokit/Models/Response/Deployment.cs

@@ -13,9 +13,10 @@ public class Deployment
     {
         public Deployment() { }
 
-        public Deployment(int id, string sha, string url, User creator, IReadOnlyDictionary<string, string> payload, DateTimeOffset createdAt, DateTimeOffset updatedAt, string description, string statusesUrl, bool transientEnvironment, bool productionEnvironment)
+        public Deployment(int id, string nodeId, string sha, string url, User creator, IReadOnlyDictionary<string, string> payload, DateTimeOffset createdAt, DateTimeOffset updatedAt, string description, string statusesUrl, bool transientEnvironment, bool productionEnvironment)
         {
             Id = id;
+            NodeId = nodeId;
             Sha = sha;
             Url = url;
             Creator = creator;
@@ -33,6 +34,11 @@ public Deployment(int id, string sha, string url, User creator, IReadOnlyDiction
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// 
         /// </summary>

Octokit/Models/Response/DeploymentStatus.cs

@@ -11,9 +11,10 @@ public class DeploymentStatus
     {
         public DeploymentStatus() { }
 
-        public DeploymentStatus(int id, string url, DeploymentState state, User creator, IReadOnlyDictionary<string, string> payload, string targetUrl, string logUrl, string environmentUrl, DateTimeOffset createdAt, DateTimeOffset updatedAt, string description)
+        public DeploymentStatus(int id, string nodeId, string url, DeploymentState state, User creator, IReadOnlyDictionary<string, string> payload, string targetUrl, string logUrl, string environmentUrl, DateTimeOffset createdAt, DateTimeOffset updatedAt, string description)
         {
             Id = id;
+            NodeId = nodeId;
             Url = url;
             State = state;
             Creator = creator;
@@ -31,6 +32,11 @@ public DeploymentStatus(int id, string url, DeploymentState state, User creator,
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The API URL for this deployment status.
         /// </summary>

Octokit/Models/Response/EventInfo.cs

@@ -11,9 +11,10 @@ public class EventInfo
     {
         public EventInfo() { }
 
-        public EventInfo(int id, string url, User actor, User assignee, Label label, EventInfoState @event, string commitId, DateTimeOffset createdAt)
+        public EventInfo(int id, string nodeId, string url, User actor, User assignee, Label label, EventInfoState @event, string commitId, DateTimeOffset createdAt)
         {
             Id = id;
+            NodeId = nodeId;
             Url = url;
             Actor = actor;
             Assignee = assignee;
@@ -28,6 +29,11 @@ public EventInfo(int id, string url, User actor, User assignee, Label label, Eve
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The URL for this event.
         /// </summary>

Octokit/Models/Response/Gist.cs

@@ -10,10 +10,11 @@ public class Gist
     {
         public Gist() { }
 
-        public Gist(string url, string id, string description, bool @public, User owner, IReadOnlyDictionary<string, GistFile> files, int comments, string commentsUrl, string htmlUrl, string gitPullUrl, string gitPushUrl, DateTimeOffset createdAt, DateTimeOffset updatedAt, IReadOnlyList<GistFork> forks, IReadOnlyList<GistHistory> history)
+        public Gist(string url, string id, string nodeId, string description, bool @public, User owner, IReadOnlyDictionary<string, GistFile> files, int comments, string commentsUrl, string htmlUrl, string gitPullUrl, string gitPushUrl, DateTimeOffset createdAt, DateTimeOffset updatedAt, IReadOnlyList<GistFork> forks, IReadOnlyList<GistHistory> history)
         {
             Url = url;
             Id = id;
+            NodeId = nodeId;
             Description = description;
             Public = @public;
             Owner = owner;
@@ -42,6 +43,11 @@ public Gist(string url, string id, string description, bool @public, User owner,
         /// </remarks>
         public string Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// A description of the <see cref="Gist"/>.
         /// </summary>

Octokit/Models/Response/GistComment.cs

@@ -9,9 +9,10 @@ public class GistComment
     {
         public GistComment() { }
 
-        public GistComment(int id, string url, string body, User user, DateTimeOffset createdAt, DateTimeOffset? updatedAt)
+        public GistComment(int id, string nodeId, string url, string body, User user, DateTimeOffset createdAt, DateTimeOffset? updatedAt)
         {
             Id = id;
+            NodeId = nodeId;
             Url = url;
             Body = body;
             User = user;
@@ -24,6 +25,11 @@ public GistComment(int id, string url, string body, User user, DateTimeOffset cr
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The URL for this gist comment.
         /// </summary>

Octokit/Models/Response/GistFork.cs

@@ -9,13 +9,19 @@ public class GistFork
     {
         public GistFork() { }
 
-        public GistFork(User user, string url, DateTimeOffset createdAt)
+        public GistFork(string nodeId, User user, string url, DateTimeOffset createdAt)
         {
+            NodeId = nodeId;
             User = user;
             Url = url;
             CreatedAt = createdAt;
         }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The <see cref="User"/> that created this <see cref="GistFork"/>
         /// </summary>

Octokit/Models/Response/GitHubCommit.cs

@@ -11,8 +11,8 @@ public class GitHubCommit : GitReference
     {
         public GitHubCommit() { }
 
-        public GitHubCommit(string url, string label, string @ref, string sha, User user, Repository repository, Author author, string commentsUrl, Commit commit, Author committer, string htmlUrl, GitHubCommitStats stats, IReadOnlyList<GitReference> parents, IReadOnlyList<GitHubCommitFile> files)
-            : base(url, label, @ref, sha, user, repository)
+        public GitHubCommit(string nodeId, string url, string label, string @ref, string sha, User user, Repository repository, Author author, string commentsUrl, Commit commit, Author committer, string htmlUrl, GitHubCommitStats stats, IReadOnlyList<GitReference> parents, IReadOnlyList<GitHubCommitFile> files)
+            : base(nodeId, url, label, @ref, sha, user, repository)
         {
             Author = author;
             CommentsUrl = commentsUrl;

Octokit/Models/Response/GitReference.cs

@@ -9,8 +9,9 @@ public class GitReference
     {
         public GitReference() { }
 
-        public GitReference(string url, string label, string @ref, string sha, User user, Repository repository)
+        public GitReference(string nodeId, string url, string label, string @ref, string sha, User user, Repository repository)
         {
+            NodeId = nodeId;
             Url = url;
             Label = label;
             Ref = @ref;
@@ -19,6 +20,11 @@ public GitReference(string url, string label, string @ref, string sha, User user
             Repository = repository;
         }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The URL associated with this reference.
         /// </summary>