Browse Source

Merge pull request #609 from rollup/report-transform-error-file

report files causing transform errors
gh-669
Rich Harris 9 years ago
parent
commit
b998c91560
  1. 7
      src/utils/transform.js
  2. 17
      test/function/report-transform-error-file/_config.js
  3. 3
      test/function/report-transform-error-file/foo.js
  4. 3
      test/function/report-transform-error-file/main.js

7
src/utils/transform.js

@ -39,5 +39,10 @@ export default function transform ( source, id, transformers ) {
}, Promise.resolve( source.code ) ) }, Promise.resolve( source.code ) )
.then( code => ({ code, originalCode, ast, sourceMapChain }) ); .then( code => ({ code, originalCode, ast, sourceMapChain }) )
.catch( err => {
err.id = id;
err.message = `Error loading ${id}: ${err.message}`;
throw err;
});
} }

17
test/function/report-transform-error-file/_config.js

@ -0,0 +1,17 @@
var assert = require( 'assert' );
module.exports = {
description: 'reports which file caused a transform error',
options: {
plugins: [{
transform: function ( code, id ) {
if ( /foo/.test( id ) ) {
throw new Error( 'nope' );
}
}
}]
},
error: function ( err ) {
assert.ok( ~err.message.indexOf( 'foo.js' ) );
}
};

3
test/function/report-transform-error-file/foo.js

@ -0,0 +1,3 @@
export default function () {
console.log( 'foo' );
}

3
test/function/report-transform-error-file/main.js

@ -0,0 +1,3 @@
import foo from './foo.js';
foo();
Loading…
Cancel
Save