(NFC) dev/core#2029 - Make assertions in PrevNextTest more skimmable
Overview
--------
The test appears to have a random failure. Making it more readable may help figure out why.
Before
------
`testDeleteByCacheKey()` and the related `testFillArray()` both have some assertions in these two forms:
```
// Form #1, more verbose
$all = ...getSelections($cacheKey, $action);
$this->assertEquals([...expected...], array_keys($all)...);
// Form #2, more pithy
$this->assertSelections([...expected...], $action, $cacheKey);
```
After
-----
The verbose form is replaced with the pithier form.
In the pithier form, some of the default inputs are made explicit.
Comment
-------
It is confusing that the method `getSelections()` has a parameter `$action`
which can be `get` or `getall` -- and that `getall` does not (in fact) return
selections. It returns all! (The fact that the contract is weird makes the
unit-test helpful imho...)