Opened 71 minutes ago

Last modified 10 minutes ago

#37181 assigned Cleanup/optimization

History view Selenium test uses assertTrue instead of assertEqual

Reported by: Varun Kasyap Pentamaraju Owned by: Varun Kasyap Pentamaraju
Component: contrib.admin Version: dev
Severity: Normal Keywords:
Cc: Varun Kasyap Pentamaraju Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

in admin_views.test_history_view.SeleniumTests.test_pagination,

there is an assertion that check last page pagination number.

self.assertTrue(last_page_link.text, "20")

but, seems like it is intended to be assertEqual()

also, setUp creates 1100 records. with admin page size of 100, the last page should be "11", not "20".

Change History (1)

comment:1 by Tim Graham, 10 minutes ago

Triage Stage: UnreviewedAccepted

Probably you can make typo fixes like this without a ticket.

Note: See TracTickets for help on using tickets.
Back to Top