Fix failing test cases

MaddyGuthridge · MaddyGuthridge · commit 9e17beb07def · 2025-02-12T21:04:16.000+11:00
diff --git a/README.md b/README.md
@@ -46,9 +46,7 @@ Learn more by reading [the documentation](https://comp1010unsw.github.io/pyhtml-
     <h1>
       Hello, world!
     </h1>
-    <p>
-      This is my amazing website!
-    </p>
+    <p>This is my amazing website!</p>
   </body>
 </html>
 
diff --git a/docs/compatibility.md b/docs/compatibility.md
@@ -29,16 +29,16 @@ modify the original instance, as I found the old behaviour confusing and
 bug-prone.
 
 ```py
->>> para = p.p("Base paragraph")
->>> para2 = para("Extra text")
->>> para2
-<p>
+>>> span1 = p.span("Base paragraph")
+>>> span2 = span1("Extra text")
+>>> span2
+<span>
   Base paragraph
   Extra text
-</p>
->>> para
-<p>
+</span>
+>>> span1
+<span>
   Base paragraph
-</p>
+</span>
 
 ```
diff --git a/docs/index.md b/docs/index.md
@@ -32,9 +32,7 @@ Build HTML documents in Python with a simple and learnable syntax.
     <h1>
       Hello, world!
     </h1>
-    <p>
-      This is my amazing website!
-    </p>
+    <p>This is my amazing website!</p>
   </body>
 </html>
 
diff --git a/tests/basic_rendering_test.py b/tests/basic_rendering_test.py
@@ -16,7 +16,6 @@
     head,
     html,
     input,
-    p,
     script,
     span,
     title,
@@ -46,16 +45,16 @@ def test_renders_elements_with_children():
 def test_renders_deeply_nested_children():
     doc = body(
         div(
-            p("Hello world"),
+            span("Hello world"),
         ),
     )
 
     assert str(doc) == '\n'.join([
         '<body>',
         '  <div>',
-        '    <p>',
+        '    <span>',
         '      Hello world',
-        '    </p>',
+        '    </span>',
         '  </div>',
         '</body>',
     ])
@@ -158,7 +157,7 @@ def test_larger_page():
         ),
         body(
             h1("Hello, world!"),
-            p("This is my amazing website rendered with PyHTML Enhanced!"),
+            span("This is my amazing website rendered with PyHTML Enhanced!"),
         ),
     )
 
@@ -176,9 +175,9 @@ def test_larger_page():
         '    <h1>',
         '      Hello, world!',
         '    </h1>',
-        '    <p>',
+        '    <span>',
         '      This is my amazing website rendered with PyHTML Enhanced!',
-        '    </p>',
+        '    </span>',
         '  </body>',
         '</html>',
     ])
@@ -196,16 +195,16 @@ def test_flatten_element_lists():
     If a list of elements is given as a child element, each element should be
     considered as a child.
     """
-    doc = body([p("Hello"), p("world")])
+    doc = body([span("Hello"), span("world")])
 
     assert str(doc) == "\n".join([
         "<body>",
-        "  <p>",
+        "  <span>",
         "    Hello",
-        "  </p>",
-        "  <p>",
+        "  </span>",
+        "  <span>",
         "    world",
-        "  </p>",
+        "  </span>",
         "</body>",
     ])
 
diff --git a/tests/end_to_end/flask_test.py b/tests/end_to_end/flask_test.py
@@ -1,6 +1,7 @@
 """
 Tests to ensure our code works nicely with Flask
 """
+
 import pytest
 from flask import Flask
 from flask.testing import FlaskClient
@@ -12,12 +13,14 @@
 
 @app.get("/")
 def simple_route():
-    return str(p.html(
-        p.body(
-            p.p("This app is to test Flask's integration with PyHTML."),
-            p.a(href="/no_str")("Click here to get a 500 error."),
+    return str(
+        p.html(
+            p.body(
+                p.p("Testing Flask's integration with PyHTML."),
+                p.a(href="/no_str")("Click here to get a 500 error."),
+            )
         )
-    ))
+    )
 
 
 # This route intentionally returns the wrong type
@@ -27,9 +30,7 @@ def no_stringify():
     This route intentionally returns an un-stringified PyHTML object. We use
     it to test that a reasonable error message is given
     """
-    return p.html(
-        p.body("Hello, world!")
-    )
+    return p.html(p.body("Hello, world!"))
 
 
 @pytest.fixture
@@ -51,19 +52,19 @@ def test_simple_stringify(client: FlaskClient):
     response = client.get("/")
     assert response.status_code == 200
 
-    assert response.text == '\n'.join([
-        '<!DOCTYPE html>',
-        '<html>',
-        '  <body>',
-        '    <p>',
-        '      This app is to test Flask&#x27;s integration with PyHTML.',
-        '    </p>',
-        '    <a href="/no_str">',
-        '      Click here to get a 500 error.',
-        '    </a>',
-        '  </body>',
-        '</html>',
-    ])
+    assert response.text == "\n".join(
+        [
+            "<!DOCTYPE html>",
+            "<html>",
+            "  <body>",
+            "    <p>Testing Flask&#x27;s integration with PyHTML.</p>",
+            '    <a href="/no_str">',
+            "      Click here to get a 500 error.",
+            "    </a>",
+            "  </body>",
+            "</html>",
+        ]
+    )
 
 
 def test_failed_to_stringify(client: FlaskClient):
@@ -78,5 +79,5 @@ def test_failed_to_stringify(client: FlaskClient):
 
 
 # Ignore coverage since this won't be used when running tests automatically
-if __name__ == '__main__':  # pragma: no cover
+if __name__ == "__main__":  # pragma: no cover
     app.run(debug=True)
