errors: Reorder component-unavailable message parts

The ordering of some of the parts of the component-unavailable
error message was unfortunate, resulting in confusion.  This
reorders those so that errors are slightly clearer.

Fixes: #1768

Signed-off-by: Daniel Silverstone <[email protected]>

Author: kinnison

src/errors.rs

@@ -267,7 +267,7 @@ error_chain! {
         }
         RequestedComponentsUnavailable(c: Vec<Component>, manifest: Manifest, toolchain: String) {
             description("some requested components are unavailable to download")
-            display("{} for channel '{}'", component_unavailable_msg(&c, &manifest), toolchain)
+            display("{} for channel '{}'\n{}", component_unavailable_msg(&c, &manifest), toolchain, TOOLSTATE_MSG)
         }
         UnknownMetadataVersion(v: String) {
             description("unknown metadata version")
@@ -341,19 +341,11 @@ fn component_unavailable_msg(cs: &[Component], manifest: &Manifest) -> String {
         if same_target {
             let mut cs_strs = cs.iter().map(|c| format!("'{}'", c.short_name(manifest)));
             let cs_str = cs_strs.join(", ");
-            let _ = write!(
-                buf,
-                "some components unavailable for download: {}\n{}",
-                cs_str, TOOLSTATE_MSG,
-            );
+            let _ = write!(buf, "some components unavailable for download: {}", cs_str,);
         } else {
             let mut cs_strs = cs.iter().map(|c| c.description(manifest));
             let cs_str = cs_strs.join(", ");
-            let _ = write!(
-                buf,
-                "some components unavailable for download: {}\n{}",
-                cs_str, TOOLSTATE_MSG,
-            );
+            let _ = write!(buf, "some components unavailable for download: {}", cs_str,);
         }
     }
 

tests/cli-misc.rs

@@ -699,7 +699,7 @@ fn update_unavailable_rustc() {
             config,
             &["rustup", "update", "nightly"],
             format!(
-                "some components unavailable for download: 'rustc', 'cargo'\n{}",
+                "some components unavailable for download: 'rustc', 'cargo' for channel 'nightly'\n{}",
                 TOOLSTATE_MSG
             )
             .as_str(),