fix: python field tests

delta-io · Nov 5, 2023 · 12fa820 · 12fa820
1 parent 9aa301e
commit 12fa820
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/crates/deltalake-core/src/kernel/schema.rs b/crates/deltalake-core/src/kernel/schema.rs
@@ -458,7 +458,7 @@ impl Display for PrimitiveType {
             PrimitiveType::Date => write!(f, "date"),
             PrimitiveType::Timestamp => write!(f, "timestamp"),
             PrimitiveType::Decimal(precision, scale) => {
-                write!(f, "decimal({}, {})", precision, scale)
+                write!(f, "decimal({},{})", precision, scale)
             }
         }
     }

diff --git a/python/tests/test_schema.py b/python/tests/test_schema.py
@@ -175,7 +175,10 @@ def test_delta_field():
         assert field.name == name
         assert field.type == (PrimitiveType(ty) if isinstance(ty, str) else ty)
         assert field.nullable == nullable
-        assert field.metadata == (metadata or {})
+        if metadata:
+            assert json.loads(field.metadata["x"]) == {"y": 3}
+        else:
+            assert field.metadata == {}
 
         # Field metadata doesn't roundtrip currently
         # See: https://github.com/apache/arrow-rs/issues/478