Merge pull request #286 from shepmaster/snafu-suffix

shepmaster · Jul 5, 2021 · 43257dd · 43257dd
2 parents 7fe0965 + 947eab3
commit 43257dd
Show file tree

Hide file tree

Showing 59 changed files with 522 additions and 312 deletions.
diff --git a/README.md b/README.md
@@ -26,9 +26,9 @@ type Result<T, E = Error> = std::result::Result<T, E>;
 
 fn process_data() -> Result<()> {
     let path = "config.toml";
-    let configuration = fs::read_to_string(path).context(ReadConfiguration { path })?;
+    let configuration = fs::read_to_string(path).context(ReadConfigurationSnafu { path })?;
     let path = unpack_config(&configuration);
-    fs::write(&path, b"My complex calculation").context(WriteResult { path })?;
+    fs::write(&path, b"My complex calculation").context(WriteResultSnafu { path })?;
     Ok(())
 }
 

diff --git a/compatibility-tests/backtrace-shim/tests/backtrace.rs b/compatibility-tests/backtrace-shim/tests/backtrace.rs
@@ -8,12 +8,12 @@ enum Error {
 type Result<T, E = Error> = std::result::Result<T, E>;
 
 fn check_less_than(user_id: i32) -> Result<()> {
-    ensure!(user_id >= 42, InvalidUser { user_id });
+    ensure!(user_id >= 42, InvalidUserSnafu { user_id });
     Ok(())
 }
 
 fn check_greater_than(user_id: i32) -> Result<()> {
-    ensure!(user_id <= 42, InvalidUser { user_id });
+    ensure!(user_id <= 42, InvalidUserSnafu { user_id });
     Ok(())
 }
 

diff --git a/compatibility-tests/backtrace-shim/tests/backtrace_delegation.rs b/compatibility-tests/backtrace-shim/tests/backtrace_delegation.rs
@@ -9,7 +9,7 @@ mod house {
     }
 
     pub fn answer_telephone() -> Result<(), Error> {
-        Fatal.fail()
+        FatalSnafu.fail()
     }
 }
 
@@ -28,7 +28,7 @@ enum Error {
 }
 
 fn delegate_example() -> Result<(), Error> {
-    house::answer_telephone().context(MovieTrope)?;
+    house::answer_telephone().context(MovieTropeSnafu)?;
 
     Ok(())
 }
@@ -47,7 +47,7 @@ fn backtrace_comes_from_delegated_error() {
 }
 
 fn delegate_and_rename_example() -> Result<(), Error> {
-    house::answer_telephone().context(SourceAndBacktraceAttrs)
+    house::answer_telephone().context(SourceAndBacktraceAttrsSnafu)
 }
 
 #[test]

