remove some more stubouts and fakes
This commit is contained in:
@@ -16,7 +16,6 @@
|
||||
|
||||
import json
|
||||
|
||||
import stubout
|
||||
import webob
|
||||
|
||||
from nova import flags
|
||||
@@ -41,7 +40,7 @@ def fake_admin_check(self, req):
|
||||
class AccountsTest(test.TestCase):
|
||||
def setUp(self):
|
||||
super(AccountsTest, self).setUp()
|
||||
self.stubs = stubout.StubOutForTesting()
|
||||
self.flags(allow_admin_api=True)
|
||||
self.stubs.Set(accounts.Controller, '__init__',
|
||||
fake_init)
|
||||
self.stubs.Set(accounts.Controller, '_check_admin',
|
||||
@@ -52,8 +51,6 @@ class AccountsTest(test.TestCase):
|
||||
fakes.stub_out_rate_limiting(self.stubs)
|
||||
fakes.stub_out_auth(self.stubs)
|
||||
|
||||
self.allow_admin = FLAGS.allow_admin_api
|
||||
FLAGS.allow_admin_api = True
|
||||
fakemgr = fakes.FakeAuthManager()
|
||||
joeuser = User('id1', 'guy1', 'acc1', 'secret1', False)
|
||||
superuser = User('id2', 'guy2', 'acc2', 'secret2', True)
|
||||
@@ -62,11 +59,6 @@ class AccountsTest(test.TestCase):
|
||||
fakemgr.create_project('test1', joeuser)
|
||||
fakemgr.create_project('test2', superuser)
|
||||
|
||||
def tearDown(self):
|
||||
self.stubs.UnsetAll()
|
||||
FLAGS.allow_admin_api = self.allow_admin
|
||||
super(AccountsTest, self).tearDown()
|
||||
|
||||
def test_get_account(self):
|
||||
req = webob.Request.blank('/v1.0/accounts/test1')
|
||||
res = req.get_response(fakes.wsgi_app())
|
||||
|
||||
@@ -17,7 +17,6 @@
|
||||
|
||||
import datetime
|
||||
|
||||
import stubout
|
||||
import webob
|
||||
import webob.dec
|
||||
|
||||
@@ -43,7 +42,6 @@ class Test(test.TestCase):
|
||||
fakes.stub_out_networking(self.stubs)
|
||||
|
||||
def tearDown(self):
|
||||
self.stubs.UnsetAll()
|
||||
fakes.fake_data_store = {}
|
||||
super(Test, self).tearDown()
|
||||
|
||||
@@ -237,7 +235,6 @@ class TestFunctional(test.TestCase):
|
||||
class TestLimiter(test.TestCase):
|
||||
def setUp(self):
|
||||
super(TestLimiter, self).setUp()
|
||||
self.stubs = stubout.StubOutForTesting()
|
||||
self.stubs.Set(nova.api.openstack.auth.AuthMiddleware,
|
||||
'__init__', fakes.fake_auth_init)
|
||||
self.stubs.Set(context, 'RequestContext', fakes.FakeRequestContext)
|
||||
@@ -246,7 +243,6 @@ class TestLimiter(test.TestCase):
|
||||
fakes.stub_out_networking(self.stubs)
|
||||
|
||||
def tearDown(self):
|
||||
self.stubs.UnsetAll()
|
||||
fakes.fake_data_store = {}
|
||||
super(TestLimiter, self).tearDown()
|
||||
|
||||
|
||||
@@ -15,7 +15,6 @@
|
||||
|
||||
import json
|
||||
|
||||
import stubout
|
||||
import webob
|
||||
|
||||
from nova import flags
|
||||
@@ -41,7 +40,7 @@ def fake_admin_check(self, req):
|
||||
class UsersTest(test.TestCase):
|
||||
def setUp(self):
|
||||
super(UsersTest, self).setUp()
|
||||
self.stubs = stubout.StubOutForTesting()
|
||||
self.flags(allow_admin_api=True)
|
||||
self.stubs.Set(users.Controller, '__init__',
|
||||
fake_init)
|
||||
self.stubs.Set(users.Controller, '_check_admin',
|
||||
@@ -58,16 +57,10 @@ class UsersTest(test.TestCase):
|
||||
fakes.stub_out_auth(self.stubs)
|
||||
|
||||
self.allow_admin = FLAGS.allow_admin_api
|
||||
FLAGS.allow_admin_api = True
|
||||
fakemgr = fakes.FakeAuthManager()
|
||||
fakemgr.add_user(User('id1', 'guy1', 'acc1', 'secret1', False))
|
||||
fakemgr.add_user(User('id2', 'guy2', 'acc2', 'secret2', True))
|
||||
|
||||
def tearDown(self):
|
||||
self.stubs.UnsetAll()
|
||||
FLAGS.allow_admin_api = self.allow_admin
|
||||
super(UsersTest, self).tearDown()
|
||||
|
||||
def test_get_user_list(self):
|
||||
req = webob.Request.blank('/v1.0/users')
|
||||
res = req.get_response(fakes.wsgi_app())
|
||||
|
||||
Reference in New Issue
Block a user