fix(migrations): fix migrations with targets that are out of bounds

[samuelmasse]

Fixes a bug where asking the server to migrate up to a version lower than the current server version, or migrate down to a version higher than the current server version, would result in 0 migrations getting run and the server version getting wrongly updated in the meta table

Closes DEV-2137

[linear]

DEV-2137 Fix messaging migration system allowing to up migrate to a lower version

This is a loophole in the migration system. You can ask to migrate up to a lower version, and the migration system will always succeed change the server version to the target version even if it didn't run the required migrations

[laurentlp]

So if you do an up migration to a version (v1.5.6) higher than the latest one, will it save v1.5.6 in the meta table? Same goes for the down migration.

Also, can you make sure to update the PR title so it better reflects what is this fix

[samuelmasse]

@laurentlp Good point I added a check to prevent migration to a version higher than the version of the server's package.

[samuelmasse]

For the down migration, you could technically migrate down to 0.0.0 but that wouldn't cause any missing migration so it's correct behavior