Merge pull request #59 from oreillymedia/bug-fix-leftover-print

Remove a troublesome print statement
oreillymedia · Feb 14, 2024 · bf16dde · bf16dde
2 parents 54ffc6a + 3df6983
commit bf16dde
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/jupyter_book_to_htmlbook/file_processing.py b/jupyter_book_to_htmlbook/file_processing.py
@@ -186,13 +186,12 @@ def get_main_section(soup):
         try:
             main_title = article.find('h1').get_text()
         except AttributeError:
-            main_title = soup.find("h1")
+            main_title = soup.find("h1").get_text()
         err_msg = f"The chapter with title '{main_title}' " + \
-                  "has extra <section>s " + \
+                  "has extra sections " + \
                   "that will not be processed. Please check the " + \
                   "notebook source files."
         logging.warning(err_msg)
-        print(err_msg)
     bibliography = soup.find('section', id="bibliography")
 
     return main, bibliography

diff --git a/tests/test_file_processing.py b/tests/test_file_processing.py
@@ -81,7 +81,6 @@ def test_process_chapter_single_file_with_multiple_h1s(self,
         process_chapter((test_env / 'many_a_levels.html'),
                         test_env, test_out)
         log = caplog.text
-        assert "will not be processed" in capsys.readouterr().out
         assert "will not be processed" in log
 
     def test_process_chapter_single_file_bibliogrpahy(self,