[5.6] Integration test to avoid repetitive pagination PR with columns #23259
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As Taylor recently said, in the last week 3 people already tried PRing a change to the Builder that passes the
$columns
into thegetCountForPagination()
. Unfortunately, that doesn't work when you use more than one column and it breaks the SQL being generated.This is an attempt to avoid people from trying that again in the future. They'll be able to see for themselves that their change breaks on TravisCI. If somebody can apply a change that doesn't break this test, maybe that would be a PR worth looking into.
References:
#23255, #23214, #23209
Edit: StyleCI is failing but I don't know what is the issue as it's failing to analyze. ping @GrahamCampbell