diff options
author | Pushpesh Sharma <psharma@redhat.com> | 2014-01-16 11:46:47 +0530 |
---|---|---|
committer | Luis Pabon <lpabon@redhat.com> | 2014-01-21 10:12:02 -0800 |
commit | d7af44b204f98d4a2e908d9ac1d67a115049f54e (patch) | |
tree | d3d0fe72192634e9ecc2d15f9010c725272015b4 | |
parent | 63315ccecd2d398cbbadd1767fd29be593e7a73a (diff) |
Provide more generic string in testAssert of some TCshavana
CLI test case were failing on RHEL systems as the error string
on Fedora19 and RHEL6.4 systems are different however a more gereric
string is there on both of the systems and this is about that only.
It replaces the 'ECONNREFUSED' & 'No address associated with hostname'
with 'Check that the admin_url is valid'.Now the TCs pass on both of
the OS.
Change-Id: I3ba0f169dddf29368f0adb74027256dcdc894b89
Signed-off-by: Pushpesh Sharma <psharma@redhat.com>
Reviewed-on: http://review.gluster.org/6719
Reviewed-by: Luis Pabon <lpabon@redhat.com>
Tested-by: Luis Pabon <lpabon@redhat.com>
-rw-r--r-- | test/functional_auth/gswauth/test_gswauth_cli.py | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/test/functional_auth/gswauth/test_gswauth_cli.py b/test/functional_auth/gswauth/test_gswauth_cli.py index 29a48b3..418c445 100644 --- a/test/functional_auth/gswauth/test_gswauth_cli.py +++ b/test/functional_auth/gswauth/test_gswauth_cli.py @@ -126,11 +126,11 @@ class TestSwauthPrep(unittest.TestCase): in output,True, 'Invalid swauth-prep request accepted: '+output) (status,output)=Utils.swauthPrep(authurl='http://127.0.0.1:80/auth/') - self.assertEqual('ECONNREFUSED' in output, True, + self.assertEqual('Check that the admin_url is valid' in output, True, 'Invalid swauth-prep request accepted(wrong admin-url provided): %s' % output) (status,output)=Utils.swauthPrep(authurl='http://127.0.1:80/auth/') - self.assertEqual('No address associated with hostname' in output, True, + self.assertEqual('Check that the admin_url is valid' in output, True, 'Invalid swauth-prep request accepted(wrong admin-url provided): %s' % output) #TODO:More cases for invalid url and admin user @@ -187,11 +187,11 @@ class TestAccount(unittest.TestCase): in output,True, 'Invalid account creation request accepted: '+output) (status,output)=Utils.addAccount('test2', authurl='http://127.0.0.1:80/auth/') - self.assertEqual('ECONNREFUSED' in output, True, + self.assertEqual('Check that the admin_url is valid' in output, True, 'Invalid account creation request accepted(wrong admin-url provided): %s' % output) (status,output)=Utils.addAccount('test2', authurl='http://127.0.1:80/auth/') - self.assertEqual('No address associated with hostname' in output, True, + self.assertEqual('Check that the admin_url is valid' in output, True, 'Invalid account creation request accepted(wrong admin-url provided): %s' % output) def testAddAccountNonSuperAdminUsers(self): @@ -246,11 +246,11 @@ class TestAccount(unittest.TestCase): self.assertEqual('Delete account failed: 404 Not Found: Account accountdoesnotexist does not exist' in output,True, 'account deletion failed for test account'+output) (status,output)=Utils.deleteAccount('test3', authurl='http://127.0.0.1:80/auth/') - self.assertEqual('ECONNREFUSED' in output, True, + self.assertEqual('Check that the admin_url is valid' in output, True, 'Invalid deletion request accepted(wrong admin-url provided): %s' % output) (status,output)=Utils.deleteAccount('test3', authurl='http://127.0.1:80/auth/') - self.assertEqual('No address associated with hostname' in output, True, + self.assertEqual('Check that the admin_url is valid' in output, True, 'Invalid deletion request accepted(wrong admin-url provided): %s' % output) def testDeleteAccountNonSuperAdminUsers(self): @@ -287,11 +287,11 @@ class TestAccount(unittest.TestCase): 'swauth-list failed:\n%s' % output) (status,output)=Utils.listAccounts(authurl='http://127.0.0.1:80/auth/') - self.assertEqual('ECONNREFUSED' in output, True, + self.assertEqual('Check that the admin_url is valid' in output, True, 'Invalid list request accepted(wrong admin-url provided): %s' % output) (status,output)=Utils.listAccounts(authurl='http://127.0.1:80/auth/') - self.assertEqual('No address associated with hostname' in output, True, + self.assertEqual('Check that the admin_url is valid' in output, True, 'Invalid list request accepted(wrong admin-url provided): %s' % output) (status,output)=Utils.listAccounts('-j') @@ -379,11 +379,11 @@ class TestUser(unittest.TestCase): self.assertEqual(status, 0, 'User creation request failed, where accountdoesnotexist: '+output) (status,output)=Utils.addAdminUser('test', 'admin2', 'adminpwd', authurl='http://127.0.0.1:80/auth/') - self.assertEqual('ECONNREFUSED' in output, True, + self.assertEqual('Check that the admin_url is valid' in output, True, 'Invalid add user request accepted(wrong admin-url provided): %s' % output) (status,output)=Utils.addAdminUser('test', 'admin2', 'adminpwd', authurl='http://127.0.1:80/auth/') - self.assertEqual('No address associated with hostname' in output, True, + self.assertEqual('Check that the admin_url is valid' in output, True, 'Invalid add user request accepted(wrong admin-url provided): %s' % output) def testAddUserNonSuperAdminUsers (self): @@ -500,11 +500,11 @@ class TestUser(unittest.TestCase): self.assertNotEqual(status, 0, 'Invalid user deletion request accepted, accountdoesnotexist:'+output) #TODO:more testcases? (status,output)=Utils.deleteUser('test', 'admin2', authurl='http://127.0.0.1:80/auth/') - self.assertEqual('ECONNREFUSED' in output, True, + self.assertEqual('Check that the admin_url is valid' in output, True, 'Invalid delete user request accepted(wrong admin-url provided): %s' % output) (status,output)=Utils.deleteUser('test', 'admin2', authurl='http://127.0.1:80/auth/') - self.assertEqual('No address associated with hostname' in output, True, + self.assertEqual('Check that the admin_url is valid' in output, True, 'Invalid delete user request accepted(wrong admin-url provided): %s' % output) |