Merge pull request #20697 from emberjs/loader-bug

Bugfix: new loader feature was missing cache update
emberjs · May 14, 2024 · 775dcdc · 775dcdc
2 parents 2fd542f + b1ae249
commit 775dcdc
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/packages/loader/lib/index.js b/packages/loader/lib/index.js
@@ -74,7 +74,7 @@ var define, require;
 
     var result = callback.apply(this, reified);
     if (!deps.includes('exports') || result !== undefined) {
-      exports = result;
+      exports = seen[name] = result;
     }
 
     return exports;

diff --git a/tests/node/template-compiler-test.js b/tests/node/template-compiler-test.js
@@ -34,6 +34,14 @@ QUnit.module('ember-template-compiler.js', function () {
       assert.notEqual(typeof templateCompiler._Ember.ENV, null, '_Ember.ENV is not null');
     });
 
+    QUnit.test('_Ember.ENV (private API used by ember-cli-htmlbars) is stable', function (assert) {
+      assert.strictEqual(
+        templateCompiler._Ember.ENV,
+        templateCompiler._Ember.ENV,
+        '_Ember.ENV is stable'
+      );
+    });
+
     QUnit.test(
       'can access _Ember.FEATURES (private API used by ember-cli-htmlbars)',
       function (assert) {