Toggle check order to match spec.

kaj · Oct 12, 2024 · d856867 · d856867
1 parent 2268ee1
commit d856867
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/rsass/src/variablescope.rs b/rsass/src/variablescope.rs
@@ -316,14 +316,14 @@ impl Scope {
             let module = self
                 .get_module(&modulename)
                 .ok_or(ScopeError::NoModule(modulename))?;
+            // Refuse to declare new var from outside module:
+            let _check_existence = module.get(&name)?;
             if module
                 .get_local_or_none(&Name::from_static("@scope_name@"))
                 .is_some()
             {
                 return Err(ScopeError::ModifiedBuiltin);
             } else {
-                // Refuse to declare new var from outside module:
-                let _check_existence = module.get(&name)?;
                 return module.set_variable(name, val, default, global);
             }
         }

diff --git a/rsass/tests/spec/core_functions/general.rs b/rsass/tests/spec/core_functions/general.rs
@@ -45,7 +45,6 @@ mod error {
     }
 
     #[test]
-    #[ignore] // wrong error
     fn set_variable() {
         let runner = runner().with_cwd("set_variable");
         assert_eq!(