First is that if it failed to get scroll id, it would fail partly through object construction but would still try to delete the scroll.
Second is that even if all of the results were included in the first request, it would still try to fetch from the scroller a second time before stopping.
Also fix the User-Agent string being passed to the constructor, and thus the UA used by the tests.
Fixes two issues with the scroller.
First is that if it failed to get scroll id, it would fail partly through object construction but would still try to delete the scroll.
Second is that even if all of the results were included in the first request, it would still try to fetch from the scroller a second time before stopping.
Also fix the User-Agent string being passed to the constructor, and thus the UA used by the tests.