rename snapshot and make tests work cross-platform

prefix-dev · Dec 14, 2023 · bacc78b · bacc78b
1 parent a38b2bc
commit bacc78b
Show file tree

Hide file tree

Showing 3 changed files with 40 additions and 20 deletions.
diff --git a/src/recipe/parser.rs b/src/recipe/parser.rs
@@ -275,6 +275,7 @@ impl Recipe {
 #[cfg(test)]
 mod tests {
     use insta::assert_yaml_snapshot;
+    use rattler_conda_types::Platform;
 
     use crate::{assert_miette_snapshot, variant_config::ParseErrors};
 
@@ -283,12 +284,24 @@ mod tests {
     #[test]
     fn it_works() {
         let recipe = include_str!("../../examples/xtensor/recipe.yaml");
-        let recipe = Recipe::from_yaml(recipe, SelectorConfig::default());
-        assert!(recipe.is_ok());
-        #[cfg(target_family = "unix")]
-        insta::assert_debug_snapshot!(recipe.unwrap());
-        #[cfg(target_family = "windows")]
-        insta::assert_debug_snapshot!("recipe_windows", recipe.unwrap());
+
+        let selector_config_win = SelectorConfig {
+            target_platform: Platform::Win64,
+            ..SelectorConfig::default()
+        };
+
+        let selector_config_unix = SelectorConfig {
+            target_platform: Platform::Linux64,
+            ..SelectorConfig::default()
+        };
+
+        let unix_recipe = Recipe::from_yaml(recipe, selector_config_unix);
+        let win_recipe = Recipe::from_yaml(recipe, selector_config_win);
+        assert!(unix_recipe.is_ok());
+        assert!(win_recipe.is_ok());
+
+        insta::assert_debug_snapshot!("unix_recipe", unix_recipe.unwrap());
+        insta::assert_debug_snapshot!("recipe_windows", win_recipe.unwrap());
     }
 
     #[test]

diff --git a/src/recipe/snapshots/rattler_build__recipe__parser__tests__recipe_windows.snap b/src/recipe/snapshots/rattler_build__recipe__parser__tests__recipe_windows.snap
@@ -1,6 +1,6 @@
 ---
 source: src/recipe/parser.rs
-expression: recipe.unwrap()
+expression: win_recipe.unwrap()
 ---
 Recipe {
     package: Package {
@@ -254,18 +254,25 @@ Recipe {
             from_package: {},
         },
     },
-    test: Test {
-        imports: [],
-        commands: [
-            "if not exist %LIBRARY_PREFIX%\\include\\xtensor\\xarray.hpp (exit 1)",
-            "if not exist %LIBRARY_PREFIX%\\share\\cmake\\xtensor\\xtensorConfig.cmake (exit 1)",
-            "if not exist %LIBRARY_PREFIX%\\share\\cmake\\xtensor\\xtensorConfigVersion.cmake (exit 1)",
-        ],
-        requires: [],
-        source_files: [],
-        files: [],
-        package_contents: None,
-    },
+    tests: [
+        Command(
+            CommandsTest {
+                script: [
+                    "if not exist %LIBRARY_PREFIX%\\include\\xtensor\\xarray.hpp (exit 1)",
+                    "if not exist %LIBRARY_PREFIX%\\share\\cmake\\xtensor\\xtensorConfig.cmake (exit 1)",
+                    "if not exist %LIBRARY_PREFIX%\\share\\cmake\\xtensor\\xtensorConfigVersion.cmake (exit 1)",
+                ],
+                requirements: CommandsTestRequirements {
+                    run: [],
+                    build: [],
+                },
+                files: CommandsTestFiles {
+                    source: [],
+                    recipe: [],
+                },
+            },
+        ),
+    ],
     about: About {
         homepage: Some(
             Url {

diff --git a/...ild__recipe__parser__tests__it_works.snap → ...__recipe__parser__tests__unix_recipe.snap b/...ild__recipe__parser__tests__it_works.snap → ...__recipe__parser__tests__unix_recipe.snap
@@ -1,6 +1,6 @@
 ---
 source: src/recipe/parser.rs
-expression: recipe.unwrap()
+expression: unix_recipe.unwrap()
 ---
 Recipe {
     package: Package {