Merge pull request #74 from yoshuawuyts/status-trait

Rename ResultExt to Status

Author: yoshuawuyts

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "http-types"
-version = "1.0.0"
+version = "1.0.1"
 license = "MIT OR Apache-2.0"
 repository = "https://github.com/http-rs/http-types"
 documentation = "https://docs.rs/http-types"

src/lib.rs

@@ -120,7 +120,7 @@ mod macros;
 mod method;
 mod request;
 mod response;
-mod result_ext;
+mod status;
 mod status_code;
 mod type_map;
 mod version;
@@ -130,7 +130,7 @@ pub use error::{Error, Result};
 pub use method::Method;
 pub use request::Request;
 pub use response::Response;
-pub use result_ext::ResultExt;
+pub use status::Status;
 pub use status_code::StatusCode;
 pub use version::Version;
 

src/result_ext.rs renamed to src/status.rs

@@ -2,10 +2,10 @@ use crate::{Error, StatusCode};
 use core::convert::{Infallible, TryInto};
 use std::error::Error as StdError;
 
-/// Provides the `status` method for `Result`.
+/// Provides the `status` method for `Result` and `Option`.
 ///
 /// This trait is sealed and cannot be implemented outside of `http-types`.
-pub trait ResultExt<T, E>: private::Sealed {
+pub trait Status<T, E>: private::Sealed {
     /// Wrap the error value with an additional status code.
     fn status<S>(self, status: S) -> Result<T, Error>
     where
@@ -21,7 +21,7 @@ pub trait ResultExt<T, E>: private::Sealed {
         F: FnOnce() -> S;
 }
 
-impl<T, E> ResultExt<T, E> for Result<T, E>
+impl<T, E> Status<T, E> for Result<T, E>
 where
     E: StdError + Send + Sync + 'static,
 {
@@ -49,7 +49,7 @@ where
     }
 }
 
-impl<T> ResultExt<T, Infallible> for Option<T> {
+impl<T> Status<T, Infallible> for Option<T> {
     fn status<S>(self, status: S) -> Result<T, Error>
     where
         S: TryInto<StatusCode>,

tests/error.rs

@@ -46,7 +46,7 @@ fn ensure_eq() {
 
 #[test]
 fn result_ext() {
-    use http_types::ResultExt;
+    use http_types::Status;
     fn run() -> http_types::Result<()> {
         let err = io::Error::new(io::ErrorKind::Other, "Oh no");
         Err(err).status(StatusCode::NotFound)?;
@@ -61,7 +61,7 @@ fn result_ext() {
 
 #[test]
 fn option_ext() {
-    use http_types::ResultExt;
+    use http_types::Status;
     fn run() -> http_types::Result<()> {
         None.status(StatusCode::NotFound)
     }