Auto merge of #4381 - Turbo87:search-qps, r=Turbo87

tests/search: Add test case for query parameter forwarding

This should ensure to not break the functionality that was added in #1543 when working on implementing #491.

Author: bors

tests/acceptance/search-test.js

@@ -172,4 +172,23 @@ module('Acceptance | search', function (hooks) {
     await settled();
     assert.dom('[data-test-crate-row]').exists({ count: 1 });
   });
+
+  test('passes query parameters to the backend', async function (assert) {
+    this.server.get('/api/v1/crates', function (schema, request) {
+      assert.step('/api/v1/crates');
+
+      assert.deepEqual(request.queryParams, {
+        all_keywords: 'fire ball',
+        page: '3',
+        per_page: '15',
+        q: 'rust',
+        sort: 'new',
+      });
+
+      return { crates: [], meta: { total: 0 } };
+    });
+
+    await visit('/search?q=rust&page=3&per_page=15&sort=new&all_keywords=fire ball');
+    assert.verifySteps(['/api/v1/crates']);
+  });
 });