pages.t Too Slow
Bug #653188 reported by
Keith Fader
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
MVHub |
In Progress
|
Medium
|
Keith Fader |
Bug Description
Pages.t executes too slow due to unnecessary repetitive page fetches.
Related branches
lp:~kfader/mvhub/pages.t-too-slow
- Dan MacNeil: Approve
-
Diff: 448 lines (+99/-33)6 files modifiedapp-mvhub/t/bugs/admin_logout_bug.t (+2/-2)
app-mvhub/t/bugs/categorize_link_missing_when_agency_login.t (+6/-8)
app-mvhub/t/bugs/ql_display_agency_home_interaction.t (+4/-3)
app-mvhub/t/bugs/ql_works_after_admin_login.t (+3/-3)
app-mvhub/t/mech/pages.t (+53/-17)
lib-mvhub/t/lib/TestHelper.pm (+31/-0)
Changed in mvhub: | |
status: | Fix Committed → In Progress |
importance: | Undecided → Medium |
To post a comment you must log in.
Bug fixed.
WWW:Mechanize objects are now cached in @pages hash inside pages.t. :pages_ get_ok( ) fetches pages, responses are cached inside mech objects. Thereafter, since the mech objects are cached in the pages hash, there is no need to ever fetch the pages again.
TestHelper:
Test time has now decreased from 60 seconds to 20.