Merge pull request sigmavirus24#551 from itsmemattchung/tests/migrate-repos

sigmavirus24 · sigmavirus24 · commit bcb81739d372 · 2016-01-24T12:49:48.000-06:00
Migrated update_label test to tests/{integration/unit}
diff --git a/LATEST_VERSION_NOTES.rst b/LATEST_VERSION_NOTES.rst
@@ -10,7 +10,9 @@ Breaking Changes (since 1.0.0a2)
 
 - Move ``Users#delete_email_addresses`` to ``GitHub#delete_email_addresses``.
 
-- Remove ``Users#add_email_address`` and ``Users#delete_eail_address``.
+- Remove ``Users#add_email_address`` and ``Users#delete_email_address``.
+
+- Remove ``Repository#update_label``.
 
 Features Added (since 1.0.0a2)
 ``````````````````````````````
diff --git a/github3/repos/repo.py b/github3/repos/repo.py
@@ -1916,22 +1916,6 @@ def tree(self, sha):
             json = self._json(self._get(url), 200)
         return self._instance_or_null(Tree, json)
 
-    @requires_auth
-    def update_label(self, name, color, new_name=''):
-        """Update the label ``name``.
-
-        :param str name: (required), name of the label
-        :param str color: (required), color code
-        :param str new_name: (optional), new name of the label
-        :returns: bool
-        """
-        label = self.label(name)
-        resp = False
-        if label:
-            upd = label.update
-            resp = upd(new_name, color) if new_name else upd(name, color)
-        return resp
-
     def weekly_commit_count(self):
         """Retrieve the total commit counts.
 
diff --git a/tests/test_repos.py b/tests/test_repos.py
@@ -18,26 +18,6 @@ def setUp(self):
     def test_repr(self):
         assert repr(self.repo) == '<Repository [sigmavirus24/github3.py]>'
 
-    def test_update_label(self):
-        self.response('label')
-        self.patch(self.api + 'labels/Bug')
-        self.conf = {'data': {'name': 'big_bug', 'color': 'fafafa'}}
-
-        self.assertRaises(github3.GitHubError, self.repo.update_label,
-                          'foo', 'bar')
-        self.not_called()
-
-        self.login()
-        with mock.patch.object(repos.Repository, 'label') as l:
-            l.return_value = None
-            assert self.repo.update_label('foo', 'bar') is False
-            self.not_called()
-
-        with mock.patch.object(repos.Repository, 'label') as l:
-            l.return_value = github3.issues.label.Label(load('label'), self.g)
-            assert self.repo.update_label('big_bug', 'fafafa')
-
-        self.mock_assertions()
 
     def test_equality(self):
         assert self.repo == repos.Repository(load('repo'))
diff --git a/tests/unit/test_repos_repo.py b/tests/unit/test_repos_repo.py
@@ -1452,9 +1452,3 @@ def test_teams(self):
         """Show that a user must be authenticated to list teams on a repo."""
         with pytest.raises(GitHubError):
             self.instance.teams()
-
-    def test_update_label(self):
-        """
-        Show that a user must be authenticated to update a label on a repo.
-        """
-        self.assert_requires_auth(self.instance.update_label)
