Fix Bug in Asset Server Error Message Formatter (#1340)

bevyengine · Jan 29, 2021 · af67231 · af67231
1 parent 8e69ff2
commit af67231
Showing 1 changed file with 23 additions and 1 deletion.
diff --git a/crates/bevy_asset/src/asset_server.rs b/crates/bevy_asset/src/asset_server.rs
@@ -30,7 +30,7 @@ pub enum AssetServerError {
 }
 
 fn format_missing_asset_ext(exts: &[String]) -> String {
-    if exts.is_empty() {
+    if !exts.is_empty() {
         format!(
             " for the following extension{}: {}",
             if exts.len() > 1 { "s" } else { "" },
@@ -572,6 +572,28 @@ mod test {
         )
     }
 
+    #[test]
+    fn missing_asset_loader_error_messages() {
+        assert_eq!(
+            AssetServerError::MissingAssetLoader { extensions: vec![] }.to_string(),
+            "no `AssetLoader` found"
+        );
+        assert_eq!(
+            AssetServerError::MissingAssetLoader {
+                extensions: vec!["png".into()]
+            }
+            .to_string(),
+            "no `AssetLoader` found for the following extension: png"
+        );
+        assert_eq!(
+            AssetServerError::MissingAssetLoader {
+                extensions: vec!["1.2.png".into(), "2.png".into(), "png".into()]
+            }
+            .to_string(),
+            "no `AssetLoader` found for the following extensions: 1.2.png, 2.png, png"
+        );
+    }
+
     #[test]
     fn filename_with_dots() {
         let asset_server = setup();