Functional Tests for gswauth_cli
This test suite plans to tests following gswauth commands:- 1. swauth-prep 2. swauth-add-account 3. swauth-delete-account 4. swauth-add-user 5. swauth-delete-user 6. swauth-set-account-service 7. swauth-cleanup-tokens 8. swauth-list It uses 'commands' python module to execute gswauthcommands, and capture the status and output of commands followed by asserts to verify if the captured status and output is same as expected ones.This initial draft has tests for first 1-5 commands. Change-Id: I8591884e87c6ffd88446aacd04a1db2daab2d189 Signed-off-by: Pushpesh Sharma <psharma@redhat.com> Reviewed-on: http://review.gluster.org/6277 Reviewed-by: Luis Pabon <lpabon@redhat.com> Tested-by: Luis Pabon <lpabon@redhat.com>
This commit is contained in:
Pushpesh Sharma
Luis Pabon
parent
47f401b2ef
commit
3f18f021bd
@ -15,6 +15,10 @@ auth_prefix = /auth/
|
||||
admin_key = gswauthkey
|
||||
admin_user = .super_admin
|
||||
|
||||
# Gluster setup information
|
||||
devices = /mnt/gluster-object
|
||||
gsmetadata_volume = gsmetadata
|
||||
|
||||
# Primary functional test account (needs admin access to the account)
|
||||
account = test
|
||||
username = tester
|
||||
|
||||
246
test/functional_auth/gswauth/test_gswauth_cli.py
Normal file
246
test/functional_auth/gswauth/test_gswauth_cli.py
Normal file
@ -0,0 +1,246 @@
|
||||
#!/usr/bin/python
|
||||
|
||||
# Copyright (c) 2010-2012 OpenStack Foundation
|
||||
#
|
||||
# Licensed under the Apache License, Version 2.0 (the "License");
|
||||
# you may not use this file except in compliance with the License.
|
||||
# You may obtain a copy of the License at
|
||||
#
|
||||
# http://www.apache.org/licenses/LICENSE-2.0
|
||||
#
|
||||
# Unless required by applicable law or agreed to in writing, software
|
||||
# distributed under the License is distributed on an "AS IS" BASIS,
|
||||
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
|
||||
# implied.
|
||||
# See the License for the specific language governing permissions and
|
||||
# limitations under the License.
|
||||
|
||||
import unittest
|
||||
from nose import SkipTest
|
||||
import commands
|
||||
import os
|
||||
from test import get_config
|
||||
|
||||
config = get_config('func_test')
|
||||
|
||||
class Utils:
|
||||
|
||||
@classmethod
|
||||
def addAccount(self,account_name,authurl='http://127.0.0.1:8080/auth/',user=config['admin_user'],key=config['admin_key']):
|
||||
return commands.getstatusoutput('swauth-add-account %s -A %s -U %s -K %s' % (account_name,authurl, user, key))
|
||||
|
||||
@classmethod
|
||||
def deleteAccount(self,account_name,authurl='http://127.0.0.1:8080/auth/',user=config['admin_user'],key=config['admin_key']):
|
||||
return commands.getstatusoutput('swauth-delete-account %s -A %s -U %s -K %s' % (account_name,authurl, user, key))
|
||||
|
||||
@classmethod
|
||||
def swauthPrep(self,authurl='http://127.0.0.1:8080/auth/',user=config['admin_user'],key=config['admin_key']):
|
||||
return commands.getstatusoutput('swauth-prep -A %s -U %s -K %s' % (authurl, user, key))
|
||||
|
||||
@classmethod
|
||||
def addAdminUser(self,account_name,username,password,authurl='http://127.0.0.1:8080/auth/',user=config['admin_user'],key=config['admin_key']):
|
||||
return commands.getstatusoutput('swauth-add-user -a %s %s %s -A %s -U %s -K %s'% (account_name,username,password,authurl, user, key))
|
||||
|
||||
@classmethod
|
||||
def addUser(self,account_name,username,password,authurl='http://127.0.0.1:8080/auth/',user=config['admin_user'],key=config['admin_key']):
|
||||
return commands.getstatusoutput('swauth-add-user %s %s %s -A %s -U %s -K %s'% (account_name,username,password,authurl, user, key))
|
||||
|
||||
@classmethod
|
||||
def addResellerAdminUser(self,account_name,username,password,authurl='http://127.0.0.1:8080/auth/',user=config['admin_user'],key=config['admin_key']):
|
||||
return commands.getstatusoutput('swauth-add-user -r %s %s %s -A %s -U %s -K %s'% (account_name, username, password, authurl, user, key))
|
||||
|
||||
@classmethod
|
||||
def deleteUser(self,account_name,username,authurl='http://127.0.0.1:8080/auth/',user=config['admin_user'],key=config['admin_key']):
|
||||
return commands.getstatusoutput('swauth-delete-user %s %s -A %s -U %s -K %s'% (account_name, username, authurl, user, key))
|
||||
|
||||
@classmethod
|
||||
def cleanAll(self):
|
||||
#TODO:It's a dirty hack,any suggestions?
|
||||
commands.getstatusoutput('rm -rf '+os.path.join(config['devices'], config['gsmetadata_volume'], '*'))
|
||||
commands.getstatusoutput('rm -rf '+os.path.join(config['devices'], config['gsmetadata_volume'], '.*'))
|
||||
|
||||
|
||||
class TestSwauthPrep(unittest.TestCase):
|
||||
|
||||
def setUp(self):
|
||||
pass
|
||||
|
||||
def tearDown(self):
|
||||
Utils.cleanAll()
|
||||
|
||||
def testSwauthPrep(self):
|
||||
(status,output)=Utils.swauthPrep()
|
||||
self.assertEqual(status, 0, 'swuath prep failed with valid credentials'+output)
|
||||
|
||||
(status,output)=Utils.swauthPrep(key='')
|
||||
self.assertEqual('Usage' in output,True, 'Invalid swauth-prep request accepted(no key provided): '+output)
|
||||
|
||||
(status,output)=Utils.swauthPrep(key='notavalidkey')
|
||||
self.assertNotEqual(status, 0, 'Invalid swauth-prep request accepted(wrong key provided):'+output)
|
||||
#TODO:In place of this error message 'Auth subsystem prep failed: 403 Forbidden, Invalid user/key' would be good to have
|
||||
self.assertEqual('Auth subsystem prep failed: 403 Forbidden' in output,True, 'Invalid swauth-prep request accepted: '+output)
|
||||
#TODO:More cases for invalid url and admin user
|
||||
|
||||
|
||||
class TestAccount(unittest.TestCase):
|
||||
|
||||
def setUp(self):
|
||||
(status,output)=Utils.swauthPrep()
|
||||
self.assertEqual(status, 0, 'setup swauth-prep failed'+output)
|
||||
|
||||
def tearDown(self):
|
||||
Utils.cleanAll()
|
||||
|
||||
def setTestDeleteAccountEnv(self):
|
||||
#add some account
|
||||
(status,output)=Utils.addAccount('test')
|
||||
self.assertEqual(status, 0, 'setTestDeleteAccountEnv failed'+output)
|
||||
|
||||
(status,output)=Utils.addAccount('test2')
|
||||
self.assertEqual(status, 0, 'setTestDeleteAccountEnv failed'+output)
|
||||
|
||||
#add some user to this account
|
||||
(status,output) = Utils.addAdminUser('test2','tester','testing')
|
||||
self.assertEqual(status, 0, 'setTestDeleteAccountEnv'+output)
|
||||
|
||||
(status,output) = Utils.addUser('test2','tester2','testing2')
|
||||
self.assertEqual(status, 0, 'setTestDeleteAccountEnv'+output)
|
||||
|
||||
(status,output) = Utils.addResellerAdminUser('test2','tester3','testing3')
|
||||
self.assertEqual(status, 0, 'setTestDeleteAccountEnv'+output)
|
||||
|
||||
def testAddAccount(self):
|
||||
(status,output)=Utils.addAccount('test')
|
||||
self.assertEqual(status, 0, 'account creation failed'+output)
|
||||
|
||||
(status,output)=Utils.addAccount('accountvolumedoesnotexist')
|
||||
#TODO:decide on expected behavior,currently it creates it
|
||||
self.assertEqual(status, 0, 'account creation failed std err was: '+output)
|
||||
|
||||
(status,output)=Utils.addAccount('testnokey',key='')
|
||||
#self.assertEqual(status, 0, 'account creation failed std err was: '+output)
|
||||
self.assertEqual('Usage:' in output, True, 'Invalid account creation request accepted : '+output)
|
||||
|
||||
(status,output)=Utils.addAccount('testinvalidkey',key='invalidkey')
|
||||
#self.assertEqual(status, 0, 'account creation failed std err was: '+output)
|
||||
#assert for better error message 403 Forbidden, Invalid user/key would be good to have
|
||||
self.assertEqual('403 Forbidden' in output,True, 'Invalid account creation request accepted: '+output)
|
||||
#TODO:more cases?
|
||||
|
||||
def testDeleteAccount(self):
|
||||
self.setTestDeleteAccountEnv()
|
||||
|
||||
#valid request to delete an account with no users
|
||||
(status,output)=Utils.deleteAccount('test')
|
||||
self.assertEqual(status, 0, 'account deletion failed for test account'+output)
|
||||
|
||||
#Invalid request to delete an account with users
|
||||
(status,output)=Utils.deleteAccount('test2')
|
||||
self.assertNotEqual(status, 0, 'account deletion failed for test2 account'+output)
|
||||
#TODO:decide on expected behavior 'there are active users,users needs to be deleted first'?
|
||||
self.assertEqual('Conflict' in output,True, 'account deletion failed for test account'+output)
|
||||
|
||||
#delete all users in above account and then try again
|
||||
(status,output) = Utils.deleteUser('test2','tester')
|
||||
self.assertEqual(status, 0, 'setTestDeleteAccountEnv'+output)
|
||||
|
||||
(status,output) = Utils.deleteUser('test2','tester2')
|
||||
self.assertEqual(status, 0, 'setTestDeleteAccountEnv'+output)
|
||||
|
||||
(status,output) = Utils.deleteUser('test2','tester3')
|
||||
self.assertEqual(status, 0, 'setTestDeleteAccountEnv'+output)
|
||||
|
||||
(status,output)=Utils.deleteAccount('test2')
|
||||
self.assertEqual(status, 0, 'account deletion failed for test2 account'+output)
|
||||
|
||||
(status,output)=Utils.deleteAccount('accountdoesnotexist')
|
||||
#TODO:decide on expected behavior
|
||||
self.assertNotEqual(status, 0, 'account deletion failed for accountdoesnotexist'+output)
|
||||
#TODO:more cases
|
||||
|
||||
|
||||
class TestUser(unittest.TestCase):
|
||||
|
||||
def setUp(self):
|
||||
(status,output)=Utils.swauthPrep()
|
||||
self.assertEqual(status, 0, 'setup swauth-prep failed'+output)
|
||||
|
||||
def tearDown(self):
|
||||
Utils.cleanAll()
|
||||
|
||||
def setTestaddAdminUserEnv(self):
|
||||
#add test account
|
||||
(status,output)=Utils.addAccount('test')
|
||||
self.assertEqual(status, 0, 'setTestaddAdminUserEnv (add test account) failed'+output)
|
||||
|
||||
def setTestDeleteUserEnv(self):
|
||||
#add test account
|
||||
(status,output)=Utils.addAccount('test')
|
||||
self.assertEqual(status, 0, 'setTestaddAdminUserEnv (add test account) failed'+output)
|
||||
|
||||
(status,output) = Utils.addAdminUser('test','testadminuser','testadminuser')
|
||||
self.assertEqual(status, 0, 'user addition failed'+output)
|
||||
|
||||
(status,output) = Utils.addUser('test','testuser','testuser')
|
||||
self.assertEqual(status, 0, 'user addition failed'+output)
|
||||
|
||||
(status,output) = Utils.addResellerAdminUser('test','testreselleradminuser','testreselleradminuser')
|
||||
self.assertEqual(status, 0, 'user addition failed'+output)
|
||||
|
||||
def testaddAdminUser(self):
|
||||
#set the env for test
|
||||
self.setTestaddAdminUserEnv()
|
||||
|
||||
(status,output) = Utils.addAdminUser('test','testadminuser','testadminuser')
|
||||
self.assertEqual(status, 0, 'user addition failed'+output)
|
||||
|
||||
(status,output) = Utils.addUser('test','testuser','testuser')
|
||||
self.assertEqual(status, 0, 'user addition failed'+output)
|
||||
|
||||
(status,output) = Utils.addResellerAdminUser('test','testreselleradminuser','testreselleradminuser')
|
||||
self.assertEqual(status, 0, 'user addition failed'+output)
|
||||
|
||||
(status,output) = Utils.addAdminUser('test', '', '')
|
||||
self.assertEqual('Usage:' in output, True, 'Invalid user creation request accepted: '+output)
|
||||
|
||||
(status,output) = Utils.addAdminUser('test', 'testcli', '')
|
||||
self.assertEqual('Usage:' in output, True, 'Invalid user creation request accepted'+output)
|
||||
|
||||
(status,output) = Utils.addAdminUser('test', '', 'testcli')
|
||||
self.assertEqual('Usage:' in output, True, 'Invalid user creation request accepted'+output)
|
||||
|
||||
(status,output) = Utils.addAdminUser('accountdoesnotexist', 'testcli', 'testcli')
|
||||
#TODO: decide on behavior,below is just place holder, right now it accepts this request and create both user and account
|
||||
self.assertEqual(status, 0, 'Invalid user creation request accepted,accountdoesnotexist: '+output)
|
||||
#TODO: more test cases?
|
||||
|
||||
def testDeleteUser(self):
|
||||
#set the env for test
|
||||
self.setTestDeleteUserEnv()
|
||||
|
||||
(status,output) = Utils.deleteUser('test','testadminuser')
|
||||
self.assertEqual(status, 0, 'valid user deletion failed:'+output)
|
||||
|
||||
(status,output) = Utils.deleteUser('test','testuser')
|
||||
self.assertEqual(status, 0, 'valid user deletion failed:'+output)
|
||||
|
||||
(status,output) = Utils.deleteUser('test','testreselleradminuser')
|
||||
self.assertEqual(status, 0, 'valid user deletion failed:'+output)
|
||||
|
||||
(status,output) = Utils.deleteUser('test', '')
|
||||
self.assertEqual('Usage:' in output, True, 'Invalid user deletion request accepted : '+output)
|
||||
|
||||
(status,output) = Utils.deleteUser('','testcli')
|
||||
self.assertEqual('Usage:' in output, True, 'Invalid user deletion request accepted : '+output)
|
||||
|
||||
(status,output) = Utils.deleteUser('test', 'userdoesnotexist')
|
||||
self.assertNotEqual(status, 0, 'Invalid user deletion request accepted,userdoesnotexist:'+output)
|
||||
#TODO:decide on expected behavior,current is '404 Not Found'
|
||||
|
||||
(status,output) = Utils.deleteUser('accountisnothere', 'testcli')
|
||||
self.assertNotEqual(status, 0, 'Invalid user deletion request accepted, accountdoesnotexist:'+output)
|
||||
#TODO:decide on expected behavior,current is '404 Not Found'
|
||||
|
||||
#TODO:more testcases?
|
||||
|
||||
|
||||
Loading…
Reference in New Issue
Block a user