Projet

Général

Profil

0002-misc-simplify-lingo-manager-tests.patch

Lauréline Guérin, 01 octobre 2021 10:53

Télécharger (4,33 ko)

Voir les différences:

Subject: [PATCH 2/2] misc: simplify lingo manager tests

 tests/test_lingo_manager.py | 30 +++++++++++++-----------------
 1 file changed, 13 insertions(+), 17 deletions(-)
tests/test_lingo_manager.py
27 27
    return PaymentBackend.objects.create(label='test1', slug='test1', service='dummy', service_options={})
28 28

  
29 29

  
30
@pytest.fixture
31
def regie(payment_backend):
32
    return Regie.objects.create(label='Test', slug='test', payment_backend=payment_backend)
33

  
34

  
30 35
def login(app, username='admin', password='admin'):
31 36
    login_page = app.get('/login/')
32 37
    login_form = login_page.forms[0]
......
44 49

  
45 50

  
46 51
def test_add_regie(app, admin_user, payment_backend):
47
    Regie.objects.all().delete()
48 52
    app = login(app)
49 53
    resp = app.get('/manage/lingo/regies/', status=200)
50 54
    resp = resp.click('New')
......
61 65
    assert regie.is_default is True
62 66

  
63 67

  
64
def test_edit_regie(app, admin_user, payment_backend):
65
    test_add_regie(app, admin_user, payment_backend)
68
def test_edit_regie(app, admin_user, regie):
66 69
    app = login(app)
67 70
    resp = app.get('/manage/lingo/regies/', status=200)
68 71
    resp = resp.click('Test')
......
71 74
    # callback URL is shown
72 75
    assert (
73 76
        '<p><label>Callback URL:</label> http://testserver/lingo/callback-payment-backend/%s/</p>'
74
        % payment_backend.slug
77
        % regie.payment_backend.slug
75 78
        in resp
76 79
    )
77
    regie = Regie.objects.all()[0]
78 80
    assert '<p><label>Return URL:</label> http://testserver/lingo/return/%s/</p>' % regie.pk in resp
79 81

  
80 82
    resp.forms[0]['description'] = 'other description'
......
142 144
    assert regie.transaction_options == {}
143 145

  
144 146

  
145
def test_delete_regie(app, admin_user, payment_backend):
146
    test_add_regie(app, admin_user, payment_backend)
147
def test_delete_regie(app, admin_user, regie):
147 148
    app = login(app)
148 149
    resp = app.get('/manage/lingo/regies/', status=200)
149 150
    resp = resp.click('remove')
......
152 153
    assert Regie.objects.count() == 0
153 154

  
154 155

  
155
def test_add_second_regie(app, admin_user, payment_backend):
156
    test_add_regie(app, admin_user, payment_backend)
157
    regie = Regie.objects.all()[0]
158

  
156
def test_add_second_regie(app, admin_user, payment_backend, regie):
159 157
    app = login(app)
160 158
    resp = app.get('/manage/lingo/regies/', status=200)
161 159
    resp = resp.click('New')
......
171 169
    assert Regie.objects.exclude(id=regie.id)[0].is_default is False
172 170

  
173 171

  
174
def test_download_transaction(app, admin_user, payment_backend):
175
    test_add_regie(app, admin_user, payment_backend)
176
    regie = Regie.objects.filter(slug='test')[0]
172
def test_download_transaction(app, admin_user, regie):
177 173
    user = User.objects.create_user('dimebag', 'dime@bag.pan', 'pwd')
178 174
    user.last_name = 'Darëll'
179 175
    user.first_name = 'Dimebag'
......
562 558

  
563 559
    resp = app.get('/manage/pages/%s/' % page.id, status=200)
564 560
    assert 'lingorecenttransactionscell' in resp.text  # because there's a regie
565
    assert not 'lingo_activeitems' in resp.text  # because there's no webservice
561
    assert 'lingo_activeitems' not in resp.text  # because there's no webservice
566 562

  
567 563
    regie.webservice_url = 'http://example.net/'
568 564
    regie.save()
......
651 647
    assert 'disabled' in resp.form['service'].attrs
652 648

  
653 649
    # deprecated parameters are not shown
654
    assert not 'next_url' in resp.form.fields
655
    assert not 'direct_notification_url' in resp.form.fields
650
    assert 'next_url' not in resp.form.fields
651
    assert 'direct_notification_url' not in resp.form.fields
656 652

  
657 653
    resp.form['label'] = 'label1-modified'
658 654
    resp.form['consider_all_response_signed'] = True
659
-