Code review comment for lp:~kfader/mvhub/pages.t-too-slow

Revision history for this message
Dan MacNeil (omacneil) wrote :

tests fail.

Before submitting merge request, all should pass

run them with:

   mv_prove -r app-mvhub/t lib-mvhub/t

app-mvhub/t/perltidy.t fails.

...and anything that uses TestHelper::pages_get_ok()

sub routine was in a library for a reason...

Need to either add caching to routine (quick little & little ugly) or change every test file that uses it.

review: Needs Fixing

« Back to merge proposal