anshprat
diff --git a/‎tests/unit/test_repos_repo.py
Lines changed: 60 additions & 2 deletions b/‎tests/unit/test_repos_repo.py
Lines changed: 60 additions & 2 deletions
@@ -5,6 +5,7 @@
 from github3 import GitHubError
 from github3.null import NullObject
 from github3.repos.repo import Repository
+from github3.models import GitHubCore
 
 from . import helper
 
@@ -248,19 +249,65 @@ def test_create_milestone_accepted_state(self):
             }
         )
 
-    def test_create_pull(self):
+    def test_create_pull_private_required_data(self):
+        """Verify the request for creating a pull request."""
+        with helper.mock.patch.object(GitHubCore, '_remove_none') as rm_none:
+            data = {}
+            self.instance._create_pull(data)
+            rm_none.assert_called_once_with({})
+            assert self.session.post.called is False
+
+    def test_create_pull_private(self):
         """Verify the request for creating a pull request."""
         data = {
             'title': 'foo',
             'base': 'master',
             'head': 'feature_branch'
         }
-        self.instance.create_pull(**data)
+        self.instance._create_pull(data)
         self.post_called_with(
             url_for('pulls'),
             data=data
         )
 
+    def test_create_pull(self):
+        """Verify the request for creating a pull request."""
+        data = {
+            'title': 'foo',
+            'base': 'master',
+            'head': 'feature_branch',
+            'body': 'body'
+        }
+        with helper.mock.patch.object(Repository, '_create_pull') as pull:
+            self.instance.create_pull(**data)
+            pull.assert_called_once_with(
+                data
+            )
+
+    def test_create_pull_from_issue(self):
+        """Verify the request for creating a pull request from an issue."""
+        with helper.mock.patch.object(Repository, '_create_pull') as pull:
+            data = {
+                'issue': 1,
+                'base': 'master',
+                'head': 'feature_branch'
+            }
+            self.instance.create_pull_from_issue(
+                **data
+            )
+            pull.assert_called_once_with(data)
+
+    def test_create_pull_from_issue_required_issue_number(self):
+        """Verify the request for creating a pull request from an issue."""
+        with helper.mock.patch.object(Repository, '_create_pull') as pull:
+            pull_request = self.instance.create_pull_from_issue(
+                issue=-1,
+                base='master',
+                head='feature_branch'
+            )
+            assert pull.called is False
+            assert pull_request is None
+
     def test_create_ref(self):
         """Verify the request to create a reference."""
         self.instance.create_ref('refs/heads/foo', 'my-fake-sha')
@@ -903,6 +950,17 @@ def test_create_pull(self):
         with pytest.raises(GitHubError):
             self.instance.create_pull(title='foo', base='master')
 
+    def test_create_pull_from_issue(self):
+        """
+        Verify that creating a pull request from issue requires authentication.
+        """
+        with pytest.raises(GitHubError):
+            self.instance.create_pull_from_issue(
+                issue=1,
+                title='foo',
+                base='master'
+            )
+
     def test_hooks(self):
         """Show that a user must be authenticated to list hooks."""
         with pytest.raises(GitHubError):