root
/
webpack
mirror of https://github.com/webpack/webpack.git
1
0
Fork 0
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Merge pull request #4188 from webpack/bugfix/issue-4179 

fixes #4179
This commit is contained in:
Tobias Koppers 2017-02-08 09:39:31 +01:00 committed by GitHub
parent a76ea83e3a e8b3170e29
commit 0de9fec841
4 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/dependencies/HarmonyExportExpressionDependency.js
View File

@ -37,8 +37,8 @@ HarmonyExportExpressionDependency.Template = class HarmonyExportDependencyTempla
const content = this.getContent(dep.originModule, used);
if(dep.range) {
source.replace(dep.rangeStatement[0], dep.range[0] - 1, content);
source.replace(dep.range[1], dep.rangeStatement[1] - 1, ";");
source.replace(dep.rangeStatement[0], dep.range[0] - 1, content + "(");
source.replace(dep.range[1], dep.rangeStatement[1] - 1, ");");
return;
}

8
test/cases/parsing/issue-4179/index.js Normal file
View File

@ -0,0 +1,8 @@
import def from "./module?harmony";
import * as mod from "./module?harmony-start"
it("should export a sequence expression correctly", function() {
require("./module?cjs").should.be.eql({ default: 2 });
def.should.be.eql(2);
mod.default.should.be.eql(2);
});

1
test/cases/parsing/issue-4179/module.js Normal file
View File

@ -0,0 +1 @@
export default (1, 2);

2
test/configCases/code-generation/use-strict/index.js
View File

@ -19,7 +19,7 @@ it("should include only one use strict per module", function() {
"Object.defineProperty(__webpack_exports__, \"__esModule\", { value: true });",
"Object.defineProperty(__webpack_exports__, \"__esModule\", { value: true });",
"Object.defineProperty(__webpack_exports__, \"__esModule\", { value: true });",
"/* unused harmony default export */ var _unused_webpack_default_export = \"a\";",
"/* unused harmony default export */ var _unused_webpack_default_export = (\"a\");",
"Object.defineProperty(__webpack_exports__, \"__esModule\", { value: true });",
"it(\"should include only one use strict per module\", function() {"
]);