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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lsolesen’s picture

Or would it make sense to have one of the demo views to allow this?

dsnopek’s picture

If 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.

lsolesen’s picture

The demo views does not right now have enough options. I was proposing to create enough :) But a panopoly_test module might also cut it.

dsnopek’s picture

Component: Code » Admin
Parent issue: » #2223989: [meta] Refactor Behat tests
dsnopek’s picture

Component: Admin » Tests / Continuous Integration
joegraduate’s picture

Assigned: Unassigned » joegraduate

Gonna take a crack at this.

joegraduate’s picture

Status: Active » Needs work
FileSize
6.77 KB

Adding a WIP patch that I started at the DrupalCon sprint. Hoping to finish this up when I get home.