fix: list(None) is logically equivalent to list("/") (#19)

Signed-off-by: R. Tyler Croy <[email protected]>

Author: rtyler

src/lib.rs

@@ -330,7 +330,7 @@ impl ObjectStore for HdfsObjectStore {
         let status_stream = self
             .client
             .list_status_iter(
-                &prefix.map(make_absolute_dir).unwrap_or("".to_string()),
+                &prefix.map(make_absolute_dir).unwrap_or("/".to_string()),
                 true,
             )
             .into_stream()

tests/test_object_store.rs

@@ -79,6 +79,11 @@ mod test {
         assert_eq!(list.len(), 1);
         assert_eq!(list[0].as_ref().unwrap().location, Path::from("/testfile"));
 
+        // A list of None should be logically equivalent to /
+        let list: Vec<object_store::Result<ObjectMeta>> = store.list(None).collect().await;
+        assert_eq!(list.len(), 1);
+        assert_eq!(list[0].as_ref().unwrap().location, Path::from("/testfile"));
+
         // Listing of a prefix that doesn't exist should return an empty result, not an error
         assert_eq!(
             store.list(Some(&Path::from("/doesnt/exist"))).count().await,