misc: remove some create_environment calls from tests
This commit is contained in:
parent
2cac15452c
commit
d44d6b216e
|
@ -289,16 +289,26 @@ def test_backoffice_forms(pub):
|
|||
|
||||
def test_backoffice_management_css_class(pub):
|
||||
create_superuser(pub)
|
||||
create_environment(pub)
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = []
|
||||
formdef.store()
|
||||
app = login(get_app(pub))
|
||||
resp = app.get('/backoffice/management/form-title/')
|
||||
assert resp.pyquery.find('body.section-management')
|
||||
|
||||
|
||||
def test_backoffice_form_access_forbidden(pub):
|
||||
create_environment(pub, set_receiver=False)
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = []
|
||||
formdef.store()
|
||||
user = create_user(pub)
|
||||
formdata = FormDef.get_by_urlname('form-title').data_class().select()[0]
|
||||
formdata = formdef.data_class()()
|
||||
formdata.just_created()
|
||||
formdata.store()
|
||||
resp = login(get_app(pub)).get(formdata.get_backoffice_url(), status=403)
|
||||
assert 'Access Forbidden' in resp.text
|
||||
assert 'data-gadjo="true"' in resp.text # backoffice style
|
||||
|
@ -496,12 +506,19 @@ def test_backoffice_listing_fts(pub):
|
|||
|
||||
def test_backoffice_legacy_urls(pub):
|
||||
create_superuser(pub)
|
||||
create_environment(pub)
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = []
|
||||
formdef.store()
|
||||
formdata = formdef.data_class()()
|
||||
formdata.just_created()
|
||||
formdata.store()
|
||||
app = login(get_app(pub))
|
||||
resp = app.get('/backoffice/form-title/')
|
||||
assert resp.location == 'http://example.net/backoffice/management/form-title/'
|
||||
resp = app.get('/backoffice/form-title/1/')
|
||||
assert resp.location == 'http://example.net/backoffice/management/form-title/1/'
|
||||
resp = app.get('/backoffice/form-title/%s/' % formdata.id)
|
||||
assert resp.location == 'http://example.net/backoffice/management/form-title/%s/' % formdata.id
|
||||
resp = app.get('/backoffice/form-title/listing/?bla')
|
||||
assert resp.location == 'http://example.net/backoffice/management/form-title/listing/?bla'
|
||||
resp = app.get('/backoffice/form-title/listing/foo?bla')
|
||||
|
@ -511,9 +528,12 @@ def test_backoffice_legacy_urls(pub):
|
|||
|
||||
def test_backoffice_form_category_permissions(pub):
|
||||
user = create_user(pub)
|
||||
create_environment(pub)
|
||||
|
||||
formdef = FormDef.get_by_urlname('form-title')
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = []
|
||||
formdef.workflow_roles = {'_receiver': 1}
|
||||
formdef.store()
|
||||
|
||||
app = login(get_app(pub))
|
||||
resp = app.get('/backoffice/management/form-title/')
|
||||
|
@ -928,7 +948,6 @@ def test_backoffice_handling(pub):
|
|||
|
||||
def test_backoffice_handling_global_action(pub):
|
||||
create_user(pub)
|
||||
create_environment(pub)
|
||||
|
||||
formdef = FormDef()
|
||||
formdef.name = 'test global action'
|
||||
|
@ -965,7 +984,6 @@ def test_backoffice_handling_global_action(pub):
|
|||
|
||||
def test_backoffice_global_remove_action(pub):
|
||||
create_user(pub)
|
||||
create_environment(pub)
|
||||
|
||||
formdef = FormDef()
|
||||
formdef.name = 'test global remove'
|
||||
|
@ -999,7 +1017,6 @@ def test_backoffice_global_remove_action(pub):
|
|||
|
||||
def test_backoffice_global_action_jump_to_current_status(pub):
|
||||
create_user(pub)
|
||||
create_environment(pub)
|
||||
|
||||
formdef = FormDef()
|
||||
formdef.name = 'test jump to current status'
|
||||
|
@ -2079,7 +2096,12 @@ def test_global_listing(pub):
|
|||
|
||||
def test_global_listing_parameters_from_query_string(pub):
|
||||
create_user(pub)
|
||||
create_environment(pub)
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = []
|
||||
formdef.workflow_roles = {'_receiver': 1}
|
||||
formdef.store()
|
||||
app = login(get_app(pub))
|
||||
resp = app.get('/backoffice/management/listing')
|
||||
assert resp.forms['listing-settings']['status'].value == 'waiting'
|
||||
|
@ -2136,7 +2158,6 @@ def test_global_listing_user_label(pub):
|
|||
|
||||
def test_management_views_with_no_formdefs(pub):
|
||||
create_user(pub)
|
||||
create_environment(pub)
|
||||
FormDef.wipe()
|
||||
|
||||
from wcs.sql import drop_global_views, get_connection_and_cursor
|
||||
|
@ -2329,12 +2350,16 @@ def test_global_map(pub):
|
|||
create_user(pub)
|
||||
create_environment(pub)
|
||||
|
||||
formdef = FormDef.get_by_urlname('form-title')
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = []
|
||||
formdef.geolocations = {'base': 'Geolocafoobar'}
|
||||
formdef.store()
|
||||
for formdata in formdef.data_class().select():
|
||||
formdata.geolocations = {'base': {'lat': 48.83, 'lon': 2.32}}
|
||||
formdata.store()
|
||||
formdata = formdef.data_class()()
|
||||
formdata.just_created()
|
||||
formdata.geolocations = {'base': {'lat': 48.83, 'lon': 2.32}}
|
||||
formdata.store()
|
||||
|
||||
app = login(get_app(pub))
|
||||
resp = app.get('/backoffice/management/listing')
|
||||
|
@ -2354,12 +2379,19 @@ def test_global_map(pub):
|
|||
|
||||
def test_formdata_lookup(pub):
|
||||
create_user(pub)
|
||||
create_environment(pub, set_receiver=False)
|
||||
|
||||
formdef = FormDef.get_by_urlname('form-title')
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = []
|
||||
formdef.enable_tracking_codes = True
|
||||
formdef.store()
|
||||
formdata, formdata2 = formdef.data_class().select(order_by='id')[:2]
|
||||
formdata = formdef.data_class()()
|
||||
formdata.just_created()
|
||||
formdata.store()
|
||||
formdata2 = formdef.data_class()()
|
||||
formdata2.just_created()
|
||||
formdata2.store()
|
||||
code = pub.tracking_code_class()
|
||||
code.formdata = formdata
|
||||
|
||||
|
@ -2562,16 +2594,28 @@ def test_count_open(pub):
|
|||
|
||||
def test_count_backoffice_drafts(pub):
|
||||
user = create_user(pub)
|
||||
create_environment(pub)
|
||||
|
||||
formdef = FormDef.get_by_urlname('form-title')
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = [
|
||||
fields.StringField(
|
||||
id='1', label='1st field', type='string', display_locations=['validation', 'summary', 'listings']
|
||||
),
|
||||
]
|
||||
formdef.backoffice_submission_roles = user.roles[:]
|
||||
formdef.store()
|
||||
|
||||
for i in range(3):
|
||||
formdata = formdef.data_class()()
|
||||
formdata.data = {'1': 'foo'}
|
||||
formdata.just_created()
|
||||
formdata.store()
|
||||
|
||||
resp = login(get_app(pub)).get('/backoffice/submission/count')
|
||||
assert resp.json['count'] == 0
|
||||
|
||||
formdata1, formdata2, formdata3 = formdef.data_class().select()[:3]
|
||||
formdata1, formdata2, formdata3 = formdef.data_class().select()
|
||||
for formdata in (formdata1, formdata2, formdata3):
|
||||
formdata.status = 'draft'
|
||||
formdata.store()
|
||||
|
@ -2596,8 +2640,8 @@ def test_count_backoffice_drafts(pub):
|
|||
|
||||
|
||||
def test_menu_json(pub):
|
||||
FormDef.wipe()
|
||||
create_user(pub)
|
||||
create_environment(pub)
|
||||
resp = login(get_app(pub)).get('/backoffice/menu.json')
|
||||
menu_json_str = resp.text
|
||||
assert len(resp.json) == 1
|
||||
|
@ -2627,7 +2671,16 @@ def test_backoffice_resume_folded(pub):
|
|||
|
||||
def test_backoffice_backoffice_submission_in_listings(pub):
|
||||
create_superuser(pub)
|
||||
create_environment(pub)
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = []
|
||||
formdef.workflow_roles = {'_receiver': 1}
|
||||
formdef.store()
|
||||
formdata = formdef.data_class()()
|
||||
formdata.just_created()
|
||||
formdata.jump_status('new')
|
||||
formdata.store()
|
||||
|
||||
app = login(get_app(pub))
|
||||
resp = app.get('/backoffice/management/form-title/')
|
||||
|
@ -2644,7 +2697,16 @@ def test_backoffice_backoffice_submission_in_listings(pub):
|
|||
|
||||
def test_backoffice_backoffice_submission_in_global_listing(pub):
|
||||
create_superuser(pub)
|
||||
create_environment(pub)
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = []
|
||||
formdef.workflow_roles = {'_receiver': 1}
|
||||
formdef.store()
|
||||
formdata = formdef.data_class()()
|
||||
formdata.just_created()
|
||||
formdata.jump_status('new')
|
||||
formdata.store()
|
||||
|
||||
app = login(get_app(pub))
|
||||
resp = app.get('/backoffice/management/listing?limit=100')
|
||||
|
@ -2663,7 +2725,16 @@ def test_backoffice_backoffice_submission_in_global_listing(pub):
|
|||
|
||||
def test_backoffice_advisory_lock(pub):
|
||||
create_superuser(pub)
|
||||
create_environment(pub)
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = []
|
||||
formdef.workflow_roles = {'_receiver': 1}
|
||||
formdef.store()
|
||||
formdata = formdef.data_class()()
|
||||
formdata.just_created()
|
||||
formdata.jump_status('new')
|
||||
formdata.store()
|
||||
|
||||
second_user = pub.user_class(name='foobar')
|
||||
second_user.roles = pub.role_class.keys()
|
||||
|
@ -2808,7 +2879,6 @@ def test_backoffice_advisory_lock_related_formdatas(pub):
|
|||
|
||||
def test_backoffice_resubmit(pub):
|
||||
user = create_user(pub)
|
||||
create_environment(pub)
|
||||
|
||||
wf = Workflow(name='resubmit')
|
||||
st1 = wf.add_status('Status1')
|
||||
|
@ -2825,8 +2895,13 @@ def test_backoffice_resubmit(pub):
|
|||
|
||||
wf.store()
|
||||
|
||||
formdef = FormDef.get_by_urlname('form-title')
|
||||
formdef.fields[0].varname = 'foo'
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = [
|
||||
fields.StringField(id='1', label='1st field', type='string', varname='foo'),
|
||||
]
|
||||
formdef.workflow_roles = {'_receiver': 1}
|
||||
formdef.workflow_id = wf.id
|
||||
formdef.store()
|
||||
|
||||
|
@ -3496,7 +3571,6 @@ def test_workflow_jump_previous_auto(pub):
|
|||
|
||||
def test_backoffice_fields(pub):
|
||||
create_user(pub)
|
||||
create_environment(pub)
|
||||
|
||||
wf = Workflow(name='bo fields')
|
||||
wf.backoffice_fields_formdef = WorkflowBackofficeFieldsFormDef(wf)
|
||||
|
@ -3508,7 +3582,11 @@ def test_backoffice_fields(pub):
|
|||
wf.add_status('Status1')
|
||||
wf.store()
|
||||
|
||||
formdef = FormDef.get_by_urlname('form-title')
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = []
|
||||
formdef.workflow_roles = {'_receiver': 1}
|
||||
formdef.workflow_id = wf.id
|
||||
formdef.store()
|
||||
|
||||
|
@ -3647,8 +3725,8 @@ def test_backoffice_logged_errors(pub):
|
|||
|
||||
def test_backoffice_formdata_named_wscall(http_requests, pub):
|
||||
user = create_user(pub)
|
||||
create_environment(pub)
|
||||
|
||||
FormDef.wipe()
|
||||
NamedWsCall.wipe()
|
||||
|
||||
wscall = NamedWsCall()
|
||||
|
@ -3710,8 +3788,8 @@ query_string_allowed_vars = foo,bar
|
|||
'''
|
||||
)
|
||||
|
||||
FormDef.wipe()
|
||||
user = create_user(pub)
|
||||
create_environment(pub)
|
||||
|
||||
formdef = FormDef()
|
||||
formdef.name = 'test'
|
||||
|
@ -3743,7 +3821,6 @@ query_string_allowed_vars = foo,bar
|
|||
|
||||
def test_backoffice_display_message(pub):
|
||||
user = create_user(pub)
|
||||
create_environment(pub)
|
||||
|
||||
workflow = Workflow(name='test')
|
||||
st1 = workflow.add_status('Status1', 'st1')
|
||||
|
@ -3762,11 +3839,17 @@ def test_backoffice_display_message(pub):
|
|||
|
||||
workflow.store()
|
||||
|
||||
formdef = FormDef.get_by_urlname('form-title')
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = []
|
||||
formdef.workflow_roles = {'_receiver': 1}
|
||||
formdef.workflow = workflow
|
||||
formdef.store()
|
||||
|
||||
formdata = formdef.data_class().select()[0]
|
||||
formdata = formdef.data_class()()
|
||||
formdata.data = {}
|
||||
formdata.just_created()
|
||||
formdata.jump_status('st1')
|
||||
formdata.store()
|
||||
|
||||
|
@ -3862,7 +3945,6 @@ def test_backoffice_forms_condition_on_button(pub):
|
|||
|
||||
def test_workflow_comment_required(pub):
|
||||
create_user(pub)
|
||||
create_environment(pub)
|
||||
|
||||
wf = Workflow(name='blah')
|
||||
st1 = wf.add_status('Comment')
|
||||
|
@ -3871,8 +3953,11 @@ def test_workflow_comment_required(pub):
|
|||
commentable.by = ['_submitter', '_receiver']
|
||||
wf.store()
|
||||
|
||||
formdef = FormDef.get_by_urlname('form-title')
|
||||
formdef.data_class().wipe()
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = []
|
||||
formdef.workflow_roles = {'_receiver': 1}
|
||||
formdef.workflow = wf
|
||||
formdef.store()
|
||||
|
||||
|
@ -4354,7 +4439,6 @@ def test_backoffice_create_carddata_from_formdata(pub):
|
|||
|
||||
def test_backoffice_after_submit_location(pub):
|
||||
create_superuser(pub)
|
||||
create_environment(pub)
|
||||
|
||||
workflow = Workflow(name='test')
|
||||
st1 = workflow.add_status('Status1', 'st1')
|
||||
|
@ -4365,14 +4449,18 @@ def test_backoffice_after_submit_location(pub):
|
|||
|
||||
workflow.store()
|
||||
|
||||
formdef = FormDef.get_by_urlname('form-title')
|
||||
formdef.store()
|
||||
FormDef.wipe()
|
||||
formdef = FormDef()
|
||||
formdef.name = 'form title'
|
||||
formdef.fields = []
|
||||
formdef.workflow_roles = {'_receiver': 1}
|
||||
formdef.workflow_id = workflow.id
|
||||
formdef.store()
|
||||
|
||||
for formdata in formdef.data_class().select():
|
||||
formdata.status = 'wf-%s' % st1.id
|
||||
formdata.store()
|
||||
formdata = formdef.data_class()()
|
||||
formdata.just_created()
|
||||
formdata.status = 'wf-%s' % st1.id
|
||||
formdata.store()
|
||||
|
||||
app = login(get_app(pub))
|
||||
|
||||
|
@ -4409,7 +4497,6 @@ def test_backoffice_http_basic_auth(pub):
|
|||
|
||||
def test_backoffice_dispatch_lose_access(pub):
|
||||
user = create_user(pub)
|
||||
create_environment(pub)
|
||||
|
||||
role1 = pub.role_class(name='xxx1')
|
||||
role1.store()
|
||||
|
@ -4452,7 +4539,6 @@ def test_backoffice_dispatch_lose_access(pub):
|
|||
|
||||
def test_backoffice_dispatch_multi(pub):
|
||||
user = create_user(pub)
|
||||
create_environment(pub)
|
||||
|
||||
role1 = pub.role_class(name='xxx1')
|
||||
role1.store()
|
||||
|
@ -4511,7 +4597,6 @@ def test_backoffice_dispatch_single_user(pub, user_template):
|
|||
user = create_user(pub)
|
||||
user.name_identifiers = ['0123456789']
|
||||
user.store()
|
||||
create_environment(pub)
|
||||
|
||||
formdef = FormDef()
|
||||
formdef.name = 'test dispatch user'
|
||||
|
|
Loading…
Reference in New Issue