Do we have a test checking the case that fails without this sort? I'm not sure I understand what case that would be. Unless I'm missing something, it seems like adding this sort is a bugfix unrelated to the feature added in this pull. If that's the case, for clarity looking back at the history I'd rather leave it out of this pull and do it as a separate pull with its own added test.