Change __webpack_require__.e to be assigned as a function property (e = fn() { ... }) instead of using Object.defineProperty. This allows it to be set/changed temporarily by tests, such as the test for issue-758.

Author: jharris4

lib/HotModuleReplacement.runtime.js

@@ -43,34 +43,31 @@ module.exports = function() {
 			};
 		};
 		for(var name in $require$) {
-			if(Object.prototype.hasOwnProperty.call($require$, name)) {
+			if(Object.prototype.hasOwnProperty.call($require$, name) && name !== "e") {
 				Object.defineProperty(fn, name, ObjectFactory(name));
 			}
 		}
-		Object.defineProperty(fn, "e", {
-			enumerable: true,
-			value: function(chunkId) {
-				if(hotStatus === "ready")
-					hotSetStatus("prepare");
-				hotChunksLoading++;
-				return $require$.e(chunkId).then(finishChunkLoading, function(err) {
-					finishChunkLoading();
-					throw err;
-				});
+		fn.e = function(chunkId) {
+			if(hotStatus === "ready")
+				hotSetStatus("prepare");
+			hotChunksLoading++;
+			return $require$.e(chunkId).then(finishChunkLoading, function(err) {
+				finishChunkLoading();
+				throw err;
+			});
 
-				function finishChunkLoading() {
-					hotChunksLoading--;
-					if(hotStatus === "prepare") {
-						if(!hotWaitingFilesMap[chunkId]) {
-							hotEnsureUpdateChunk(chunkId);
-						}
-						if(hotChunksLoading === 0 && hotWaitingFiles === 0) {
-							hotUpdateDownloaded();
-						}
+			function finishChunkLoading() {
+				hotChunksLoading--;
+				if(hotStatus === "prepare") {
+					if(!hotWaitingFilesMap[chunkId]) {
+						hotEnsureUpdateChunk(chunkId);
+					}
+					if(hotChunksLoading === 0 && hotWaitingFiles === 0) {
+						hotUpdateDownloaded();
 					}
 				}
 			}
-		});
+		};
 		return fn;
 	}