Fix Bug in Asset Server Error Message Formatter

bevyengine · Jan 29, 2021 · 76db027 · 76db027
1 parent cc9ed52
commit 76db027
Showing 1 changed file with 30 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,35 @@ mod test {
         )
     }
 
+    #[test]
+    fn missing_asset_loader_error_messages() {
+        assert_eq!(
+            format!(
+                "{}",
+                AssetServerError::MissingAssetLoader { extensions: vec![] }
+            ),
+            "no `AssetLoader` found"
+        );
+        assert_eq!(
+            format!(
+                "{}",
+                AssetServerError::MissingAssetLoader {
+                    extensions: vec!["png".into()]
+                }
+            ),
+            "no `AssetLoader` found for the following extension: png"
+        );
+        assert_eq!(
+            format!(
+                "{}",
+                AssetServerError::MissingAssetLoader {
+                    extensions: vec!["1.2.png".into(), "2.png".into(), "png".into()]
+                }
+            ),
+            "no `AssetLoader` found for the following extensions: 1.2.png, 2.png, png"
+        );
+    }
+
     #[test]
     fn filename_with_dots() {
         let asset_server = setup();