Remove sorting in case the switch is off

[Skarlso]

Description

Don't sort in case the duplication suppression is not enabled.

Closes #5089

Incoming local crud test run result...

Checklist

• Added tests that cover your change (if possible)
• Added/modified documentation as required (such as the README.md, or the userdocs directory)
• Manually tested
• Made sure the title of the PR is a good description that can go into the release notes
• (Core team) Added labels for change area (e.g. area/nodegroup) and kind (e.g. kind/improvement)

BONUS POINTS checklist: complete for good vibes and maybe prizes?! 🤯

• Backfilled missing tests for code in same general area 🎉
• Refactored something and made the world a better place 🌟

[aclevername]

is it worth changing it so the sorting logic just mutates a local copy of the identities?

[Skarlso]

is it worth changing it so the sorting logic just mutates a local copy of the identities?

Thought about that. For now, it's fine, since, we don't really integration test that flag, and I don't really think we ever will because it's not worth the resource. So I decided to just leave it. It's a quicker and smaller fix, easier to review too instead of me adding a copy(dst, src) shenanigans. :)

[Skarlso]

Yay, all iamseriveaccount tests passed:

Ran 11 of 43 Specs in 1286.224 seconds
SUCCESS! -- 11 Passed | 0 Failed | 0 Pending | 32 Skipped
PASS | FOCUSED