The WYSIWYG testing function that clicks buttons in the editing toolbar has a flaw: if the button is not found for any reason, the whole test will crash with a fatal error.

Possible causes for this issue:

  • Title of button has changed
  • Button has been moved from the always-available row to the kitchen sink row
  • Button has been removed altogether

The test should instead throw a regular testing failure/exception if the button is not found.

CommentFileSizeAuthor
#1 panopoly-wysiwyg-test-2292829-1.patch1.17 KBcboyden

Comments

cboyden’s picture

cboyden commented
Status: Active » Needs review
StatusFileSize
new panopoly-wysiwyg-test-2292829-1.patch1.17 KB

Patch attached.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Needs review » Fixed

Looks great, thanks! :-)

  • dsnopek committed 01ace17 on 7.x-1.x 
    Issue #2292829 by cboyden: Fixed WYSIWYG Behat test crashes if toolbar...

Status: Fixed » Closed (fixed)

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