refactor(core)!: rename into_item to build for Builder

Author: gadomski

core/README.md

@@ -50,7 +50,7 @@ Then, items created from rasters will include the projection and raster extensio
 #[cfg(feature = "gdal")]
 {
     use stac::{extensions::{Raster, Projection}, Extensions, item::Builder};
-    let item = Builder::new("an-id").asset("data", "assets/dataset_geo.tif").into_item().unwrap();
+    let item = Builder::new("an-id").asset("data", "assets/dataset_geo.tif").build().unwrap();
     assert!(item.has_extension::<Projection>());
     assert!(item.has_extension::<Raster>());
 }

core/src/gdal.rs

@@ -172,7 +172,7 @@ mod tests {
     fn raster_data_type() {
         let mut item = Builder::new("an-id")
             .asset("data", "assets/dataset.tif")
-            .into_item()
+            .build()
             .unwrap();
         super::update_item(&mut item, false, true).unwrap();
         assert!(item.has_extension::<Raster>());
@@ -193,7 +193,7 @@ mod tests {
     fn raster_spatial_resolution() {
         let mut item = Builder::new("an-id")
             .asset("data", "assets/dataset_geo.tif")
-            .into_item()
+            .build()
             .unwrap();
         super::update_item(&mut item, false, true).unwrap();
         let raster: Raster = item
@@ -213,7 +213,7 @@ mod tests {
     fn projection() {
         let mut item = Builder::new("an-id")
             .asset("data", "assets/dataset_geo.tif")
-            .into_item()
+            .build()
             .unwrap();
         super::update_item(&mut item, false, true).unwrap();
         let projection: Projection = item.extension().unwrap().unwrap();

core/src/item.rs

@@ -306,17 +306,17 @@ impl Builder {
         self
     }
 
-    /// Creates an [Item] by consuming this builder.
+    /// Builds an [Item] from this builder.
     ///
     /// # Examples
     ///
     /// ```
     /// use stac::item::Builder;
     /// let builder = Builder::new("an-id").asset("data", "assets/dataset.tif");
-    /// let item = builder.into_item().unwrap();
+    /// let item = builder.build().unwrap();
     /// assert_eq!(item.assets.len(), 1);
     /// ```
-    pub fn into_item(self) -> Result<Item> {
+    pub fn build(self) -> Result<Item> {
         let mut item = Item::new(self.id);
         for (key, mut asset) in self.assets {
             if Url::parse(&asset.href).is_err() && self.canonicalize_paths {
@@ -889,7 +889,7 @@ mod tests {
     #[test]
     fn builder() {
         let builder = Builder::new("an-id").asset("data", "assets/dataset.tif");
-        let item = builder.into_item().unwrap();
+        let item = builder.build().unwrap();
         assert_eq!(item.assets.len(), 1);
         let asset = item.assets.get("data").unwrap();
         assert!(asset
@@ -902,7 +902,7 @@ mod tests {
         let builder = Builder::new("an-id")
             .canonicalize_paths(false)
             .asset("data", "assets/dataset.tif");
-        let item = builder.into_item().unwrap();
+        let item = builder.build().unwrap();
         let asset = item.assets.get("data").unwrap();
         assert_eq!(asset.href, "assets/dataset.tif");
     }
@@ -911,7 +911,7 @@ mod tests {
     fn builder_asset_roles() {
         let item = Builder::new("an-id")
             .asset("data", Asset::new("assets/dataset.tif").role("data"))
-            .into_item()
+            .build()
             .unwrap();
         let asset = item.assets.get("data").unwrap();
         assert_eq!(asset.roles, vec!["data"]);
@@ -921,7 +921,7 @@ mod tests {
     fn builder_uses_gdal() {
         let item = Builder::new("an-id")
             .asset("data", "assets/dataset.tif")
-            .into_item()
+            .build()
             .unwrap();
         if cfg!(feature = "gdal") {
             assert!(item.has_extension::<Raster>());