dfinity · NikolaMilosa · Aug 8, 2024 · Aug 8, 2024
@@ -58,7 +58,13 @@ impl Args {
             .ok_or(anyhow::anyhow!("No home dir present"))?
             .join(PathBuf::from_str(XNET_TESTING_IDENTITY_PATH)?);
         match path.exists() {
-            true => Ok(()),
+            true => {
+                let metadata = path.metadata()?;
+                match metadata.len() {
+                    0 => anyhow::bail!("Xnet-testing identity is present on path {} but is empty", path.display()),
+                    _ => Ok(()),
+                }
+            }
             false => anyhow::bail!("Missing xnet-testing identity on path: {}", path.display()),
         }
     }

@@ -37,6 +37,9 @@ async fn main() -> anyhow::Result<()> {
         .map_err(|e| anyhow::anyhow!("Checking connectivity failed: {}", e.to_string()))?;
 
     let args = Args::parse();
+    if args.version_to_qualify.is_empty() {
+        anyhow::bail!("Version to qualify is required, but empty string is passed.");
+    }
     info!("Running qualification for {}", args.version_to_qualify);
     info!("Generating keys for farm testnets...");
     let (neuron_id, private_key_pem) = args.ensure_test_key()?;