django-react-templatetags icon indicating copy to clipboard operation
django-react-templatetags copied to clipboard

Some tests misusing assertTrue for comparisons

Open code-review-doctor opened this issue 3 years ago • 0 comments

assertTrue is not for comparing arguments, should use assertEqual for that.

The developer's intent of the test was to compare argument 1 with argument 2, which is not happening. Really what is happening is the test is passing because first argument is truthy. The correct method to use is assertEqual. more details

https://github.com/Frojd/django-react-templatetags/blob/cb781d6eaa7b19b49840543ee22f7bd243d97dec/django_react_templatetags/tests/test_ssr_hypernova_service.py#L205

I found this issue automatically, see other issues here

code-review-doctor avatar Feb 15 '22 16:02 code-review-doctor