Replace assert_raises with assertRaises

Author: Harmon758

git/test/lib/asserts.py

@@ -7,11 +7,9 @@
 from unittest.mock import patch
 
 from nose.tools import (
-    assert_raises,      # @UnusedImport
     raises,             # @UnusedImport
     assert_true,        # @UnusedImport
     assert_false        # @UnusedImport
 )
 
-__all__ = ['assert_raises', 'patch', 'raises',
-           'assert_true', 'assert_false']
+__all__ = ['patch', 'raises', 'assert_true', 'assert_false']

git/test/test_base.py

@@ -19,7 +19,6 @@
 from git.objects.util import get_object_type_by_name
 from git.test.lib import (
     TestBase,
-    assert_raises,
     with_rw_repo,
     with_rw_and_rw_remote_repo
 )
@@ -96,7 +95,7 @@ def test_get_object_type_by_name(self):
             assert base.Object in get_object_type_by_name(tname).mro()
         # END for each known type
 
-        assert_raises(ValueError, get_object_type_by_name, b"doesntexist")
+        self.assertRaises(ValueError, get_object_type_by_name, b"doesntexist")
 
     def test_object_resolution(self):
         # objects must be resolved to shas so they compare equal

git/test/test_remote.py

@@ -27,8 +27,7 @@
     with_rw_repo,
     with_rw_and_rw_remote_repo,
     fixture,
-    GIT_DAEMON_PORT,
-    assert_raises
+    GIT_DAEMON_PORT
 )
 from git.util import rmtree, HIDE_WINDOWS_FREEZE_ERRORS
 import os.path as osp
@@ -626,7 +625,7 @@ def test_multiple_urls(self, rw_repo):
         self.assertEqual(list(remote.urls), [test1, test2])
 
         # will raise: fatal: --add --delete doesn't make sense
-        assert_raises(GitCommandError, remote.set_url, test2, add=True, delete=True)
+        self.assertRaises(GitCommandError, remote.set_url, test2, add=True, delete=True)
 
         # Testing on another remote, with the add/delete URL
         remote = rw_repo.create_remote('another', url=test1)
@@ -640,7 +639,7 @@ def test_multiple_urls(self, rw_repo):
         remote.delete_url(test1)
         self.assertEqual(list(remote.urls), [test3])
         # will raise fatal: Will not delete all non-push URLs
-        assert_raises(GitCommandError, remote.delete_url, test3)
+        self.assertRaises(GitCommandError, remote.delete_url, test3)
 
     def test_fetch_error(self):
         rem = self.rorepo.remote('origin')