Skip to content

Commit bfd0c0f

Browse files
author
Jean-Francois Thuong
committed
📑Added marker for tests using Github (to easily skip them)
1 parent 436a2fb commit bfd0c0f

File tree

10 files changed

+42
-56
lines changed

10 files changed

+42
-56
lines changed

‎docs/nlp_data.html‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -372,11 +372,13 @@ <h1 id="Tests">Tests<a class="anchor-link" href="#Tests"> </a></h1>
372372
<span class="k">with</span> <span class="n">pytest</span><span class="o">.</span><span class="n">raises</span><span class="p">(</span><span class="ne">FileExistsError</span><span class="p">):</span>
373373
<span class="n">textual</span> <span class="o">=</span> <span class="n">Textual</span><span class="o">.</span><span class="n">from_path</span><span class="p">(</span><span class="s2">&quot;does_not_exist.txt&quot;</span><span class="p">)</span>
374374

375+
<span class="nd">@pytest</span><span class="o">.</span><span class="n">mark</span><span class="o">.</span><span class="n">github</span>
375376
<span class="k">def</span> <span class="nf">test_from_url</span><span class="p">(</span><span class="bp">self</span><span class="p">,</span> <span class="n">check_connection_github</span><span class="p">,</span> <span class="n">local_textual</span><span class="p">):</span>
376377
<span class="sd">&quot;&quot;&quot;Test extract Textual from URL&quot;&quot;&quot;</span>
377378
<span class="n">textual</span> <span class="o">=</span> <span class="n">Textual</span><span class="o">.</span><span class="n">from_url</span><span class="p">(</span><span class="n">GITHUB_TEST_TXT</span><span class="p">)</span>
378379
<span class="k">assert</span> <span class="n">textual</span><span class="o">.</span><span class="n">text</span> <span class="o">==</span> <span class="n">local_textual</span><span class="o">.</span><span class="n">text</span><span class="p">,</span> <span class="sa">f</span><span class="s2">&quot;URL text: </span><span class="si">{</span><span class="n">textual</span><span class="o">.</span><span class="n">text</span><span class="si">}</span><span class="s2">&quot;</span>
379380

381+
<span class="nd">@pytest</span><span class="o">.</span><span class="n">mark</span><span class="o">.</span><span class="n">github</span>
380382
<span class="k">def</span> <span class="nf">test_from_url_non_ascii</span><span class="p">(</span><span class="bp">self</span><span class="p">,</span> <span class="n">check_connection_github</span><span class="p">):</span>
381383
<span class="sd">&quot;&quot;&quot;Test extract Textual from URL with non-ascii characters&quot;&quot;&quot;</span>
382384
<span class="n">textual</span> <span class="o">=</span> <span class="n">Textual</span><span class="o">.</span><span class="n">from_url</span><span class="p">(</span><span class="n">GITHUB_TEST_TXT_UTF8</span><span class="p">)</span>

‎docs/nlp_pdf.html‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -220,11 +220,13 @@ <h1 id="Tests">Tests<a class="anchor-link" href="#Tests"> </a></h1>
220220
<span class="n">extra_pages</span> <span class="o">=</span> <span class="nb">set</span><span class="p">(</span><span class="n">pages_extracted</span><span class="p">)</span> <span class="o">-</span> <span class="nb">set</span><span class="p">(</span><span class="nb">str</span><span class="p">(</span><span class="n">p</span> <span class="o">+</span> <span class="mi">1</span><span class="p">)</span> <span class="k">for</span> <span class="n">p</span> <span class="ow">in</span> <span class="n">pages</span><span class="p">)</span>
221221
<span class="k">assert</span> <span class="n">extra_pages</span> <span class="o">==</span> <span class="nb">set</span><span class="p">(),</span> <span class="sa">f</span><span class="s2">&quot;Extra pages extracted: </span><span class="si">{</span><span class="n">extra_pages</span><span class="si">}</span><span class="s2">&quot;</span>
222222

