diff --git a/packages/node-resolve/src/index.js b/packages/node-resolve/src/index.js index f9895be3c..8daafcffe 100644 --- a/packages/node-resolve/src/index.js +++ b/packages/node-resolve/src/index.js @@ -281,11 +281,6 @@ export function nodeResolve(opts = {}) { if (resolvedResolved.external) { return false; } - // Allow other plugins to take over resolution. Rollup core will not - // change the id if it corresponds to an existing file - if (resolvedResolved.id !== resolved.id) { - return resolvedResolved; - } // Pass on meta information added by other plugins return { ...resolved, meta: resolvedResolved.meta }; } diff --git a/packages/node-resolve/test/test.js b/packages/node-resolve/test/test.js index 9c08ce211..50cbcac82 100755 --- a/packages/node-resolve/test/test.js +++ b/packages/node-resolve/test/test.js @@ -1,4 +1,4 @@ -import { join, resolve, dirname } from 'path'; +import { join, resolve } from 'path'; import test from 'ava'; import { rollup } from 'rollup'; @@ -581,31 +581,3 @@ test('passes on meta information from other plugins', async (t) => { ] }); }); - -test('allow other plugins to take over resolution', async (t) => { - await rollup({ - input: 'entry/main.js', - onwarn: failOnWarn(t), - plugins: [ - nodeResolve(), - { - name: 'change-resolution', - resolveId(importee) { - if (importee.endsWith('main.js')) { - return { - id: join(dirname(importee), 'other.js'), - meta: { 'change-resolution': 'changed' } - }; - } - return null; - }, - - load(id) { - const info = this.getModuleInfo(id); - t.is(info.id, join(__dirname, 'fixtures', 'entry', 'other.js')); - t.deepEqual(info.meta, { 'change-resolution': 'changed' }); - } - } - ] - }); -});