pablobm left a comment (openstreetmap/openstreetmap-website#6470)
I think we should have some tests about this. It's all very obscure at the
moment. With tests in place, we can also refactor this logic with more
confidence, as it's not pretty at the moment.
It doesn't need to be comprehensive from the start, but at least stub out a
test module and test some cases, like the ones introduced here.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6470#issuecomment-3441995032
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6470/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev