10000 Improve unit test for basic multi-page handling · stevejaker/python-readability@816c664 · GitHub
[go: up one dir, main page]
Skip to content

Commit 816c664

Browse files
jcharummitechie
jcharum
authored and
mitechie
committed
Improve unit test for basic multi-page handling
The test now actually asserts something instead of just printing some stuff out for manual inspection. Conflicts: src/readability_lxml/readability.py
1 parent 99d5fc0 commit 816c664

File tree

2 files changed

+124
-2
lines changed

2 files changed

+124
-2
lines changed

‎src/readability_lxml/readability.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,11 @@
99
from collections import namedtuple
1010
from lxml.etree import tostring
1111
from lxml.etree import tounicode
12+
from lxml.html.diff import htmldiff
1213
from lxml.html import document_fromstring
1314
from lxml.html import fragment_fromstring
1415

1516
from cleaners import clean_attributes
16-
17-
1817
from cleaners import html_cleaner
1918
from htmls import build_doc
2019
from htmls import get_body

‎src/tests/regression_test_data/basic-multi-page-expected.html

Lines changed: 123 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,123 @@
1+
<div id="article">
2+
<h1>A Simple Multi-Page Article For Testing</h1>
3+
<p>
4+
Lorem ipsum dolor sit amet, consectetur adipiscing elit. Nulla et
5+
laoreet ligula. Nulla facilisi. Morbi condimentum molestie enim in
6+
fermentum. Phasellus sit amet vehicula turpis. Sed eu dolor tortor,
7+
et accumsan purus. Aliquam velit nisl, facilisis quis suscipit in,
8+
porttitor at lorem. Ut adipiscing suscipit augue, id interdum arcu
9+
ultricies et. Etiam risus sapien, suscipit et ultricies vel,
10+
suscipit posuere velit. Proin est orci, sollicitudin at luctus
11+
feugiat, consectetur a justo. Etiam nec sem vel massa consectetur
12+
vulputate non interdum est. Donec sem dui, ultricies a adipiscing
13+
eu, placerat sed sem.
14+
</p>
15+
<p>
16+
Nunc lacinia varius justo, at lacinia felis ultricies vel. Proin
17+
vestibulum vehicula eleifend. Ut vitae risus eros. Pellentesque
18+
habitant morbi tristique senectus et netus et malesuada fames ac
19+
turpis egestas. In hac habitasse platea dictumst. Vivamus magna
20+
libero, blandit vitae hendrerit porta, dapibus eget eros. Nunc
21+
turpis felis, facilisis eu vestibulum sed, porta a ipsum. Vivamus
22+
est velit, molestie sed molestie quis, tincidunt a diam. Quisque et
23+
neque a ante fermentum tempus in at nunc. Nunc sit amet egestas
24+
nisi.
25+
</p>
26+
<p>
27+
Proin in lacus dolor, sit amet molestie quam. Morbi nisi turpis,
28+
pharetra at consequat tristique, convallis nec turpis. Vestibulum
29+
sit amet magna vitae sem bibendum tincidunt. Maecenas quis tortor
30+
eget velit mollis tempor vel a nisl. Vivamus posuere tristique
31+
ante, cursus rhoncus tortor malesuada eu. Praesent faucibus viverra
32+
orci ac porttitor. Maecenas dui purus, aliquam sed aliquam nec,
33+
dignissim vitae libero. Nunc at mauris et ante accumsan
34+
pellentesque. In placerat pretium suscipit. Phasellus tellus est,
35+
venenatis eu consectetur non, vehicula vel metus. Curabitur
36+
venenatis sem fringilla ante elementum eget faucibus nulla tempus.
37+
Aenean convallis sapien et dolor lobortis interdum. Phasellus odio
38+
risus, sagittis ut elementum ut, porttitor non libero. Integer
39+
fringilla magna quis augue dapibus malesuada. Nulla consectetur
40+
nisi mi. Suspendisse faucibus lobortis ornare. Nunc venenatis
41+
tortor in urna pulvinar pulvinar. Sed et mi nec justo hendrerit
42+
cursus ac nec mauris. Morbi et ante a lorem iaculis rutrum vitae eu
43+
massa.
44+
</p>
45+
<h1>A Simple Multi-Page Article For Testing : Page 2</h1>
46+
<p>
47+
Nunc non blandit velit. Maecenas suscipit sem sed velit tristique
48+
facilisis. Quisque condimentum, nisi vitae dictum euismod, diam
49+
risus vehicula nibh, in scelerisque lorem risus et risus. Aliquam
50+
erat volutpat. Pellentesque habitant morbi tristique senectus et
51+
netus et malesuada fames ac turpis egestas. Donec blandit venenatis
52+
feugiat. Ut quis turpis ac urna consectetur sagittis. Vestibulum
53+
aliquet eros et orci placerat vitae tempus tellus pretium. Quisque
54+
rutrum sapien quis nibh facilisis quis posuere ipsum elementum. In
55+
ac pretium justo. Sed egestas luctus mollis. Donec rutrum leo a
56+
turpis facilisis commodo. Nam quis quam eget mi malesuada
57+
scelerisque. Pellentesque semper condimentum sagittis. Nam
58+
lobortis, tortor ut placerat viverra, ante felis vehicula sem,
59+
blandit ultricies purus urna eget elit. Pellentesque habitant morbi
60+
tristique senectus et netus et malesuada fames ac turpis egestas.
61+
Sed vel nulla sollicitudin dolor adipiscing dapibus aliquam vitae
62+
leo. Phasellus at turpis tempus lectus pellentesque faucibus.
63+
</p>
64+
<p>
65+
Quisque egestas congue metus quis semper. Integer in ornare nunc.
66+
Nunc in est eget risus pulvinar tincidunt. Nullam eu tempus tortor.
67+
Suspendisse potenti. Aliquam erat volutpat. Praesent sem leo,
68+
molestie a dignissim eget, aliquet sit amet est. Suspendisse sed
69+
libero in urna tincidunt viverra. Maecenas posuere risus non elit
70+
adipiscing a tristique nibh aliquet. Nullam varius risus vitae
71+
turpis lacinia pharetra bibendum magna aliquam. Nam consectetur
72+
mattis lectus, vitae hendrerit lectus iaculis ut. Curabitur commodo
73+
pharetra nibh mollis pulvinar. Nulla in metus dui, vitae ultrices
74+
nibh. Cum sociis natoque penatibus et magnis dis parturient montes,
75+
nascetur ridiculus mus. Cras sed condimentum mi. Morbi vitae velit
76+
in neque tincidunt imperdiet quis quis orci. Proin molestie, erat
77+
convallis vulputate consectetur, diam odio interdum arcu, non
78+
semper neque ante a dolor.
79+
</p>
80+
<h1>A Simple Multi-Page Article For Testing : Page 3</h1>
81+
<p>
82+
Nullam laoreet, nibh non faucibus dictum, tellus libero varius
83+
erat, lobortis varius est massa quis metus. Donec vitae justo
84+
lacus, nec convallis metus. Suspendisse potenti. Nunc et rutrum
85+
justo. Maecenas ultrices ipsum in magna fermentum eleifend. Fusce
86+
sagittis pretium aliquam. Vestibulum et gravida lorem. Sed turpis
87+
quam, placerat ac ultrices eu, tempor sit amet elit. Curabitur eu
88+
imperdiet velit. Quisque pharetra ornare nunc, a volutpat metus
89+
aliquam quis. Vivamus semper aliquam cursus. Nullam ac nibh nulla,
90+
luctus pharetra nunc. Etiam ut sapien sem. Fusce vehicula, sem sit
91+
amet viverra pretium, magna tortor suscipit nisi, id interdum lorem
92+
orci in tellus. Vivamus vel ipsum eros. Fusce porttitor convallis
93+
ultricies. Etiam in risus diam, viverra suscipit felis. Duis vitae
94+
imperdiet est.
95+
</p>
96+
<p>
97+
Nunc nunc magna, facilisis blandit venenatis ut, scelerisque ac
98+
tortor. Cras condimentum fermentum lectus ac convallis. Suspendisse
99+
cursus, lacus sit amet sodales molestie, dui erat varius velit, non
100+
tincidunt metus dui sed nulla. Aliquam lacus orci, convallis ut
101+
pellentesque ac, molestie et dolor. Ut pretium enim ut nunc auctor
102+
eget placerat magna luctus. Duis mollis ligula a orci ultrices in
103+
facilisis felis feugiat. Morbi eget odio eget erat pulvinar
104+
placerat sed nec erat. Duis dignissim, dolor a lacinia commodo,
105+
metus erat laoreet dui, in lacinia felis lacus vitae nulla. Fusce
106+
imperdiet condimentum volutpat. Vivamus ut lacus a eros cursus
107+
scelerisque non sit amet orci. Phasellus id quam odio. Nulla
108+
adipiscing venenatis lorem nec feugiat. Aenean sit amet nisl odio,
109+
tincidunt scelerisque nisl. Curabitur ut nisl a dui facilisis
110+
vulputate. Mauris eu elit et felis hendrerit blandit. Cras magna
111+
dolor, imperdiet eget rutrum tempus, euismod nec augue.
112+
</p>
113+
<p>
114+
Ut in sem sit amet felis scelerisque elementum. Suspendisse vitae
115+
neque magna, in laoreet felis. Aenean elit ligula, tempor in
116+
vestibulum ac, porttitor nec lacus. Aenean urna mi, dictum feugiat
117+
placerat eget, congue nec dolor. Etiam pellentesque dictum nulla id
118+
vulputate. Etiam sit amet vehicula purus. Integer quis mi nisl,
119+
gravida malesuada enim. Donec malesuada felis nisi. Etiam id magna
120+
a libero pulvinar ullamcorper in nec neque. Duis pulvinar massa nec
121+
magna scelerisque vitae vulputate ipsum luctus.
122+
</p>
123+
</div>

0 commit comments

Comments
 (0)
0