Add handling for leftover list elements after our main loop finishes

[sondrelg]

Fixes upstream test nr 10, but cheats (!) by changing the expected result. I reverted the div removal

[sondrelg]

We weren't handling leftover list elements, so if we ended on a list element we were getting bad results.

[sourcery-ai]

Sourcery Code Quality Report

❌  Merging this PR will decrease code quality in the affected files by 0.29%.

Quality metrics	Before	After	Change
Complexity	6.99 ⭐	7.22 ⭐	0.23 👎
Method Length	62.25 🙂	63.25 🙂	1.00 👎
Working memory	9.53 🙂	9.46 🙂	-0.07 👍
Quality	66.49% 🙂	66.20% 🙂	-0.29% 👎

Other metrics	Before	After	Change
Lines	141	145	4

Changed files	Quality Before	Quality After	Quality Change
sanity_html/renderer.py	66.49% 🙂	66.20% 🙂	-0.29% 👎

Here are some functions in these files that still need a tune-up:

File	Function	Complexity	Length	Working Memory	Quality	Recommendation
sanity_html/renderer.py	SanityBlockRenderer._render_list	9 🙂	128 😞	15 😞	48.74% 😞	Try splitting into smaller methods. Extract out complex expressions
sanity_html/renderer.py	SanityBlockRenderer._render_node	6 ⭐	65 🙂	10 😞	68.19% 🙂	Extract out complex expressions

Legend and Explanation

The emojis denote the absolute quality of the code:

• ⭐ excellent
• 🙂 good
• 😞 poor
• ⛔ very poor

The 👍 and 👎 indicate whether the quality has improved or gotten worse with this pull request.

---

Please see our documentation here for details on how these metrics are calculated.

We are actively working on this report - lots more documentation and extra metrics to come!

Let us know what you think of it by mentioning @sourcery-ai in a comment.