Fix transformation of import/requires wrapped into Promise.resolve().

[SmartArray]

↪️ Pull Request

Previously we noticed that parcel incorrectly transforms Promise.resolve(import('foo')).then(...) as described in #8116.

The transformer that rewrites the nodes of the code Promise.resolve().then(() => return require('foo')) erroneously restructures the above code too.
However, there is no need to transform this require/import statement, when wrapped into Promise.resolve() because it is used to promisify the import in the first place.

This PR adds a unit test to illustrate the issue, and a condition that prevents the incorrect processing of the statement.

💻 Examples

// main.js
module.exports = Promise.resolve(require('./async')).then(x => x + 1335);

// async.js
module.exports = 2;

Expected behaviour

const n = await import('./main');
n === 1337

Current behaviour

The current code causes the runtime to crash because it injects a variable called res incorrectly, and thus destroys the code.

🚨 Test instructions

yarn workspace @parcel/integration-tests test -g Promise.resolve

Or use the minimal reproduction code here

✔️ PR Todo

• Added/updated unit tests for this change
• Filled out test instructions (In case there aren't any unit tests)
• Included links to related issues/PRs