rename PrepareError::Unknown to PrepareError::Uncategorized

to match std::io::ErrorKind::Uncategorized

Author: Skgland

src/build.rs

@@ -196,7 +196,7 @@ impl BuildDirectory {
         let mut prepare = Prepare::new(&self.workspace, toolchain, krate, &source_dir, patches);
         prepare.prepare().map_err(|err| {
             if err.downcast_ref::<PrepareError>().is_none() {
-                err.context(PrepareError::Unknown)
+                err.context(PrepareError::Uncategorized)
             } else {
                 err
             }

src/prepare.rs

@@ -390,10 +390,10 @@ pub enum PrepareError {
     /// Some of the dependencies do not exist anymore.
     #[error("the crate depends on missing dependencies: \n\n{0}")]
     MissingDependencies(String),
-    /// Some not further detaild error
+    /// Uncategorized error
     #[doc(hidden)]
-    #[error("prepare failed without further details")]
-    Unknown,
+    #[error("uncategorized prepare error")]
+    Uncategorized,
 }
 
 #[cfg(test)]

tests/buildtest/mod.rs

@@ -261,42 +261,42 @@ test_prepare_error_stderr!(
     "error: no matching package named `macro` found"
 );
 
-test_prepare_unknown_err!(
+test_prepare_uncategorized_err!(
     test_invalid_cargotoml_content_deps,
     "invalid-cargotoml-content-deps",
     BrokenDependencies,
     "failed to parse the version requirement `0.11\t` for dependency `parking_lot`"
 );
 
-test_prepare_unknown_err!(
+test_prepare_uncategorized_err!(
     test_invalid_cargotoml_syntax_deps,
     "invalid-cargotoml-syntax-deps",
     BrokenDependencies,
     "error: invalid table header"
 );
 
-test_prepare_unknown_err!(
+test_prepare_uncategorized_err!(
     test_invalid_lockfile_syntax,
     "invalid-lockfile-syntax",
     InvalidCargoLock,
     "error: failed to parse lock file at"
 );
 
-test_prepare_unknown_err!(
+test_prepare_uncategorized_err!(
     test_missing_deps_typo,
     "missing-deps-typo",
     MissingDependencies,
     "error: no matching package found"
 );
 
-test_prepare_unknown_err!(
+test_prepare_uncategorized_err!(
     test_invalid_cargotoml_cyclic_feature,
     "invalid-cargotoml-cyclic-feature",
     BrokenDependencies,
     "error: cyclic feature dependency: feature"
 );
 
-test_prepare_unknown_err!(
+test_prepare_uncategorized_err!(
     test_invalid_cargotoml_cyclic_package,
     "invalid-cargotoml-cyclic-package",
     BrokenDependencies,
@@ -309,42 +309,42 @@ test_prepare_error!(
     InvalidCargoTomlSyntax
 );
 
-test_prepare_unknown_err!(
+test_prepare_uncategorized_err!(
     test_invalid_cargotoml_missing_override,
     "invalid-cargotoml-missing-override",
     MissingDependencies,
     "no matching package for override `https://github.com/rust-lang/crates.io-index#[email protected]` found"
 );
 
-test_prepare_unknown_err!(
+test_prepare_uncategorized_err!(
     test_missing_deps_registry_version,
     "missing-deps-registry-version",
     MissingDependencies,
     "error: failed to select a version for the requirement `empty-library = \"=0.5.0\"`"
 );
 
-test_prepare_unknown_err!(
+test_prepare_uncategorized_err!(
     test_invalid_cargotoml_content_type_in_deps,
     "invalid-cargotoml-content-type-in-deps",
     BrokenDependencies,
     "error: invalid type: map, expected a string"
 );
 
-test_prepare_unknown_err!(
+test_prepare_uncategorized_err!(
     test_invalid_cargotoml_conflicting_links,
     "invalid-cargotoml-conflicting-links",
     InvalidCargoLock,
     "error: Attempting to resolve a dependency with more than one crate with links=ring-asm"
 );
 
-test_prepare_unknown_err!(
+test_prepare_uncategorized_err!(
     test_lockfile_collision,
     "lockfile-collision",
     BrokenDependencies,
     "error: package collision in the lockfile: packages lockfile-collision v0.1.0 "
 );
 
-test_prepare_unknown_err!(
+test_prepare_uncategorized_err!(
     test_invalid_cargotoml_missing_patch,
     "invalid-cargotoml-missing-patch",
     MissingDependencies,

tests/buildtest/runner.rs

@@ -131,7 +131,7 @@ macro_rules! test_prepare_error_stderr {
     };
 }
 
-macro_rules! test_prepare_unknown_err {
+macro_rules! test_prepare_uncategorized_err {
     ($name:ident, $krate:expr, $expected:ident $(,$expected_output:expr)?) => {
         $crate::buildtest::runner::test_prepare_error!($name, $krate, Unknown);
     };