Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The live preview is activated as default. That means that tests for this are important.
Comment | File | Size | Author |
---|---|---|---|
#9 | panopoly-behat-livepreview-2150051-9.patch | 13.45 KB | dsnopek |
#8 | panopoly-behat-livepreview-2150051-8.patch | 10.93 KB | dsnopek |
#7 | panopoly-behat-livepreview-2150051-7.patch | 8.08 KB | dsnopek |
#4 | panopoly-behat-livepreview-2150051-4.patch | 4.46 KB | dsnopek |
#1 | livepreview-test.patch | 2.82 KB | lsolesen |
Comments
Comment #1
lsolesen CreditAttribution: lsolesen commentedI cannot quite figure out how to test the live preview good enough. But here is a starting point.
Comment #2
dsnopekThere are a bunch of issues that fix bugs in live preview which really need to be tested. Maybe the bugs can guide the tests for it?
Comment #3
dsnopekAnother idea is that some widgets will give validation errors if submitted before all the data is in, for example, the "Links" widget will complain if you put in a title with out an href. You could test that filling in the title will show the validation error without even having to press anything else?
Comment #4
dsnopekHere is an updated Behat test which resembles the manual tests I was doing for some of the Panopoly Magic issues.
There still a lot of stuff I'd like to add to it, though.
Comment #5
lsolesen CreditAttribution: lsolesen commentedMight en worth committing and adding to it lager?
Comment #6
dsnopekYeah, but I'm hoping to extend it later today or tomorrow. After doing all the Panopoly Magic testing, I'm really, really looking forward to automating it. :-)
If for some reason I don't come back to this soon, I'll just commit it and finish it later.
Comment #7
dsnopekI got a little bit further today and found a whole bunch more bugs. :-/ So, these tests don't pass.
Comment #8
dsnopekOk, here is a version of this test that is as complete as I'm going to make it for now.
These bugs will need to be fixed before it will pass:
Comment #9
dsnopekUpdated for the changes to live preview in #2016527: Panopoly Magic: use #parents instead of checking under exposed.
Comment #10
dsnopekFinally passing! Committed in c316a43.
Comment #12
dsnopek