Autotest: Adjust the testBasicPagination from PaginationAJAXTest

Created on 19 April 2022, over 2 years ago
Updated 14 February 2023, over 1 year ago

Problem/Motivation

There is a strange corner case when the views pagination autotest fails with such a message:

-'?status=All&type=All&langcode=All&items_per_page=5&order=changed&sort=asc&title=&page=2'
+'?status=All&type=All&title=&langcode=All&items_per_page=5&order=changed&sort=asc&page=2'

In this particular case, the test must pass because there are the exact same GET params but the only difference is the ordering. From how I see it, we should pass it and don't take in count params ordering.

Steps to reproduce

1. Navigate to this issue https://www.drupal.org/project/drupal/issues/3067909 πŸ› View where input is required, ajax enabled and full text exposed search is empty does not show results on pagination Closed: duplicate
2. Find the patch https://www.drupal.org/files/issues/2022-04-19/3067909-11.patch β†’ and see the test results: https://www.drupal.org/pift-ci-job/2363668 β†’

Proposed resolution

Write a proper fix for the testBasicPagination so, it will parse the query string and then compare it with the expected one.

Remaining tasks

1. Write a fix
2. Test it carefully

User interface changes

No

API changes

No.

Data model changes

No.

Release notes snippet

No.

πŸ› Bug report
Status

Needs work

Version

9.5

Component
ViewsΒ  β†’

Last updated about 13 hours ago

Created by

πŸ‡²πŸ‡©Moldova andrei.vesterli Chisinau

Live updates comments and jobs are added and updated live.
  • views

    Involves, uses, or integrates with views. In Drupal 8 core, use the β€œVDC” tag instead.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024