diff --git a/compatibility-tests/backtraces-impl-backtrace-crate/src/lib.rs b/compatibility-tests/backtraces-impl-backtrace-crate/src/lib.rs
@@ -9,7 +9,7 @@ enum Error {
 type Result<T, E = Error> = std::result::Result<T, E>;
 
 fn example() -> Result<()> {
-    WithBacktrace.fail()
+    WithBacktraceSnafu.fail()
 }
 
 #[test]

diff --git a/compatibility-tests/backtraces-impl-std/src/lib.rs b/compatibility-tests/backtraces-impl-std/src/lib.rs
@@ -11,7 +11,7 @@ enum Error {
 type Result<T, E = Error> = std::result::Result<T, E>;
 
 fn example() -> Result<()> {
-    WithBacktrace.fail()
+    WithBacktraceSnafu.fail()
 }
 
 #[test]

diff --git a/compatibility-tests/compile-fail/tests/ui/structs/visibility.rs b/compatibility-tests/compile-fail/tests/ui/structs/visibility.rs
@@ -8,7 +8,7 @@ pub mod inner {
 }
 
 fn private_is_default() {
-    let _ = inner::Context.build();
+    let _ = inner::Snafu.build();
 }
 
 fn main() {}
diff --git a/compatibility-tests/compile-fail/tests/ui/structs/visibility.stderr b/compatibility-tests/compile-fail/tests/ui/structs/visibility.stderr
@@ -1,18 +1,18 @@
-error[E0603]: unit struct `Context` is private
+error[E0603]: unit struct `Snafu` is private
   --> $DIR/visibility.rs:11:20
    |
-11 |     let _ = inner::Context.build();
-   |                    ^^^^^^^ private unit struct
+11 |     let _ = inner::Snafu.build();
+   |                    ^^^^^ private unit struct
    |
-note: the unit struct `Context` is defined here
+note: the unit struct `Snafu` is defined here
   --> $DIR/visibility.rs:6:21
    |
 6  |     #[derive(Debug, Snafu)]
    |                     ^^^^^
    = note: this error originates in a derive macro (in Nightly builds, run with -Z macro-backtrace for more info)
 
 error[E0624]: associated function `build` is private
-  --> $DIR/visibility.rs:11:28
+  --> $DIR/visibility.rs:11:26
    |
-11 |     let _ = inner::Context.build();
-   |                            ^^^^^ private associated function
+11 |     let _ = inner::Snafu.build();
+   |                          ^^^^^ private associated function
diff --git a/compatibility-tests/compile-fail/tests/ui/visibility.rs b/compatibility-tests/compile-fail/tests/ui/visibility.rs
@@ -16,16 +16,16 @@ mod outer {
     }
 
     fn private_is_applied() {
-        let _ = self::inner::Private.build();
+        let _ = self::inner::PrivateSnafu.build();
     }
 }
 
 fn pub_in_path_is_applied() {
-    let _ = self::outer::inner::PubInPath.build();
+    let _ = self::outer::inner::PubInPathSnafu.build();
 }
 
 fn private_is_applied() {
-    let _ = self::outer::inner::Private.build();
+    let _ = self::outer::inner::PrivateSnafu.build();
 }
 
 fn main() {}
diff --git a/compatibility-tests/compile-fail/tests/ui/visibility.stderr b/compatibility-tests/compile-fail/tests/ui/visibility.stderr
@@ -1,56 +1,56 @@
-error[E0603]: unit struct `Private` is private
+error[E0603]: unit struct `PrivateSnafu` is private
   --> $DIR/visibility.rs:19:30
    |
-19 |         let _ = self::inner::Private.build();
-   |                              ^^^^^^^ private unit struct
+19 |         let _ = self::inner::PrivateSnafu.build();
+   |                              ^^^^^^^^^^^^ private unit struct
    |
-note: the unit struct `Private` is defined here
+note: the unit struct `PrivateSnafu` is defined here
   --> $DIR/visibility.rs:7:25
    |
 7  |         #[derive(Debug, Snafu)]
    |                         ^^^^^
    = note: this error originates in a derive macro (in Nightly builds, run with -Z macro-backtrace for more info)
 
-error[E0603]: unit struct `PubInPath` is private
+error[E0603]: unit struct `PubInPathSnafu` is private
   --> $DIR/visibility.rs:24:33
    |
-24 |     let _ = self::outer::inner::PubInPath.build();
-   |                                 ^^^^^^^^^ private unit struct
+24 |     let _ = self::outer::inner::PubInPathSnafu.build();
+   |                                 ^^^^^^^^^^^^^^ private unit struct
    |
-note: the unit struct `PubInPath` is defined here
+note: the unit struct `PubInPathSnafu` is defined here
   --> $DIR/visibility.rs:7:25
    |
 7  |         #[derive(Debug, Snafu)]
    |                         ^^^^^
    = note: this error originates in a derive macro (in Nightly builds, run with -Z macro-backtrace for more info)
 
-error[E0603]: unit struct `Private` is private
+error[E0603]: unit struct `PrivateSnafu` is private
   --> $DIR/visibility.rs:28:33
    |
-28 |     let _ = self::outer::inner::Private.build();
-   |                                 ^^^^^^^ private unit struct
+28 |     let _ = self::outer::inner::PrivateSnafu.build();
+   |                                 ^^^^^^^^^^^^ private unit struct
    |
-note: the unit struct `Private` is defined here
+note: the unit struct `PrivateSnafu` is defined here
   --> $DIR/visibility.rs:7:25
    |
 7  |         #[derive(Debug, Snafu)]
    |                         ^^^^^
    = note: this error originates in a derive macro (in Nightly builds, run with -Z macro-backtrace for more info)
 
 error[E0624]: associated function `build` is private
-  --> $DIR/visibility.rs:19:38
+  --> $DIR/visibility.rs:19:43
    |
-19 |         let _ = self::inner::Private.build();
-   |                                      ^^^^^ private associated function
+19 |         let _ = self::inner::PrivateSnafu.build();
+   |                                           ^^^^^ private associated function
 
 error[E0624]: associated function `build` is private
-  --> $DIR/visibility.rs:24:43
+  --> $DIR/visibility.rs:24:48
    |
-24 |     let _ = self::outer::inner::PubInPath.build();
-   |                                           ^^^^^ private associated function
+24 |     let _ = self::outer::inner::PubInPathSnafu.build();
+   |                                                ^^^^^ private associated function
 
 error[E0624]: associated function `build` is private
-  --> $DIR/visibility.rs:28:41
+  --> $DIR/visibility.rs:28:46
    |
-28 |     let _ = self::outer::inner::Private.build();
-   |                                         ^^^^^ private associated function
+28 |     let _ = self::outer::inner::PrivateSnafu.build();
+   |                                              ^^^^^ private associated function
diff --git a/compatibility-tests/futures/src/lib.rs b/compatibility-tests/futures/src/lib.rs
@@ -10,7 +10,7 @@ mod api {
     }
 
     pub async fn fetch_page(url: &str) -> Result<String, Error> {
-        InvalidUrl { url }.fail()
+        InvalidUrlSnafu { url }.fail()
     }
 
     pub fn keep_fetching_page<'u>(url: &'u str) -> impl TryStream<Ok = String, Error = Error> + 'u {
