Update batch_result method to create a QueryResultCollection object, the query_result method shouldn't do that. The batch_result receives a response containing multiple result ids and the collection object is for being able to page through them so makes sense to create that object here whereas query_result is expected to just return a normal array based on only the required data it needs. Its weird passing result_ids array just to create the collection object. Doesn't make sense to here. All tests passing.
Update batch_result method to create a QueryResultCollection object, the query_result method shouldn't do that. The batch_result receives a response containing multiple result ids and the collection object is for being able to page through them so makes sense to create that object here whereas query_result is expected to just return a normal array based on only the required data it needs. Its weird passing result_ids array just to create the collection object. Doesn't make sense to here. All tests passing.

javierjulio/salesforce_bulk