223+
<span class="nd">@pytest</span><span class="o">.</span><span class="n">mark</span><span class="o">.</span><span class="n">github</span>
223224
<span class="k">def</span> <span class="nf">test_from_url</span><span class="p">(</span><span class="bp">self</span><span class="p">,</span> <span class="n">check_connection_github</span><span class="p">,</span> <span class="n">local_textual</span><span class="p">):</span>
224225
<span class="sd">&quot;&quot;&quot;Test extract Textual of PDF from URL&quot;&quot;&quot;</span>
225226
<span class="n">textual</span> <span class="o">=</span> <span class="n">TextualPDF</span><span class="o">.</span><span class="n">from_url</span><span class="p">(</span><span class="n">GITHUB_TEST_PDF</span><span class="p">)</span>
226227
<span class="k">assert</span> <span class="n">textual</span><span class="o">.</span><span class="n">text</span> <span class="o">==</span> <span class="n">local_textual</span><span class="o">.</span><span class="n">text</span><span class="p">,</span> <span class="sa">f</span><span class="s2">&quot;URL text: </span><span class="si">{</span><span class="n">textual</span><span class="o">.</span><span class="n">text</span><span class="si">}</span><span class="s2">&quot;</span>
227228

229+
<span class="nd">@pytest</span><span class="o">.</span><span class="n">mark</span><span class="o">.</span><span class="n">github</span>
228230
<span class="k">def</span> <span class="nf">test_from_url_pdf</span><span class="p">(</span><span class="bp">self</span><span class="p">,</span> <span class="n">check_connection_github</span><span class="p">,</span> <span class="n">local_textual</span><span class="p">):</span>
229231
<span class="sd">&quot;&quot;&quot;Test extract Textual of PDF from URL using from_url_pdf&quot;&quot;&quot;</span>
230232
<span class="n">textual</span> <span class="o">=</span> <span class="n">TextualPDF</span><span class="o">.</span><span class="n">from_url_pdf</span><span class="p">(</span><span class="n">GITHUB_TEST_PDF</span><span class="p">)</span>

‎docs/utils.html‎

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -427,10 +427,10 @@ <h3 id="Try-hushing-various-kinds-of-info">Try hushing various kinds of info<a c
427427

428428

429429

430-
<div id="7e553850-fe0b-4d5c-98b5-b458b20ad1c7"></div>
430+
<div id="1e98ee6d-dc2f-45a9-aa69-34230fcf8995"></div>
431431
<div class="output_subarea output_javascript ">
432432
<script type="text/javascript">
433-
var element = $('#7e553850-fe0b-4d5c-98b5-b458b20ad1c7');
433+
var element = $('#1e98ee6d-dc2f-45a9-aa69-34230fcf8995');
434434

