<div dir="ltr">I don’t think the pagination changes would break the pulp-smash tests. The default page size would still remain 100. Also, looks like the page_handler code relies on the ‘next’ field which would still exist. It would just be a different parameter in the href (page vs cursor).<div><br></div><div>Agreed that introducing breaking changes in beta is unfortunate. However, if we want to change this, now is our only option.<br clear="all"><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><br></div><div>David<br></div></div></div></div></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jun 26, 2018 at 10:31 AM Jeremy Audet <<a href="mailto:jaudet@redhat.com">jaudet@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>FYI, Pulp Smash integrates closely with Pulp's current pagination behaviour:</div><div><ul><li>The <a href="http://pulp-smash.readthedocs.io/en/latest/api/pulp_smash.api.html#pulp_smash.api.page_handler" target="_blank">page_handler</a> response handler walks through pages of results and collects them into a single list. It's used extensively.<br></li><li>The <a href="http://pulp-smash.readthedocs.io/en/latest/api/pulp_smash.tests.pulp3.file.api_v3.test_pagination.html" target="_blank">pagination test case</a> "assumes that Pulp returns 100 elements in each page of
results. This is configurable, but the current default set by all known
Pulp installers."</li></ul>If pagination behaviour is changed, then this will break a significant number of tests. Also, introducing breaking changes into a fundamental aspect of Pulp's API during beta would be unfortunate.<br></div></div>
_______________________________________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com" target="_blank">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/mailman/listinfo/pulp-dev</a><br>
</blockquote></div>