Skip to content

Commit c60a1bd

Browse files
committed
Fixing a bug where addStyleEntry was not added to manifest.json
1 parent c7dd577 commit c60a1bd

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

lib/DeleteUnusedEntriesJSPlugin.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ DeleteUnusedEntriesJSPlugin.prototype.apply = function(compiler) {
3030
// remove the output file
3131
delete compilation.assets[filename];
3232
// remove the file, so that it does not dump in the manifest
33-
chunk.files.splice(chunk.files.indexOf(filename));
33+
chunk.files.splice(chunk.files.indexOf(filename), 1);
3434
}
3535
});
3636

test/functional.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -225,6 +225,10 @@ describe('Functional tests using webpack', function() {
225225
webpackAssert.assertManifestPathDoesNotExist(
226226
'styles.js'
227227
);
228+
webpackAssert.assertManifestPath(
229+
'styles.css',
230+
'/styles.css'
231+
);
228232

229233
done();
230234
});

0 commit comments

Comments
 (0)