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.
To ensure that we don't break enabling the pager again (like with #2097199: Views pager settings do not stick.) or enabling the "more link" (although, we haven't broken this before) we should have a Behat test for them! This isn't currently tested because the "Content list" widget doesn't allows those to be overridden. We may need to create a view for test purposes.
Comment | File | Size | Author |
---|---|---|---|
#7 | panopoly_test-add_behat_test_for-2154211-7.patch | 6.77 KB | joegraduate |
Comments
Comment #1
lsolesen CreditAttribution: lsolesen commentedOr would it make sense to have one of the demo views to allow this?
Comment #2
dsnopekIf one of the demo views has enough options that would work! Last time I looked though, I don't think I saw any that had options for "pager" and "more".
I think we might have to create a 'panopoly_test' module that we enable for the tests which includes a View that has every possible type of option. This would also be a good way to include another Content Type for testing, which would help with the "Content list" widget tests.
Comment #3
lsolesen CreditAttribution: lsolesen commentedThe demo views does not right now have enough options. I was proposing to create enough :) But a panopoly_test module might also cut it.
Comment #4
dsnopekComment #5
dsnopekComment #6
joegraduateGonna take a crack at this.
Comment #7
joegraduateAdding a WIP patch that I started at the DrupalCon sprint. Hoping to finish this up when I get home.