Fix index test in LocalesLocationAddIndexUpdateTest::testExistingIndex

I think given we have to sort the index on Postgres the assertions are largely pointless.

This looks great now.

Why not just check that the index exists after the update? We should trust that the database driver has tests that ensure an index is actually structured correctly, we don't need to re-assert that here.

Opened 🐛 Postrges implementation of introspectIndexSchema is incorrect Active to fix the postgres schema inspection issue - I think the postgres implementation is incorrect and sorting by column name makes the method less valuable.

@longwave pointed out we should be able to trust our APIs and not assrt against the internals - so removed the assertions on index structure.

Looks good to me.

Committed/pushed to 11.x and cherry-picked back through to 10.3.x, thanks!

catch → closed merge request !8654

Automatically closed - issue fixed for 2 weeks with no activity.