NotificationsYou must be signed in to change notification settings
Fork32.2k
Star112k

Commit5d03f6f

authored and

committed

esm: move hooks test with others

Co-authored-by: Gabriel Bota <gabriel.bota@dynatrace.com>PR-URL:#53558Reviewed-By: Matteo Collina <matteo.collina@gmail.com>Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>

1 parentceb1d5e commit5d03f6fCopy full SHA for 5d03f6f

File tree

3 files changed

+21

-11

lines changed

test
- es-module
  - test-esm-loader-hooks.mjs
  - test-esm-named-exports.js
- fixtures/es-modules
  - require-esm-throws-with-loaders.js

3 files changed

+21

-11

lines changed

`‎test/es-module/test-esm-loader-hooks.mjs`

Lines changed: 13 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -737,6 +737,19 @@ describe('Loader hooks', { concurrency: true }, () => {`
`737`	`737`	`assert.strictEqual(signal,null);`
`738`	`738`	`});`
`739`	`739`
	`740`	`+describe('should use hooks',async()=>{`
	`741`	`+const{ code, signal, stdout, stderr}=awaitspawnPromisified(process.execPath,[`
	`742`	`+'--import',`
	`743`	`+fixtures.fileURL('es-module-loaders/builtin-named-exports.mjs'),`
	`744`	`+fixtures.path('es-modules/require-esm-throws-with-loaders.js'),`
	`745`	`+]);`
	`746`	`+`
	`747`	`+assert.strictEqual(stderr,'');`
	`748`	`+assert.strictEqual(stdout,'');`
	`749`	`+assert.strictEqual(code,0);`
	`750`	`+assert.strictEqual(signal,null);`
	`751`	`+});`
	`752`	`+`
`740`	`753`	`it('should support source maps in commonjs translator',async()=>{`
`741`	`754`	`constreadFile=async()=>{};`
`742`	`755`	consthook=`

`‎test/es-module/test-esm-named-exports.js`

Lines changed: 0 additions & 11 deletions

This file was deleted.

`‎test/fixtures/es-modules/require-esm-throws-with-loaders.js`

Lines changed: 8 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,8 @@`
	`1`	`+'use strict';`
	`2`	`+const{ readFile, __fromLoader}=require('fs');`
	`3`	`+constassert=require('assert');`
	`4`	`+`
	`5`	`+assert.throws(()=>require('./test-esm-ok.mjs'),{code:'ERR_REQUIRE_ESM'});`
	`6`	`+`
	`7`	`+assert(readFile);`
	`8`	`+assert(__fromLoader);`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Commit5d03f6f

File tree

3 files changed

3 files changed

`‎test/es-module/test-esm-loader-hooks.mjs`

`‎test/es-module/test-esm-named-exports.js`

`‎test/fixtures/es-modules/require-esm-throws-with-loaders.js`

0 commit comments