Book module review.

This patch cleans up the code, adds comment blocks were necessary.

The test previously did not pass HEAD due to the contents of the page not being passed to assertPattern. Since the internal browser was changed the method that was in the SimpleTest browser automatically assumed the contents.

CommentFileSizeAuthor
#2 simpletest_review_book.patch6.83 KBboombatower
#1 simpletest_review_book.patch6.33 KBboombatower
simpletest_review_book.patch5.98 KBboombatower
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

boombatower’s picture

boombatower CreditAttribution: boombatower commented
Status: Active » Fixed
FileSize
simpletest_review_book.patch6.33 KB

Committed.

boombatower’s picture

boombatower CreditAttribution: boombatower commented
FileSize
simpletest_review_book.patch6.83 KB

Decided to override the assertPattern method so that the contents of the page don't have to be passed.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.