add test for long crate-names & versions, fix reverse migration & com…

…ment it

migrations/20240519141105_crate-version-name-field-length.down.sql

@@ -1,2 +1,10 @@
+-- this reverse migration might fail when we have other records
+-- referencing releases with longer names or versions.
+-- If this has to be reverted we probably would have to manually run 
+-- db::delete::delete_crate and db::delete::delete_version for the releases 
+-- and crates.
+DELETE FROM crates WHERE LENGTH(name) > 255;
+DELETE FROM releases WHERE LENGTH(version) > 100;
+
 ALTER TABLE crates ALTER COLUMN name TYPE VARCHAR(255);
 ALTER TABLE releases ALTER COLUMN version TYPE VARCHAR(100);

src/db/add_package.rs

@@ -1110,4 +1110,42 @@ mod test {
             Ok(())
         })
     }
+
+    #[test]
+    fn test_long_crate_name() {
+        async_wrapper(|env| async move {
+            let mut conn = env.async_db().await.async_conn().await;
+
+            let name: String = "krate".repeat(100);
+            let crate_id = initialize_crate(&mut conn, &name).await?;
+
+            let db_name = sqlx::query_scalar!("SELECT name FROM crates WHERE id = $1", crate_id)
+                .fetch_one(&mut *conn)
+                .await?;
+
+            assert_eq!(db_name, name);
+
+            Ok(())
+        })
+    }
+
+    #[test]
+    fn test_long_relaase_version() {
+        async_wrapper(|env| async move {
+            let mut conn = env.async_db().await.async_conn().await;
+
+            let crate_id = initialize_crate(&mut conn, "krate").await?;
+            let version: String = "version".repeat(100);
+            let release_id = initialize_release(&mut conn, crate_id, &version).await?;
+
+            let db_version =
+                sqlx::query_scalar!("SELECT version FROM releases WHERE id = $1", release_id)
+                    .fetch_one(&mut *conn)
+                    .await?;
+
+            assert_eq!(db_version, version);
+
+            Ok(())
+        })
+    }
 }