435435
console.info("loading toggle event: 00d9724d-55e4-4ab8-b426-5e92f71f593c")
436436
const toggle_hush = (e) =>{
@@ -487,10 +487,10 @@ <h3 id="Capable-of-open/close-verbosity-during-the-main-process">Capable of open
487487

488488

489489

490-
<div id="b44b26f6-5c06-4474-b0ad-f9e5cb0abf04"></div>
490+
<div id="afbabf0d-95c5-48e2-989d-a92b90992758"></div>
491491
<div class="output_subarea output_javascript ">
492492
<script type="text/javascript">
493-
var element = $('#b44b26f6-5c06-4474-b0ad-f9e5cb0abf04');
493+
var element = $('#afbabf0d-95c5-48e2-989d-a92b90992758');
494494

495495
console.info("loading toggle event: a5461483-be0b-4a73-8299-458f05a7c0d5")
496496
const toggle_hush = (e) =>{
@@ -552,10 +552,10 @@ <h3 id="But-we-still-want-the-error-to-be-loud">But we still want the error to b
552552

553553

554554

555-
<div id="a4a265b4-4017-4d33-b104-1398e100f8d2"></div>
555+
<div id="639ad8dd-86f0-4d95-a004-d7a16eef7928"></div>
556556
<div class="output_subarea output_javascript ">
557557
<script type="text/javascript">
558-
var element = $('#a4a265b4-4017-4d33-b104-1398e100f8d2');
558+
var element = $('#639ad8dd-86f0-4d95-a004-d7a16eef7928');
559559

560560
console.info("loading toggle event: 33d84ffd-6fef-48a7-8ddc-26ef5ec8513b")
561561
const toggle_hush = (e) =>{
@@ -1128,10 +1128,12 @@ <h4 id="url_2_text" class="doc_header"><code>url_2_text</code><a href="https://g
11281128

11291129
<div class="inner_cell">
11301130
<div class="input_area">
1131-
<div class=" highlight hl-ipython3"><pre><span></span><span class="k">def</span> <span class="nf">test_get_url_size</span><span class="p">(</span><span class="n">check_connection_github</span><span class="p">):</span>
1131+
<div class=" highlight hl-ipython3"><pre><span></span><span class="nd">@pytest</span><span class="o">.</span><span class="n">mark</span><span class="o">.</span><span class="n">github</span>
1132+
<span class="k">def</span> <span class="nf">test_get_url_size</span><span class="p">(</span><span class="n">check_connection_github</span><span class="p">):</span>
11321133
<span class="k">assert</span> <span class="n">get_url_size</span><span class="p">(</span><span class="n">url_raw_txt</span><span class="p">)</span> <span class="o">==</span> <span class="mi">264</span><span class="p">,</span> <span class="sa">f</span><span class="s2">&quot;Wrong size for </span><span class="si">{</span><span class="n">url_raw_txt</span><span class="si">}</span><span class="s2">&quot;</span>
11331134

11341135

1136+
<span class="nd">@pytest</span><span class="o">.</span><span class="n">mark</span><span class="o">.</span><span class="n">github</span>
11351137
<span class="k">def</span> <span class="nf">test_download_dest</span><span class="p">(</span><span class="n">check_connection_github</span><span class="p">,</span> <span class="n">tmpdir</span><span class="p">):</span>
11361138
<span class="sd">&quot;&quot;&quot;Test download of file to a destination&quot;&quot;&quot;</span>
11371139
<span class="n">dest</span> <span class="o">=</span> <span class="n">Path</span><span class="p">(</span><span class="n">tmpdir</span> <span class="o">/</span> <span class="s2">&quot;to_download.txt&quot;</span><span class="p">)</span>
@@ -1140,6 +1142,7 @@ <h4 id="url_2_text" class="doc_header"><code>url_2_text</code><a href="https://g
11401142
<span class="k">assert</span> <span class="n">dest</span><span class="o">.</span><span class="n">read_text</span><span class="p">()</span> <span class="o">==</span> <span class="n">test_data_txt</span>
11411143

11421144

1145+
<span class="nd">@pytest</span><span class="o">.</span><span class="n">mark</span><span class="o">.</span><span class="n">github</span>
11431146
<span class="k">def</span> <span class="nf">test_download_empty</span><span class="p">(</span><span class="n">check_connection_github</span><span class="p">,</span> <span class="n">tmpdir</span><span class="p">):</span>
11441147
<span class="sd">&quot;&quot;&quot;Test download of file without destination&quot;&quot;&quot;</span>
11451148
<span class="n">dest</span> <span class="o">=</span> <span class="n">Path</span><span class="p">(</span><span class="s2">&quot;to_download.txt&quot;</span><span class="p">)</span>
@@ -1151,6 +1154,7 @@ <h4 id="url_2_text" class="doc_header"><code>url_2_text</code><a href="https://g
11511154
<span class="n">dest</span><span class="o">.</span><span class="n">unlink</span><span class="p">()</span>
11521155

11531156

1157+
<span class="nd">@pytest</span><span class="o">.</span><span class="n">mark</span><span class="o">.</span><span class="n">github</span>
11541158
<span class="k">def</span> <span class="nf">test_url_2_text</span><span class="p">(</span><span class="n">check_connection_github</span><span class="p">):</span>
11551159
<span class="sd">&quot;&quot;&quot;Test extraction of text from URL&quot;&quot;&quot;</span>
11561160
<span class="k">assert</span> <span class="n">url_2_text</span><span class="p">(</span><span class="n">url_raw_txt</span><span class="p">)</span> <span class="o">==</span> <span class="n">test_data_txt</span>
@@ -1325,6 +1329,7 @@ <h4 id="read_csv_from_zip" class="doc_header"><code>read_csv_from_zip</code><a h
13251329
<span class="k">assert</span> <span class="nb">len</span><span class="p">(</span><span class="n">df</span><span class="p">)</span> <span class="o">==</span> <span class="mi">100</span>
13261330

13271331

1332+
<span class="nd">@pytest</span><span class="o">.</span><span class="n">mark</span><span class="o">.</span><span class="n">github</span>
13281333
<span class="nd">@pytest</span><span class="o">.</span><span class="n">mark</span><span class="o">.</span><span class="n">parametrize</span><span class="p">(</span>
13291334
<span class="s2">&quot;archive,csv&quot;</span><span class="p">,</span>
13301335
<span class="p">[</span>

‎nbs/10_utils.ipynb‎

Lines changed: 8 additions & 48 deletions
Original file line numberDiff line numberDiff line change
@@ -519,22 +519,7 @@
519519
},
520520
{
521521
"data": {
522-
"application/javascript": [
523-
"\n",
524-
" console.info(\"loading toggle event: 00d9724d-55e4-4ab8-b426-5e92f71f593c\")\n",
525-
" const toggle_hush = (e) =>{\n",
526-
" var op = document.querySelector(\".hush_output_00d9724d-55e4-4ab8-b426-5e92f71f593c\");\n",
527-
" if(op.style.display==\"none\"){\n",
528-
" op.style.display=\"block\"\n",
529-
"\n",
530-
" } else {\n",
531-
" op.style.display=\"none\"\n",
532-
" }\n",
533-
"\n",
534-
" }\n",
535-
" document.querySelector('.hush_toggle_00d9724d-55e4-4ab8-b426-5e92f71f593c').onclick=toggle_hush\n",
536-
" "
537-
],
522+
"application/javascript": "\n console.info(\"loading toggle event: 00d9724d-55e4-4ab8-b426-5e92f71f593c\")\n const toggle_hush = (e) =>{\n var op = document.querySelector(\".hush_output_00d9724d-55e4-4ab8-b426-5e92f71f593c\");\n if(op.style.display==\"none\"){\n op.style.display=\"block\"\n\n } else {\n op.style.display=\"none\"\n }\n\n }\n document.querySelector('.hush_toggle_00d9724d-55e4-4ab8-b426-5e92f71f593c').onclick=toggle_hush\n ",
538523
"text/plain": [
539524
"<IPython.core.display.Javascript object>"
540525
]
@@ -589,22 +574,7 @@
589574
},
590575
{
591576
"data": {
592-
"application/javascript": [
593-
"\n",
594-
" console.info(\"loading toggle event: a5461483-be0b-4a73-8299-458f05a7c0d5\")\n",
595-
" const toggle_hush = (e) =>{\n",
596-
" var op = document.querySelector(\".hush_output_a5461483-be0b-4a73-8299-458f05a7c0d5\");\n",
597-
" if(op.style.display==\"none\"){\n",
598-
" op.style.display=\"block\"\n",
599-
"\n",
600-
" } else {\n",
601-
" op.style.display=\"none\"\n",
602-
" }\n",
603-
"\n",
604-
" }\n",
605-
" document.querySelector('.hush_toggle_a5461483-be0b-4a73-8299-458f05a7c0d5').onclick=toggle_hush\n",
606-
" "
607-
],
577+
"application/javascript": "\n console.info(\"loading toggle event: a5461483-be0b-4a73-8299-458f05a7c0d5\")\n const toggle_hush = (e) =>{\n var op = document.querySelector(\".hush_output_a5461483-be0b-4a73-8299-458f05a7c0d5\");\n if(op.style.display==\"none\"){\n op.style.display=\"block\"\n\n } else {\n op.style.display=\"none\"\n }\n\n }\n document.querySelector('.hush_toggle_a5461483-be0b-4a73-8299-458f05a7c0d5').onclick=toggle_hush\n ",
608578
"text/plain": [
609579
"<IPython.core.display.Javascript object>"
610580
]
@@ -649,22 +619,7 @@
649619
},
650620
{
651621
"data": {
652-
"application/javascript": [
653-
"\n",
654-
" console.info(\"loading toggle event: 33d84ffd-6fef-48a7-8ddc-26ef5ec8513b\")\n",
655-
" const toggle_hush = (e) =>{\n",
656-
" var op = document.querySelector(\".hush_output_33d84ffd-6fef-48a7-8ddc-26ef5ec8513b\");\n",
657-
" if(op.style.display==\"none\"){\n",
658-
" op.style.display=\"block\"\n",
659-
"\n",
660-
" } else {\n",
661-
" op.style.display=\"none\"\n",
662-
" }\n",
663-
"\n",
664-
" }\n",
665-
" document.querySelector('.hush_toggle_33d84ffd-6fef-48a7-8ddc-26ef5ec8513b').onclick=toggle_hush\n",
666-
" "
667-
],
622+
"application/javascript": "\n console.info(\"loading toggle event: 33d84ffd-6fef-48a7-8ddc-26ef5ec8513b\")\n const toggle_hush = (e) =>{\n var op = document.querySelector(\".hush_output_33d84ffd-6fef-48a7-8ddc-26ef5ec8513b\");\n if(op.style.display==\"none\"){\n op.style.display=\"block\"\n\n } else {\n op.style.display=\"none\"\n }\n\n }\n document.querySelector('.hush_toggle_33d84ffd-6fef-48a7-8ddc-26ef5ec8513b').onclick=toggle_hush\n ",
668623
"text/plain": [
669624
"<IPython.core.display.Javascript object>"
670625
]
@@ -1174,10 +1129,12 @@
11741129
"outputs": [],
11751130
"source": [
11761131
"# exportest\n",
1132+
"@pytest.mark.github\n",
11771133
"def test_get_url_size(check_connection_github):\n",
11781134
" assert get_url_size(url_raw_txt) == 264, f\"Wrong size for {url_raw_txt}\"\n",
11791135
"\n",
11801136
"\n",
1137+
"@pytest.mark.github\n",
11811138
"def test_download_dest(check_connection_github, tmpdir):\n",
11821139
" \"\"\"Test download of file to a destination\"\"\"\n",
11831140
" dest = Path(tmpdir / \"to_download.txt\")\n",
@@ -1186,6 +1143,7 @@
11861143
" assert dest.read_text() == test_data_txt\n",
11871144
"\n",
11881145
"\n",
1146+
"@pytest.mark.github\n",
11891147
"def test_download_empty(check_connection_github, tmpdir):\n",
11901148
" \"\"\"Test download of file without destination\"\"\"\n",
11911149
" dest = Path(\"to_download.txt\")\n",
@@ -1197,6 +1155,7 @@
11971155
" dest.unlink()\n",
11981156
"\n",
11991157
"\n",
1158+
"@pytest.mark.github\n",
12001159
"def test_url_2_text(check_connection_github):\n",
12011160
" \"\"\"Test extraction of text from URL\"\"\"\n",
12021161
" assert url_2_text(url_raw_txt) == test_data_txt\n"
@@ -1807,6 +1766,7 @@
18071766
" assert len(df) == 100\n",
18081767
"\n",
18091768
"\n",
1769+
"@pytest.mark.github\n",
18101770
"@pytest.mark.parametrize(\n",
18111771
" \"archive,csv\",\n",
18121772
" [\n",

‎nbs/41_nlp_data.ipynb‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -456,11 +456,13 @@
456456
" with pytest.raises(FileExistsError):\n",
457457
" textual = Textual.from_path(\"does_not_exist.txt\")\n",
458458
"\n",
459+
" @pytest.mark.github\n",
459460
" def test_from_url(self, check_connection_github, local_textual):\n",
460461
" \"\"\"Test extract Textual from URL\"\"\"\n",
461462
" textual = Textual.from_url(GITHUB_TEST_TXT)\n",
462463
" assert textual.text == local_textual.text, f\"URL text: {textual.text}\"\n",
463464
"\n",
465+
" @pytest.mark.github\n",
464466
" def test_from_url_non_ascii(self, check_connection_github):\n",
465467
" \"\"\"Test extract Textual from URL with non-ascii characters\"\"\"\n",
466468
" textual = Textual.from_url(GITHUB_TEST_TXT_UTF8)\n",

‎nbs/42_nlp_pdf.ipynb‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -308,11 +308,13 @@
308308
" extra_pages = set(pages_extracted) - set(str(p + 1) for p in pages)\n",
309309
" assert extra_pages == set(), f\"Extra pages extracted: {extra_pages}\"\n",
310310
"\n",
311+
" @pytest.mark.github\n",
311312
" def test_from_url(self, check_connection_github, local_textual):\n",
312313
" \"\"\"Test extract Textual of PDF from URL\"\"\"\n",
313314
" textual = TextualPDF.from_url(GITHUB_TEST_PDF)\n",
314315
" assert textual.text == local_textual.text, f\"URL text: {textual.text}\"\n",
315316
"\n",
317+
" @pytest.mark.github\n",
316318
" def test_from_url_pdf(self, check_connection_github, local_textual):\n",
317319
" \"\"\"Test extract Textual of PDF from URL using from_url_pdf\"\"\"\n",
318320
" textual = TextualPDF.from_url_pdf(GITHUB_TEST_PDF)\n",

‎pytest.ini‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,6 @@
11
[pytest]
2-
testpaths = test
2+
testpaths = test
3+
markers =
4+
github: mark a test using data on github.
5+
slow: mark test as slow.
6+
magic: mark test using iPython magic.

‎test/nlp/test_data.py‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,11 +52,13 @@ def test_from_path_error(self):
5252
with pytest.raises(FileExistsError):
5353
textual = Textual.from_path("does_not_exist.txt")
5454

55+
@pytest.mark.github
5556
def test_from_url(self, check_connection_github, local_textual):
5657
"""Test extract Textual from URL"""
5758
textual = Textual.from_url(GITHUB_TEST_TXT)
5859
assert textual.text == local_textual.text, f"URL text: {textual.text}"
5960

61+
@pytest.mark.github
6062
def test_from_url_non_ascii(self, check_connection_github):
6163
"""Test extract Textual from URL with non-ascii characters"""
6264
textual = Textual.from_url(GITHUB_TEST_TXT_UTF8)

‎test/nlp/test_pdf.py‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,11 +79,13 @@ def test_from_path_pdf_pages(self, pages):
7979
extra_pages = set(pages_extracted) - set(str(p + 1) for p in pages)
8080
assert extra_pages == set(), f"Extra pages extracted: {extra_pages}"
8181

82+
@pytest.mark.github
8283
def test_from_url(self, check_connection_github, local_textual):
8384
"""Test extract Textual of PDF from URL"""
8485
textual = TextualPDF.from_url(GITHUB_TEST_PDF)
8586
assert textual.text == local_textual.text, f"URL text: {textual.text}"
8687

88+
@pytest.mark.github
8789
def test_from_url_pdf(self, check_connection_github, local_textual):
8890
"""Test extract Textual of PDF from URL using from_url_pdf"""
8991
textual = TextualPDF.from_url_pdf(GITHUB_TEST_PDF)

0 commit comments

Comments
 (0)