mirror of
https://github.com/samba-team/samba.git
synced 2025-07-29 15:42:04 +03:00
pytests: heed assertEquals deprecation warning en-masse
TestCase.assertEquals() is an alias for TestCase.assertEqual() and has been deprecated since Python 2.7. When we run our tests with in python developer mode (`PYTHONDEVMODE=1 make test`) we get 580 DeprecationWarnings about this. Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz> Reviewed-by: Noel Power <npower@samba.org>
This commit is contained in:
committed by
Noel Power
parent
3bc7acc626
commit
c247afbda0
@ -32,7 +32,7 @@ class NtACLCmdSysvolTestCase(SambaToolCmdTest):
|
||||
(result, out, err) = self.runsubcmd("ntacl", "sysvolreset",
|
||||
"--use-ntvfs")
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(out, "", "Shouldn't be any output messages")
|
||||
self.assertEqual(out, "", "Shouldn't be any output messages")
|
||||
self.assertIn("Please note that POSIX permissions have NOT been changed, only the stored NT ACL", err)
|
||||
|
||||
def test_s3fs(self):
|
||||
@ -40,34 +40,34 @@ class NtACLCmdSysvolTestCase(SambaToolCmdTest):
|
||||
"--use-s3fs")
|
||||
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(err, "", "Shouldn't be any error messages")
|
||||
self.assertEquals(out, "", "Shouldn't be any output messages")
|
||||
self.assertEqual(err, "", "Shouldn't be any error messages")
|
||||
self.assertEqual(out, "", "Shouldn't be any output messages")
|
||||
|
||||
def test_ntvfs_check(self):
|
||||
(result, out, err) = self.runsubcmd("ntacl", "sysvolreset",
|
||||
"--use-ntvfs")
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(out, "", "Shouldn't be any output messages")
|
||||
self.assertEqual(out, "", "Shouldn't be any output messages")
|
||||
self.assertIn("Please note that POSIX permissions have NOT been changed, only the stored NT ACL", err)
|
||||
# Now check they were set correctly
|
||||
(result, out, err) = self.runsubcmd("ntacl", "sysvolcheck")
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(err, "", "Shouldn't be any error messages")
|
||||
self.assertEquals(out, "", "Shouldn't be any output messages")
|
||||
self.assertEqual(err, "", "Shouldn't be any error messages")
|
||||
self.assertEqual(out, "", "Shouldn't be any output messages")
|
||||
|
||||
def test_s3fs_check(self):
|
||||
(result, out, err) = self.runsubcmd("ntacl", "sysvolreset",
|
||||
"--use-s3fs")
|
||||
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(err, "", "Shouldn't be any error messages")
|
||||
self.assertEquals(out, "", "Shouldn't be any output messages")
|
||||
self.assertEqual(err, "", "Shouldn't be any error messages")
|
||||
self.assertEqual(out, "", "Shouldn't be any output messages")
|
||||
|
||||
# Now check they were set correctly
|
||||
(result, out, err) = self.runsubcmd("ntacl", "sysvolcheck")
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(err, "", "Shouldn't be any error messages")
|
||||
self.assertEquals(out, "", "Shouldn't be any output messages")
|
||||
self.assertEqual(err, "", "Shouldn't be any error messages")
|
||||
self.assertEqual(out, "", "Shouldn't be any output messages")
|
||||
|
||||
|
||||
class NtACLCmdGetSetTestCase(SambaToolCmdTest):
|
||||
@ -83,7 +83,7 @@ class NtACLCmdGetSetTestCase(SambaToolCmdTest):
|
||||
(result, out, err) = self.runsubcmd("ntacl", "set", self.acl, tempf,
|
||||
"--use-ntvfs")
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(out, "", "Shouldn't be any output messages")
|
||||
self.assertEqual(out, "", "Shouldn't be any output messages")
|
||||
self.assertIn("Please note that POSIX permissions have NOT been changed, only the stored NT ACL", err)
|
||||
|
||||
def test_s3fs(self):
|
||||
@ -95,8 +95,8 @@ class NtACLCmdGetSetTestCase(SambaToolCmdTest):
|
||||
"--use-s3fs")
|
||||
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(err, "", "Shouldn't be any error messages")
|
||||
self.assertEquals(out, "", "Shouldn't be any output messages")
|
||||
self.assertEqual(err, "", "Shouldn't be any error messages")
|
||||
self.assertEqual(out, "", "Shouldn't be any output messages")
|
||||
|
||||
def test_ntvfs_check(self):
|
||||
path = os.environ['SELFTEST_PREFIX']
|
||||
@ -106,15 +106,15 @@ class NtACLCmdGetSetTestCase(SambaToolCmdTest):
|
||||
(result, out, err) = self.runsubcmd("ntacl", "set", self.acl, tempf,
|
||||
"--use-ntvfs")
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(out, "", "Shouldn't be any output messages")
|
||||
self.assertEqual(out, "", "Shouldn't be any output messages")
|
||||
self.assertIn("Please note that POSIX permissions have NOT been changed, only the stored NT ACL", err)
|
||||
|
||||
# Now check they were set correctly
|
||||
(result, out, err) = self.runsubcmd("ntacl", "get", tempf,
|
||||
"--use-ntvfs", "--as-sddl")
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(err, "", "Shouldn't be any error messages")
|
||||
self.assertEquals(self.acl + "\n", out, "Output should be the ACL")
|
||||
self.assertEqual(err, "", "Shouldn't be any error messages")
|
||||
self.assertEqual(self.acl + "\n", out, "Output should be the ACL")
|
||||
|
||||
def test_s3fs_check(self):
|
||||
path = os.environ['SELFTEST_PREFIX']
|
||||
@ -124,15 +124,15 @@ class NtACLCmdGetSetTestCase(SambaToolCmdTest):
|
||||
(result, out, err) = self.runsubcmd("ntacl", "set", self.acl, tempf,
|
||||
"--use-s3fs")
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(out, "", "Shouldn't be any output messages")
|
||||
self.assertEquals(err, "", "Shouldn't be any error messages")
|
||||
self.assertEqual(out, "", "Shouldn't be any output messages")
|
||||
self.assertEqual(err, "", "Shouldn't be any error messages")
|
||||
|
||||
# Now check they were set correctly
|
||||
(result, out, err) = self.runsubcmd("ntacl", "get", tempf,
|
||||
"--use-s3fs", "--as-sddl")
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(err, "", "Shouldn't be any error messages")
|
||||
self.assertEquals(self.acl + "\n", out, "Output should be the ACL")
|
||||
self.assertEqual(err, "", "Shouldn't be any error messages")
|
||||
self.assertEqual(self.acl + "\n", out, "Output should be the ACL")
|
||||
|
||||
class NtACLCmdChangedomsidTestCase(SambaToolCmdTest):
|
||||
"""Tests for samba-tool ntacl changedomsid subcommand"""
|
||||
@ -156,7 +156,7 @@ class NtACLCmdChangedomsidTestCase(SambaToolCmdTest):
|
||||
tempf,
|
||||
"--use-ntvfs")
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(out, "", "Shouldn't be any output messages")
|
||||
self.assertEqual(out, "", "Shouldn't be any output messages")
|
||||
self.assertIn("Please note that POSIX permissions have NOT been "
|
||||
"changed, only the stored NT ACL", err)
|
||||
|
||||
@ -167,7 +167,7 @@ class NtACLCmdChangedomsidTestCase(SambaToolCmdTest):
|
||||
tempf,
|
||||
"--use-ntvfs")
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(out, "", "Shouldn't be any output messages")
|
||||
self.assertEqual(out, "", "Shouldn't be any output messages")
|
||||
self.assertIn("Please note that POSIX permissions have NOT been "
|
||||
"changed, only the stored NT ACL.", err)
|
||||
|
||||
@ -178,8 +178,8 @@ class NtACLCmdChangedomsidTestCase(SambaToolCmdTest):
|
||||
"--use-ntvfs",
|
||||
"--as-sddl")
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(err, "", "Shouldn't be any error messages")
|
||||
self.assertEquals(self.new_acl + "\n", out, "Output should be the ACL")
|
||||
self.assertEqual(err, "", "Shouldn't be any error messages")
|
||||
self.assertEqual(self.new_acl + "\n", out, "Output should be the ACL")
|
||||
|
||||
def test_s3fs_check(self):
|
||||
path = os.environ['SELFTEST_PREFIX']
|
||||
@ -196,8 +196,8 @@ class NtACLCmdChangedomsidTestCase(SambaToolCmdTest):
|
||||
"--use-s3fs",
|
||||
"--service=sysvol")
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(out, "", "Shouldn't be any output messages")
|
||||
self.assertEquals(err, "", "Shouldn't be any error messages")
|
||||
self.assertEqual(out, "", "Shouldn't be any output messages")
|
||||
self.assertEqual(err, "", "Shouldn't be any error messages")
|
||||
|
||||
(result, out, err) = self.runsubcmd("ntacl",
|
||||
"changedomsid",
|
||||
@ -207,8 +207,8 @@ class NtACLCmdChangedomsidTestCase(SambaToolCmdTest):
|
||||
"--use-s3fs",
|
||||
"--service=sysvol")
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(out, "", "Shouldn't be any output messages")
|
||||
self.assertEquals(err, "", "Shouldn't be any error messages")
|
||||
self.assertEqual(out, "", "Shouldn't be any output messages")
|
||||
self.assertEqual(err, "", "Shouldn't be any error messages")
|
||||
|
||||
# Now check they were set correctly
|
||||
(result, out, err) = self.runsubcmd("ntacl",
|
||||
@ -218,5 +218,5 @@ class NtACLCmdChangedomsidTestCase(SambaToolCmdTest):
|
||||
"--as-sddl",
|
||||
"--service=sysvol")
|
||||
self.assertCmdSuccess(result, out, err)
|
||||
self.assertEquals(err, "", "Shouldn't be any error messages")
|
||||
self.assertEquals(self.new_acl + "\n", out, "Output should be the ACL")
|
||||
self.assertEqual(err, "", "Shouldn't be any error messages")
|
||||
self.assertEqual(self.new_acl + "\n", out, "Output should be the ACL")
|
||||
|
Reference in New Issue
Block a user