Error in migrate_db_chrysalis_to_stardust() if no chrysalis data exists

bindings/nodejs/CHANGELOG.md

@@ -19,7 +19,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Security -->
 
-## 1.0.10 - 2023-mm-dd
+## 1.0.10 - 2023-09-12
+
+### Changed
+
+- `migrateDbChrysalisToStardust()` returns an error if no chrysalis data was found;
 
 ### Fixed
 

bindings/nodejs/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@iota/sdk",
-    "version": "1.0.9",
+    "version": "1.0.10",
     "description": "Node.js binding to the IOTA SDK library",
     "main": "out/index.js",
     "types": "out/index.d.ts",

sdk/CHANGELOG.md

@@ -19,6 +19,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Security -->
 
+## 1.0.4 - 2023-MM-DD
+
+### Changed
+
+- `migrate_db_chrysalis_to_stardust()` returns an error if no chrysalis data was found;
+
 ## 1.0.3 - 2023-09-07
 
 ### Added

sdk/src/wallet/migration/chrysalis.rs

@@ -71,7 +71,17 @@ pub async fn migrate_db_chrysalis_to_stardust(
     // `/db` will be appended to the chrysalis storage path, because that's how it was done in the chrysalis wallet
     let chrysalis_storage_path = &(*storage_path_string).join("db");
 
+    if !chrysalis_storage_path.is_dir() {
+        return Err(crate::wallet::Error::Migration(
+            "no chrysalis data to migrate".to_string(),
+        ));
+    }
     let chrysalis_data = get_chrysalis_data(chrysalis_storage_path, password)?;
+    if chrysalis_data.is_empty() {
+        return Err(crate::wallet::Error::Migration(
+            "no chrysalis data to migrate".to_string(),
+        ));
+    }
 
     // create new accounts base on previous data
     let (new_accounts, secret_manager_dto) = migrate_from_chrysalis_data(&chrysalis_data, &storage_path_string, false)?;

sdk/tests/wallet/chrysalis_migration.rs

@@ -348,6 +348,27 @@ async fn migrate_chrysalis_stronghold() -> Result<()> {
     tear_down(storage_path)
 }
 
+#[tokio::test]
+async fn migrate_empty_chrysalis_db() -> Result<()> {
+    iota_stronghold::engine::snapshot::try_set_encrypt_work_factor(0).unwrap();
+    let storage_path = "migrate_empty_chrysalis_db";
+    setup(storage_path)?;
+
+    // Copy db so the original doesn't get modified
+    copy_folder("./tests/wallet/fixtures/check_existing_4_db_test", storage_path).unwrap();
+
+    assert!(matches!(
+        migrate_db_chrysalis_to_stardust("migrate_empty_chrysalis_db", None, None).await,
+        Err(iota_sdk::wallet::error::Error::Migration(msg)) if msg == "no chrysalis data to migrate"
+    ));
+
+    // stardust wallet data is still there
+    let wallet = Wallet::builder().with_storage_path(storage_path).finish().await?;
+    assert_eq!(wallet.get_accounts().await?.len(), 1);
+
+    tear_down(storage_path)
+}
+
 fn copy_folder(src: impl AsRef<Path>, dest: impl AsRef<Path>) -> io::Result<()> {
     fs::create_dir_all(&dest)?;
     for entry in fs::read_dir(src)? {