Merge pull request #1245 from stevcooo/do-not-use-absolute-path-in-ve…

…ndor-files Do not use absolute path in vendor files
embroider-build · Aug 31, 2022 · e017581 · e017581
2 parents 788931f + 8150fff
commit e017581
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/packages/macros/src/babel/macros-babel-plugin.ts b/packages/macros/src/babel/macros-babel-plugin.ts
@@ -47,7 +47,7 @@ export default function main(context: typeof Babel): unknown {
     FunctionDeclaration: {
       enter(path: NodePath<t.FunctionDeclaration>, state: State) {
         let id = path.get('id');
-        if (id.isIdentifier() && id.node.name === 'initializeRuntimeMacrosConfig') {
+        if (id.isIdentifier() && id.node.name === 'initializeRuntimeMacrosConfig' && state.opts.mode === 'run-time') {
           let pkg = state.owningPackage();
           if (pkg && pkg.name === '@embroider/macros') {
             inlineRuntimeConfig(path, state, context);

diff --git a/packages/macros/tests/babel/get-config.test.ts b/packages/macros/tests/babel/get-config.test.ts
@@ -157,7 +157,7 @@ describe(`getConfig`, function () {
         });
       }
 
-      test('inlines runtime config into own source', () => {
+      runTimeTest('inlines runtime config into own source', () => {
         filename = __filename;
         let code = transform(`
           function initializeRuntimeMacrosConfig() {
@@ -171,7 +171,7 @@ describe(`getConfig`, function () {
         expect(run(code, { filename }).packages[coreRoot].beverage).toEqual('coffee');
       });
 
-      test('does not inline runtime config into other packages', () => {
+      runTimeTest('does not inline runtime config into other packages', () => {
         let code = transform(`
           function initializeRuntimeMacrosConfig() {
           }