openstack/swift
Code Issues Proposed changes

Replace assertTrue(not <condition>) with assertFalse(<condition>)

Browse Source 
The replacement of assert_ with assertTrue [1] resulted in a number of
tests using calls of the form assertTrue(not <condition>). This patch
replaces those with assertFalse(<condition>).

[1] change I74705c6498249337bfdf955d62e0ad972035bc1f

Change-Id: I78b49558f4425c2335df187b1793d1e4b3c514b1
This commit is contained in:
Alistair Coles 2015-07-28 10:31:54 +01:00
parent 9f1787b653
commit 968c91a465
3 changed files with 32 additions and 31 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

54
test/functional/tests.py
View File

@ -133,7 +133,7 @@ class TestAccount(Base):
def testInvalidUTF8Path(self):
invalid_utf8 = Utils.create_utf8_name()[::-1]
container = self.env.account.container(invalid_utf8)
self.assertTrue(not container.create(cfg={'no_path_quote': True}))
self.assertFalse(container.create(cfg={'no_path_quote': True}))
self.assert_status(412)
self.assert_body('Invalid UTF8 or contains NULL')
@ -313,7 +313,7 @@ class TestAccountNoContainers(Base):
def testGetRequest(self):
for format_type in [None, 'json', 'xml']:
self.assertTrue(not self.env.account.containers(
self.assertFalse(self.env.account.containers(
parms={'format': format_type}))
if format_type is None:
@ -371,7 +371,7 @@ class TestContainer(Base):
self.assertTrue(cont.create())
self.assert_status(201)
else:
self.assertTrue(not cont.create())
self.assertFalse(cont.create())
self.assert_status(400)
def testFileThenContainerDelete(self):
@ -490,7 +490,7 @@ class TestContainer(Base):
self.assertTrue(container.delete())
container = self.env.account.container(invalid_utf8)
self.assertTrue(not container.create(cfg={'no_path_quote': True}))
self.assertFalse(container.create(cfg={'no_path_quote': True}))
self.assert_status(412)
self.assertRaises(ResponseError, container.files,
cfg={'no_path_quote': True})
@ -516,8 +516,8 @@ class TestContainer(Base):
cont_name = cont_name.encode('utf-8')
cont = self.env.account.container(cont_name)
self.assertTrue(not cont.create(cfg={'no_path_quote': True}),
'created container with name %s' % (cont_name))
self.assertFalse(cont.create(cfg={'no_path_quote': True}),
'created container with name %s' % (cont_name))
self.assert_status(404)
self.assertNotIn(cont.name, self.env.account.containers())
@ -531,7 +531,7 @@ class TestContainer(Base):
def testDeleteOnContainerThatDoesNotExist(self):
cont = self.env.account.container(Utils.create_name())
self.assertTrue(not cont.delete())
self.assertFalse(cont.delete())
self.assert_status(404)
def testDeleteOnContainerWithFiles(self):
@ -540,7 +540,7 @@ class TestContainer(Base):
file_item = cont.file(Utils.create_name())
file_item.write_random(self.env.file_size)
self.assertIn(file_item.name, cont.files())
self.assertTrue(not cont.delete())
self.assertFalse(cont.delete())
self.assert_status(409)
def testFileCreateInContainerThatDoesNotExist(self):
@ -625,8 +625,8 @@ class TestContainer(Base):
def testTooLongName(self):
cont = self.env.account.container('x' * 257)
self.assertTrue(not cont.create(),
'created container with name %s' % (cont.name))
self.assertFalse(cont.create(),
'created container with name %s' % (cont.name))
self.assert_status(400)
def testContainerExistenceCachingProblem(self):
@ -967,24 +967,24 @@ class TestFile(Base):
# invalid source container
source_cont = self.env.account.container(Utils.create_name())
file_item = source_cont.file(source_filename)
self.assertTrue(not file_item.copy(
self.assertFalse(file_item.copy(
'%s%s' % (prefix, self.env.container),
Utils.create_name()))
self.assert_status(404)
self.assertTrue(not file_item.copy('%s%s' % (prefix, dest_cont),
Utils.create_name()))
self.assertFalse(file_item.copy('%s%s' % (prefix, dest_cont),
Utils.create_name()))
self.assert_status(404)
# invalid source object
file_item = self.env.container.file(Utils.create_name())
self.assertTrue(not file_item.copy(
self.assertFalse(file_item.copy(
'%s%s' % (prefix, self.env.container),
Utils.create_name()))
self.assert_status(404)
self.assertTrue(not file_item.copy('%s%s' % (prefix, dest_cont),
Utils.create_name()))
self.assertFalse(file_item.copy('%s%s' % (prefix, dest_cont),
Utils.create_name()))
self.assert_status(404)
# invalid destination container
@ -1016,7 +1016,7 @@ class TestFile(Base):
# invalid source container
source_cont = self.env.account.container(Utils.create_name())
file_item = source_cont.file(source_filename)
self.assertTrue(not file_item.copy_account(
self.assertFalse(file_item.copy_account(
acct,
'%s%s' % (prefix, self.env.container),
Utils.create_name()))
@ -1027,7 +1027,7 @@ class TestFile(Base):
else:
self.assert_status(404)
self.assertTrue(not file_item.copy_account(
self.assertFalse(file_item.copy_account(
acct,
'%s%s' % (prefix, cont),
Utils.create_name()))
@ -1035,7 +1035,7 @@ class TestFile(Base):
# invalid source object
file_item = self.env.container.file(Utils.create_name())
self.assertTrue(not file_item.copy_account(
self.assertFalse(file_item.copy_account(
acct,
'%s%s' % (prefix, self.env.container),
Utils.create_name()))
@ -1046,7 +1046,7 @@ class TestFile(Base):
else:
self.assert_status(404)
self.assertTrue(not file_item.copy_account(
self.assertFalse(file_item.copy_account(
acct,
'%s%s' % (prefix, cont),
Utils.create_name()))
@ -1054,7 +1054,7 @@ class TestFile(Base):
# invalid destination container
file_item = self.env.container.file(source_filename)
self.assertTrue(not file_item.copy_account(
self.assertFalse(file_item.copy_account(
acct,
'%s%s' % (prefix, Utils.create_name()),
Utils.create_name()))
@ -1071,9 +1071,9 @@ class TestFile(Base):
file_item.write_random()
file_item = self.env.container.file(source_filename)
self.assertTrue(not file_item.copy(Utils.create_name(),
Utils.create_name(),
cfg={'no_destination': True}))
self.assertFalse(file_item.copy(Utils.create_name(),
Utils.create_name(),
cfg={'no_destination': True}))
self.assert_status(412)
def testCopyDestinationSlashProblems(self):
@ -1082,9 +1082,9 @@ class TestFile(Base):
file_item.write_random()
# no slash
self.assertTrue(not file_item.copy(Utils.create_name(),
Utils.create_name(),
cfg={'destination': Utils.create_name()}))
self.assertFalse(file_item.copy(Utils.create_name(),
Utils.create_name(),
cfg={'destination': Utils.create_name()}))
self.assert_status(412)
def testCopyFromHeader(self):

7
test/probe/test_account_failures.py
View File

@ -123,7 +123,7 @@ class TestAccountFailures(ReplProbeTest):
found2 = True
self.assertEqual(container['count'], 1)
self.assertEqual(container['bytes'], 4)
self.assertTrue(not found1)
self.assertFalse(found1)
self.assertTrue(found2)
# Run container updaters
@ -143,7 +143,7 @@ class TestAccountFailures(ReplProbeTest):
found2 = True
self.assertEqual(container['count'], 2)
self.assertEqual(container['bytes'], 9)
self.assertTrue(not found1)
self.assertFalse(found1)
self.assertTrue(found2)
# Restart other primary account server
@ -187,7 +187,8 @@ class TestAccountFailures(ReplProbeTest):
found2 = True
self.assertEqual(container['count'], 2)
self.assertEqual(container['bytes'], 9)
self.assertTrue(not found1)
self.assertEquals(container['bytes'], 9)
self.assertFalse(found1)
self.assertTrue(found2)

2
test/probe/test_object_async_update.py
View File

@ -54,7 +54,7 @@ class TestObjectAsyncUpdate(ReplProbeTest):
self.ipport2server, self.pids)
# Assert it does not know about container/obj
self.assertTrue(not direct_client.direct_get_container(
self.assertFalse(direct_client.direct_get_container(
cnode, cpart, self.account, container)[1])
# Run the object-updaters