feature: add thest for photo

renderer/fifty_ohm_latex_renderer.py

@@ -113,4 +113,4 @@ class FiftyOhmLaTeXRenderer(LaTeXRenderer):
 \label{{{ref}}}"""
 
     def render_photo(self, token) :
-        return self.render_picture_helper(token.id, token.ref, token.text, token.number)
+        return self.render_photo_helper(token.id, token.ref, token.text, token.number)

renderer/photo.py

@@ -10,7 +10,7 @@ class Photo(SpanToken):
     """
 
     parse_inner = False
-    pattern = re.compile(r"\[photo:(\d+):(.+):(.+)\]")
+    pattern = re.compile(r"\[photo:(\d+):([^:\]]+):([^:\]]+)\]")
 
     def __init__(self, match_object):
         self.id = match_object.group(1)

test/test_photo.py

@@ -0,0 +1,26 @@
+import mistletoe
+import pytest
+
+from renderer.fifty_ohm_html_renderer import FiftyOhmHtmlRenderer
+from renderer.fifty_ohm_latex_renderer import FiftyOhmLaTeXRenderer
+from test.util import paragraph
+
+
+@pytest.mark.html
+def test_photo_html():
+    assertions = {
+        "[photo:123:abc:Text]": FiftyOhmHtmlRenderer.render_photo_helper("123", "abc", "Text", "TODO"),
+    }
+
+    for assertion in assertions:
+        assert mistletoe.markdown(assertion, FiftyOhmHtmlRenderer) == paragraph(assertions[assertion])
+
+
+@pytest.mark.latex
+def test_photo_latex():
+    assertions = {
+        "[photo:123:abc:Text]": FiftyOhmLaTeXRenderer.render_photo_helper("123", "abc", "Text", "TODO"),
+    }
+
+    for assertion in assertions:
+        assert mistletoe.markdown(assertion, FiftyOhmLaTeXRenderer) == "\n" + assertions[assertion] + "\n"