client: Fix Client::update on empty registry storage

bytecodealliance · Jul 24, 2023 · b6f0f40 · b6f0f40
1 parent 1cbd8d5
commit b6f0f40
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/crates/client/src/storage/fs.rs b/crates/client/src/storage/fs.rs
@@ -97,6 +97,10 @@ impl RegistryStorage for FileSystemRegistryStorage {
         let mut packages = Vec::new();
 
         let packages_dir = self.base_dir.join(PACKAGE_LOGS_DIR);
+        if !packages_dir.exists() {
+            return Ok(vec![]);
+        }
+
         for entry in WalkDir::new(&packages_dir) {
             let entry = entry.with_context(|| {
                 anyhow!(

diff --git a/tests/client.rs b/tests/client.rs
@@ -80,6 +80,9 @@ async fn client_incrementally_fetches() -> Result<()> {
     // Recreate the client with the same config
     let client = create_client(&config)?;
 
+    // Regression test: update on empty registry storage
+    client.update().await?;
+
     // Fetch the package log
     client.upsert([&id]).await?;