Merge branch 'main' into main

src/lune/globals/require/alias.rs

@@ -19,7 +19,6 @@ where
     'lua: 'ctx,
 {
     let alias = alias.to_ascii_lowercase();
-    let path = path.to_ascii_lowercase();
 
     let parent = make_absolute_and_clean(source)
         .parent()

src/lune/globals/require/context.rs

@@ -70,7 +70,7 @@ impl RequireContext {
             CWD.join(&rel_path)
         };
 
-        Ok((rel_path, abs_path))
+        Ok((abs_path, rel_path))
     }
 
     /**

src/tests.rs

@@ -95,6 +95,7 @@ create_tests! {
     require_nested: "require/tests/nested",
     require_parents: "require/tests/parents",
     require_siblings: "require/tests/siblings",
+    require_state: "require/tests/state",
 
     global_g_table: "globals/_G",
     global_version: "globals/_VERSION",

tests/require/tests/state.luau

@@ -0,0 +1,14 @@
+-- the idea of this test is that state_module stores some state in one of its local
+-- variable
+local state_module = require("./state_module")
+
+-- we confirm that without anything happening, the initial value is what we expect
+assert(state_module.state == 10)
+
+-- this second file also requires state_module and calls a function that changes the local
+-- state to 11
+require("./state_second")
+
+-- with correct module caching, we should see the change done in state_secone reflected
+-- here
+assert(state_module.state == 11)

tests/require/tests/state_module.luau

@@ -0,0 +1,9 @@
+local M = {}
+
+M.state = 10
+
+function M.set_state(n: number)
+    M.state = n
+end
+
+return M

tests/require/tests/state_second.luau

@@ -0,0 +1,5 @@
+local state_module = require("./state_module")
+
+state_module.set_state(11)
+
+return {}