@@ -51,11 +51,11 @@ enum Error {
 async fn load_stock_data_sequential() -> Result<String, Error> {
     let apple = api::fetch_page("apple")
         .await
-        .context(UnableToLoadAppleStock)?;
+        .context(UnableToLoadAppleStockSnafu)?;
 
     let google = api::fetch_page("google")
         .await
-        .with_context(|| UnableToLoadGoogleStock {
+        .with_context(|| UnableToLoadGoogleStockSnafu {
             name: String::from("sequential"),
         })?;
 
@@ -73,8 +73,8 @@ async fn load_stock_data_sequential() -> Result<String, Error> {
 
 // Can be used as a `Future` combinator
 async fn load_stock_data_concurrent() -> Result<String, Error> {
-    let apple = api::fetch_page("apple").context(UnableToLoadAppleStock);
-    let google = api::fetch_page("google").with_context(|| UnableToLoadGoogleStock {
+    let apple = api::fetch_page("apple").context(UnableToLoadAppleStockSnafu);
+    let google = api::fetch_page("google").with_context(|| UnableToLoadGoogleStockSnafu {
         name: String::from("concurrent"),
     });
     let other_1 = api::fetch_page("other_1").whatever_context::<_, Error>("Oh no!");
@@ -91,11 +91,11 @@ async fn load_stock_data_concurrent() -> Result<String, Error> {
 // Return values of the combinators implement `Future`
 async fn load_stock_data_sequential_again() -> Result<String, Error> {
     let apple = api::fetch_page("apple")
-        .context(UnableToLoadAppleStock)
+        .context(UnableToLoadAppleStockSnafu)
         .await?;
 
     let google = api::fetch_page("google")
-        .with_context(|| UnableToLoadGoogleStock {
+        .with_context(|| UnableToLoadGoogleStockSnafu {
             name: String::from("sequential"),
         })
         .await?;
@@ -114,8 +114,8 @@ async fn load_stock_data_sequential_again() -> Result<String, Error> {
 
 // Can be used as a `Stream` combinator
 async fn load_stock_data_series() -> Result<String, Error> {
-    let apple = api::keep_fetching_page("apple").context(UnableToLoadAppleStock);
-    let google = api::keep_fetching_page("google").with_context(|| UnableToLoadGoogleStock {
+    let apple = api::keep_fetching_page("apple").context(UnableToLoadAppleStockSnafu);
+    let google = api::keep_fetching_page("google").with_context(|| UnableToLoadGoogleStockSnafu {
         name: String::from("stream"),
     });
     let other_1 = api::keep_fetching_page("other_1").whatever_context("Oh no!");

diff --git a/compatibility-tests/v1_34/src/lib.rs b/compatibility-tests/v1_34/src/lib.rs
@@ -26,11 +26,11 @@ mod enum_style {
     type Result<T, E = Error> = std::result::Result<T, E>;
 
     fn create_without_source() -> Result<()> {
-        WithoutSource { id: 42 }.fail()
+        WithoutSourceSnafu { id: 42 }.fail()
     }
 
     fn create_with_source() -> Result<()> {
-        io_failure().context(WithSource { id: 42 })
+        io_failure().context(WithSourceSnafu { id: 42 })
     }
 
     #[test]
@@ -59,11 +59,11 @@ mod struct_style {
     }
 
     fn create_without_source() -> Result<(), WithoutSource> {
-        WithoutSourceContext { id: 42 }.fail()
+        WithoutSourceSnafu { id: 42 }.fail()
     }
 
     fn create_with_source() -> Result<(), WithSource> {
-        io_failure().context(WithSourceContext { id: 42 })
+        io_failure().context(WithSourceSnafu { id: 42 })
     }
 
     #[test]
@@ -86,7 +86,7 @@ mod opaque_style {
     struct Opaque(Dummy);
 
     fn create() -> Result<(), Opaque> {
-        Ok(DummyContext.fail()?)
+        Ok(DummySnafu.fail()?)
     }
 
     #[test]