document tests

paleolimbot · paleolimbot · commit f7e69fb013fe · 2025-09-30T22:09:34.000-05:00
diff --git a/parquet/tests/geospatial.rs b/parquet/tests/geospatial.rs
@@ -40,6 +40,7 @@ fn read_metadata(geospatial_test_file: &str) -> ParquetMetaData {
 
 #[test]
 fn test_read_logical_type() {
+    // Some crs values are short strings
     let expected_logical_type = [
         ("crs-default.parquet", LogicalType::Geometry { crs: None }),
         (
@@ -75,6 +76,8 @@ fn test_read_logical_type() {
         assert_eq!(logical_type, expected_type);
     }
 
+    // The crs value may also contain arbitrary values (in this case some JSON
+    // a bit too lengthy to type out)
     let metadata = read_metadata("crs-arbitrary-value.parquet");
     let logical_type = metadata
         .file_metadata()

Original file line number	Diff line number	Diff line change
`@@ -40,6 +40,7 @@ fn read_metadata(geospatial_test_file: &str) -> ParquetMetaData {`
`40`	`40`
`41`	`41`	`#[test]`
`42`	`42`	`fn test_read_logical_type() {`
	`43`	`+ // Some crs values are short strings`
`43`	`44`	`let expected_logical_type = [`
`44`	`45`	`("crs-default.parquet", LogicalType::Geometry { crs: None }),`
`45`	`46`	`(`
`@@ -75,6 +76,8 @@ fn test_read_logical_type() {`
`75`	`76`	`assert_eq!(logical_type, expected_type);`
`76`	`77`	`}`
`77`	`78`
	`79`	`+ // The crs value may also contain arbitrary values (in this case some JSON`
	`80`	`+ // a bit too lengthy to type out)`
`78`	`81`	`let metadata = read_metadata("crs-arbitrary-value.parquet");`
`79`	`82`	`let logical_type = metadata`
`80`	`83`	`.file_metadata()`