Skip to content

Commit a90fc6c

Browse files
committed
Fix test which should have been skipped when QueryableList is not installed, but was not getting skipped.
1 parent 10d7d18 commit a90fc6c

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

tests/AdvancedHTMLParserTests/test_General.py

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -253,9 +253,15 @@ def test_getForms(self):
253253
assert issubclass(formEms.__class__, AdvancedHTMLParser.TagCollection) , 'Expected result of document.forms to be a TagCollection'
254254

255255
try:
256-
assert formEms.filter(id='form1').all() == [formEms[0]] , 'Expected filtering to work on TagCollection returned from document.forms'
256+
import QueryableList
257+
hasQueryableList = True
257258
except ImportError:
258-
sys.stderr.write('WARNING: .filter is disabled via ImportError. QueryableList not installed?\n\n')
259+
hasQueryableList = False
260+
sys.stderr.write('WARNING: .filter is disabled via ImportError. Skipping related test. QueryableList not installed?\n\n')
261+
262+
if hasQueryableList:
263+
assert formEms.filter(id='form1').all() == [formEms[0]] , 'Expected filtering to work on TagCollection returned from document.forms'
264+
259265

260266
def test_doctype(self):
261267
'''

0 commit comments

Comments
 (0)