Support _ in URLs

[paperdoorknob] / paperdoorknob_test.py

diff --git a/paperdoorknob_test.py b/paperdoorknob_test.py
index 14f6dd9cde0057e459c3fa7bd961c1fdf02fc436..511abb3c17d6ae676be0cd838e5e5b8587e95526 100644 (file)
--- a/paperdoorknob_test.py
+++ b/paperdoorknob_test.py
@@ -8,12 +8,15 @@
 from abc import ABC, abstractmethod
 import unittest
 import io
+import re
 import subprocess
 
 import paperdoorknob
 
 from testing.fakeserver import FakeGlowficServer
 from fetch import DirectFetcher, FakeFetcher, Fetcher
+from glowfic import ContentOnlyLayout, BesideIconLayout
+from images import FakeImageStore
 from spec import Spec
 from texify import DirectTexifier, PandocTexifier, VerifyingTexifier
 
@@ -30,39 +33,48 @@ class BaseTestProcess(ABC):
     def fetcher(self) -> Fetcher:
         raise NotImplementedError()
 
-    def testReplies(self) -> None:
-        replies = paperdoorknob.replies(
-            paperdoorknob.clean(
-                paperdoorknob.parse(
-                    self.fetcher().fetch(
-                        self.url()))))
-        assert [r.text.strip() for r in replies] == [
-            "This is glowfic",
-            "You sure?",
-            "Pretty sure."]
-
     def testProcess(self) -> None:
         buf = io.BytesIO()
         spec = Spec(
             self.url(),
             self.fetcher(),
+            FakeImageStore(),
+            lambda x: x,
+            lambda x: None,
             lambda x: x,
-            PandocTexifier('pandoc'),
-            buf)
+            ContentOnlyLayout(PandocTexifier('pandoc')),
+            'margin=20mm',
+            buf,
+            lambda _: None)
         paperdoorknob.process(spec)
-        assert buf.getvalue() == b'''\\documentclass{article}
+        assert re.match(br'''\\documentclass{article}
+(\\usepackage{[a-z]+}\n)+\\usepackage\[margin=20mm\]{geometry}
 \\begin{document}
-This is glowfic
-You \\emph{sure}?
+\\newcommand{\\href}\[2\]{#2\\footnote{\\detokenize{#1}}}
+This is \\href{https://glowfic.com}{glowfic}
+
+You \\emph{sure}\?
+
 Pretty sure.
+
 \\end{document}
-'''
+''', buf.getvalue())
 
     def testDirectTexifier(self) -> None:
         texifier = VerifyingTexifier(
             PandocTexifier('pandoc'), DirectTexifier())
         buf = io.BytesIO()
-        spec = Spec(self.url(), self.fetcher(), lambda x: x, texifier, buf)
+        spec = Spec(
+            self.url(),
+            self.fetcher(),
+            FakeImageStore(),
+            lambda x: x,
+            lambda x: None,
+            lambda x: x,
+            ContentOnlyLayout(texifier),
+            None,
+            buf,
+            lambda _: None)
         paperdoorknob.process(spec)
 
     def testPDF(self) -> None:
@@ -70,9 +82,14 @@ Pretty sure.
             spec = Spec(
                 self.url(),
                 self.fetcher(),
+                FakeImageStore(),
+                lambda x: x,
+                lambda x: None,
                 lambda x: x,
-                PandocTexifier('pandoc'),
-                out)
+                BesideIconLayout(PandocTexifier('pandoc'), 20),
+                None,
+                out,
+                lambda _: None)
             paperdoorknob.process(spec)
         subprocess.run(['pdflatex', 'test.tex'],
                        stdin=subprocess.DEVNULL, check=True)
@@ -99,7 +116,7 @@ class TestProcessFromFakeFetcher(BaseTestProcess, unittest.TestCase):
 
     def fetcher(self) -> Fetcher:
         with open('testdata/this-is-glowfic.html', 'rb') as f:
-            return FakeFetcher({'fic': f.read(9999)})
+            return FakeFetcher({'fic?view=flat': f.read(9999)})
 
 
 if __name__ == '__main__':