pReya left a comment (openstreetmap/openstreetmap-website#6452)
> This should be tested in `test/controllers/api/users_controller_test.rb` by
> extending `check_xml_details` and `check_json_details` and adding appropriate
> attributes to at least some of the users created by the test methods.
>
> There's also a lint warning that needs resolving.
Thanks for the feedback and the hint regarding the tests. I tried to fix it, to
the best of my ability. However, I'm not a Ruby programmer, so be gentle with
me, please.
Full Disclosure: I've used AI/LLM Support for these changes (as much as I hate
LLMs, for these kind of use cases I find them rather useful), as my Ruby
knowledge is very very limited.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6452#issuecomment-3422804503
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/6452/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev
