Migrated git unit tests from tests/test_git.py -> tests/unit/test_git.py

itsmemattchung · itsmemattchung · commit 93ba412c27c2 · 2015-12-03T06:18:36.000Z
Created additional octocat examples
diff --git a/tests/unit/commit_example b/tests/unit/commit_example
@@ -0,0 +1,86 @@
+{
+  "url": "https://api.github.com/repos/octocat/Hello-World/commits/6dcb09b5b57875f334f61aebed695e2e4193db5e",
+  "sha": "6dcb09b5b57875f334f61aebed695e2e4193db5e",
+  "html_url": "https://github.com/octocat/Hello-World/commit/6dcb09b5b57875f334f61aebed695e2e4193db5e",
+  "comments_url": "https://api.github.com/repos/octocat/Hello-World/commits/6dcb09b5b57875f334f61aebed695e2e4193db5e/comments",
+  "commit": {
+    "url": "https://api.github.com/repos/octocat/Hello-World/git/commits/6dcb09b5b57875f334f61aebed695e2e4193db5e",
+    "author": {
+      "name": "Monalisa Octocat",
+      "email": "support@github.com",
+      "date": "2011-04-14T16:00:49Z"
+    },
+    "committer": {
+      "name": "Monalisa Octocat",
+      "email": "support@github.com",
+      "date": "2011-04-14T16:00:49Z"
+    },
+    "message": "Fix all the bugs",
+    "tree": {
+      "url": "https://api.github.com/repos/octocat/Hello-World/tree/6dcb09b5b57875f334f61aebed695e2e4193db5e",
+      "sha": "6dcb09b5b57875f334f61aebed695e2e4193db5e"
+    },
+    "comment_count": 0
+  },
+  "author": {
+    "login": "octocat",
+    "id": 1,
+    "avatar_url": "https://github.com/images/error/octocat_happy.gif",
+    "gravatar_id": "",
+    "url": "https://api.github.com/users/octocat",
+    "html_url": "https://github.com/octocat",
+    "followers_url": "https://api.github.com/users/octocat/followers",
+    "following_url": "https://api.github.com/users/octocat/following{/other_user}",
+    "gists_url": "https://api.github.com/users/octocat/gists{/gist_id}",
+    "starred_url": "https://api.github.com/users/octocat/starred{/owner}{/repo}",
+    "subscriptions_url": "https://api.github.com/users/octocat/subscriptions",
+    "organizations_url": "https://api.github.com/users/octocat/orgs",
+    "repos_url": "https://api.github.com/users/octocat/repos",
+    "events_url": "https://api.github.com/users/octocat/events{/privacy}",
+    "received_events_url": "https://api.github.com/users/octocat/received_events",
+    "type": "User",
+    "site_admin": false
+  },
+  "committer": {
+    "login": "octocat",
+    "id": 1,
+    "avatar_url": "https://github.com/images/error/octocat_happy.gif",
+    "gravatar_id": "",
+    "url": "https://api.github.com/users/octocat",
+    "html_url": "https://github.com/octocat",
+    "followers_url": "https://api.github.com/users/octocat/followers",
+    "following_url": "https://api.github.com/users/octocat/following{/other_user}",
+    "gists_url": "https://api.github.com/users/octocat/gists{/gist_id}",
+    "starred_url": "https://api.github.com/users/octocat/starred{/owner}{/repo}",
+    "subscriptions_url": "https://api.github.com/users/octocat/subscriptions",
+    "organizations_url": "https://api.github.com/users/octocat/orgs",
+    "repos_url": "https://api.github.com/users/octocat/repos",
+    "events_url": "https://api.github.com/users/octocat/events{/privacy}",
+    "received_events_url": "https://api.github.com/users/octocat/received_events",
+    "type": "User",
+    "site_admin": false
+  },
+  "parents": [
+    {
+      "url": "https://api.github.com/repos/octocat/Hello-World/commits/6dcb09b5b57875f334f61aebed695e2e4193db5e",
+      "sha": "6dcb09b5b57875f334f61aebed695e2e4193db5e"
+    }
+  ],
+  "stats": {
+    "additions": 104,
+    "deletions": 4,
+    "total": 108
+  },
+  "files": [
+    {
+      "filename": "file1.txt",
+      "additions": 10,
+      "deletions": 2,
+      "changes": 12,
+      "status": "modified",
+      "raw_url": "https://github.com/octocat/Hello-World/raw/7ca483543807a51b6079e54ac4cc392bc29ae284/file1.txt",
+      "blob_url": "https://github.com/octocat/Hello-World/blob/7ca483543807a51b6079e54ac4cc392bc29ae284/file1.txt",
+      "patch": "@@ -29,7 +29,7 @@\n....."
+    }
+  ]
+}
diff --git a/tests/unit/reference_example b/tests/unit/reference_example
@@ -0,0 +1,9 @@
+{
+  "ref": "refs/heads/featureA",
+  "url": "https://api.github.com/repos/octocat/Hello-World/git/refs/heads/featureA",
+  "object": {
+    "type": "commit",
+    "sha": "aa218f56b14c9653891f9e74264a383fa43fefbd",
+    "url": "https://api.github.com/repos/octocat/Hello-World/git/commits/aa218f56b14c9653891f9e74264a383fa43fefbd"
+  }
+}
diff --git a/tests/unit/test_git.py b/tests/unit/test_git.py
@@ -6,6 +6,12 @@
 url_for = create_url_helper('https://api.github.com/repos/octocat/Hello-World/'
                             'trees/9fb037999f264ba9a7fc6274d15fa3ae2ab98312')
 
+delete_url_for = create_url_helper('https://api.github.com/repos/octocat/Hello-World/'
+                                   'git/refs/heads/featureA')
+
+get_commit_example_data = create_example_data_helper('commit_example')
+get_reference_example_data = create_example_data_helper('reference_example')
+
 
 class TestTree(UnitHelper):
     """Tree unit test"""
@@ -25,3 +31,51 @@ def test_recurse(self):
             url_for(),
             params={'recursive': '1'}
         )
+
+
+class TestCommit(UnitHelper):
+
+    "Commit unit test."""
+
+    described_class = github3.git.Commit
+    example_data = get_commit_example_data()
+
+    def test_repr(self):
+        assert repr(self.instance).startswith('<Commit')
+
+    def test_committer_as_User(self):
+        """Show that commit_as_User() returns instance of User."""
+        user = self.instance.committer_as_User()
+        assert isinstance(user, github3.users.User)
+
+    def test_author_as_User(self):
+        """Show that commit_as_Author() returns instance of User."""
+        user = self.instance.author_as_User()
+        assert isinstance(user, github3.users.User)
+
+
+class TestReference(UnitHelper):
+
+    """Reference unit test."""
+
+    described_class = github3.git.Reference
+    example_data = get_reference_example_data()
+
+    def test_delete(self):
+        self.instance.delete()
+        self.session.delete.assert_called_once_with(
+            delete_url_for()
+        )
+
+    def test_repr(self):
+        assert repr(self.instance).startswith('<Reference')
+        assert repr(self.instance.object).startswith('<Git Object')
+
+    def test_update(self):
+        """Show that a user can update the reference."""
+
+        self.instance.update('fakesha', True)
+        self.session.patch.assert_called_once_with(
+            delete_url_for(),
+            data='{"sha": "fakesha", "force": true}'
+        )
