Browse Source

Merge pull request #483 from tracim/fix/223/fix_beautifulsoup4_warning

Damien Accorsi 7 years ago
parent
commit
fb8ebed334
No account linked to committer's email

+ 1 - 1
install/requirements.txt View File

@@ -17,7 +17,7 @@ WebTest==1.4.2
17 17
 alembic==0.8.4
18 18
 argparse==1.2.1
19 19
 backlash==0.0.7
20
-beautifulsoup4==4.4.0
20
+beautifulsoup4==4.6.0
21 21
 caldav==0.4.0
22 22
 cliff==2.9.1
23 23
 cmd2==0.6.9

+ 3 - 3
tracim/tracim/tests/functional/test_ldap_restrictions.py View File

@@ -26,7 +26,7 @@ class TestAuthentication(LDAPTest, TracimTestController):
26 26
         home = self.app.get('/home/',)
27 27
 
28 28
         # HTML button is not here
29
-        eq_(None, BeautifulSoup(home.body).find(attrs={'class': 'change-password-btn'}))
29
+        eq_(None, BeautifulSoup(home.body, 'html.parser').find(attrs={'class': 'change-password-btn'}))
30 30
 
31 31
         # If we force passwd update, we got 403
32 32
         try_post_passwd = self.app.post(
@@ -51,12 +51,12 @@ class TestAuthentication(LDAPTest, TracimTestController):
51 51
         edit = self.app.get('/user/5/edit')
52 52
 
53 53
         # email input field is disabled
54
-        email_input = BeautifulSoup(edit.body).find(attrs={'id': 'email'})
54
+        email_input = BeautifulSoup(edit.body, 'html.parser').find(attrs={'id': 'email'})
55 55
         ok_('readonly' in email_input.attrs)
56 56
         eq_(email_input.attrs['readonly'], "readonly")
57 57
 
58 58
         # Name is not (see attributes configuration of LDAP fixtures)
59
-        name_input = BeautifulSoup(edit.body).find(attrs={'id': 'name'})
59
+        name_input = BeautifulSoup(edit.body, 'html.parser').find(attrs={'id': 'name'})
60 60
         ok_('readonly' not in name_input.attrs)
61 61
 
62 62
         # If we force edit of user, "email" field will be not updated

+ 1 - 1
tracim/tracim/tests/functional/test_root.py View File

@@ -30,7 +30,7 @@ class TestRootController(TestController):
30 30
         msg = 'copyright © 2013 - {} tracim project.'.format(h.current_year())
31 31
         ok_(msg in response)
32 32
 
33
-        forms = BeautifulSoup(response.body).find_all('form')
33
+        forms = BeautifulSoup(response.body, 'html.parser').find_all('form')
34 34
         print('FORMS = ',forms)
35 35
         eq_(1, len(forms))
36 36
         eq_('w-login-form', forms[0].get('id'))