test: check migration type as well

Author: crepererum

sqlx-core/src/migrate/migration_type.rs

@@ -13,6 +13,18 @@ pub enum MigrationType {
     ReversibleDown,
 }
 
+// It seems that deriving PartialEq leads to some weird lifetime bug I don't know why:
+//
+//   `sqlx-core/src/postgres/copy.rs` then fails to compile with:
+//   `associated function was supposed to return data with lifetime `'c` but it is returning data with lifetime `'1``
+//
+// Using a manual implementation seems to work however
+impl PartialEq for MigrationType {
+    fn eq(&self, other: &Self) -> bool {
+        core::mem::discriminant(self) == core::mem::discriminant(other)
+    }
+}
+
 impl MigrationType {
     pub fn from_filename(filename: &str) -> Self {
         if filename.ends_with(MigrationType::ReversibleUp.suffix()) {

tests/migrate/macro.rs

@@ -12,6 +12,7 @@ async fn same_output() -> anyhow::Result<()> {
     for (e, r) in EMBEDDED.iter().zip(runtime.iter()) {
         assert_eq!(e.version, r.version);
         assert_eq!(e.description, r.description);
+        assert_eq!(e.migration_type, r.migration_type);
         assert_eq!(e.sql, r.sql);
         assert_eq!(e.checksum, r.checksum);
     }