Skip to content

Commit b2e5aab

Browse files
michael-ciniawskyjoshwiens
michael-ciniawsky
authored and
joshwiens
committed
squash: more tests, small refactor
1 parent ace9914 commit b2e5aab

File tree

14 files changed

+188
-48
lines changed

14 files changed

+188
-48
lines changed

‎src/index.js‎

Lines changed: 44 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
/* eslint-disable
22
import/order,
33
import/first,
4+
arrow-parens,
45
no-undefined,
56
no-param-reassign,
67
no-useless-escape,
@@ -15,18 +16,21 @@ import minifier from 'html-minifier';
1516

1617
const schema = require('./options');
1718

18-
function randomIdent() {
19-
return `xxxHTMLLINKxxx${Math.random()}${Math.random()}xxx`;
19+
function randomize() {
20+
return `link__${Math.random()}`;
2021
}
2122

2223
export default function loader(html) {
2324
const options = loaderUtils.getOptions(this) || {};
2425

2526
validateOptions(schema, options, 'HTML Loader');
2627

28+
// eslint-disable-next-line
29+
const root = options.root;
30+
2731
let attributes = ['img:src'];
2832

29-
if (options.attrs === undefined) {
33+
if (options.attrs !== undefined) {
3034
if (typeof options.attrs === 'string') attributes = options.attrs.split(' ');
3135
else if (Array.isArray(options.attrs)) attributes = options.attrs;
3236
else if (options.attrs === false) attributes = [];
@@ -38,11 +42,12 @@ export default function loader(html) {
3842
}
3943
}
4044

41-
const { root } = options;
42-
43-
// eslint-disable-next-line
4445
const links = attrs(html, (tag, attr) => {
45-
return attributes.indexOf(`${tag}:${attr}`) >= 0;
46+
const item = `${tag}:${attr}`;
47+
48+
const result = attributes.find((a) => item.indexOf(a) >= 0);
49+
50+
return !!result;
4651
});
4752

4853
links.reverse();
@@ -62,14 +67,15 @@ export default function loader(html) {
6267
link.value = uri.format();
6368
link.length = link.value.length;
6469
}
65-
// eslint-disable-next-line
66-
var ident;
67-
do { ident = randomIdent(); } while (data[ident]);
70+
71+
let ident;
72+
do { ident = randomize(); } while (data[ident]);
6873
data[ident] = link.value;
6974

7075
const item = html.pop();
7176

7277
html.push(item.substr(link.start + link.length));
78+
// eslint-disable-next-line
7379
html.push(ident);
7480
html.push(item.substr(0, link.start));
7581
});
@@ -79,7 +85,7 @@ export default function loader(html) {
7985
if (options.interpolate === 'require') {
8086
const regex = /\$\{require\([^)]*\)\}/g;
8187
// eslint-disable-next-line
82-
var result;
88+
let result;
8389

8490
const requires = [];
8591

@@ -98,42 +104,41 @@ export default function loader(html) {
98104

99105
requires.forEach((link) => {
100106
const item = html.pop();
101-
// eslint-disable-next-line
102-
var ident
103-
do { ident = randomIdent(); } while (data[ident]);
107+
108+
let ident;
109+
do { ident = randomize(); } while (data[ident]);
104110
data[ident] = link.value.substring(11, link.length - 3);
105111

106112
html.push(item.substr(link.start + link.length));
113+
// eslint-disable-next-line
107114
html.push(ident);
108115
html.push(item.substr(0, link.start));
109116
});
110117

111118
html = html.reverse().join('');
112119
}
113120

114-
if (typeof options.minimize === 'boolean' ? options.minimize : this.minimize) {
115-
const minimizeOptions = Object.assign({}, options);
116-
117-
[
118-
'removeComments',
119-
'removeCommentsFromCDATA',
120-
'removeCDATASectionsFromCDATA',
121-
'collapseWhitespace',
122-
'conservativeCollapse',
123-
'removeAttributeQuotes',
124-
'useShortDoctype',
125-
'keepClosingSlash',
126-
'minifyJS',
127-
'minifyCSS',
128-
'removeScriptTypeAttributes',
129-
'removeStyleTypeAttributes',
130-
].forEach((name) => {
131-
if (typeof minimizeOptions[name] === 'undefined') {
132-
minimizeOptions[name] = true;
133-
}
121+
if (options.minimize || this.minimize) {
122+
let minimize = Object.create({
123+
collapseWhitespace: true,
124+
conservativeCollapse: true,
125+
useShortDoctype: true,
126+
keepClosingSlash: true,
127+
minifyJS: true,
128+
minifyCSS: true,
129+
removeComments: true,
130+
removeAttributeQuotes: true,
131+
removeStyleTypeAttributes: true,
132+
removeScriptTypeAttributes: true,
133+
removeCommentsFromCDATA: true,
134+
removeCDATASectionsFromCDATA: true,
134135
});
135136

136-
html = minifier.minify(html, minimizeOptions);
137+
if (typeof options.minimize === 'object') {
138+
minimize = Object.assign(minimize, options.minimize);
139+
}
140+
141+
html = minifier.minify(html, minimize);
137142
}
138143

139144
// TODO
@@ -148,8 +153,10 @@ export default function loader(html) {
148153
html = JSON.stringify(html);
149154
}
150155

151-
return `export default ${html.replace(/xxxHTMLLINKxxx[0-9\.]+xxx/g, (match) => {
156+
html = html.replace(/link__[0-9\.]+/g, (match) => {
152157
if (!data[match]) return match;
153158
return `"require('${JSON.stringify(loaderUtils.urlToRequest(data[match], root))}')"`;
154-
})};`;
159+
});
160+
161+
return `export default ${html};`;
155162
}

‎src/options.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
"type": ["boolean", "object"]
1212
},
1313
"interpolate": {
14-
"type": "string"
14+
"type": ["boolean", "string"]
1515
}
1616
},
1717
"additionalProperties": false

‎test/Errors.test.js‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,9 @@ describe('Errors', () => {
2222

2323
return webpack('index.js', config)
2424
.then((result) => stats(result))
25-
.then(({ loader }) => {
26-
expect(() => eval(loader.err)).toThrowErrorMatchingSnapshot();
25+
.then(({ loaders }) => {
26+
expect(() => eval(loaders.err)).toThrow();
27+
expect(() => eval(loaders.err)).toThrowErrorMatchingSnapshot();
2728
})
2829
.catch((err) => err);
2930
});

‎test/__snapshots__/loader.test.js.snap‎

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,19 @@
11
// Jest Snapshot v1, https://goo.gl/fbAQLP
22

3-
exports[`HTML Loader Should process HTML with defaults 1`] = `"throw new Error(\\"Module build failed: \\\\n\\\\nHTML Loader\\\\n\\\\n[option.attrs] Invalid attribute value found\\\\n\\");"`;
3+
exports[`HTML Loader Should export as ES2015 Module 1`] = `"export default \\"<p>Hello world!</p>\\";"`;
4+
5+
exports[`HTML Loader Should not remove attributes by default 1`] = `"export default \\"<input type=text />\\";"`;
6+
7+
exports[`HTML Loader Should process HTML with defaults 1`] = `"export default \\"<!DOCTYPE html>\\\\n<html lang=\\\\\\"en\\\\\\">\\\\n<head>\\\\n <meta charset=\\\\\\"utf-8\\\\\\">\\\\n <title>HTML Loader</title>\\\\n</head>\\\\n<body>\\\\n <!-- I'm a comment -->\\\\n <div id=\\\\\\"app\\\\\\"></div>\\\\n</body>\\\\n</html>\\\\n\\";"`;
8+
9+
exports[`HTML Loader Should process HTML with options.attrs 1`] = `"throw new Error(\\"Module parse failed: /Users/Cini/Github/Webpack/webpack-loaders/html-loader/src/cjs.js??ref--0-0!/Users/Cini/Github/Webpack/webpack-loaders/html-loader/test/fixtures/attrs.html Unexpected token (1:33)\\\\nYou may need an appropriate loader to handle this file type.\\\\n| export default \\\\\\"<div data-attrs=\\\\\\"require('\\\\\\"./\\\\\\\\\\\\\\"foo\\\\\\\\\\\\\\"\\\\\\"')\\\\\\"></div>\\\\\\\\n\\\\\\\\n<custom-element custom-attrs=\\\\\\"require('\\\\\\"./\\\\\\\\\\\\\\"foo\\\\\\\\\\\\\\"\\\\\\"')\\\\\\"></custom-element>\\\\\\\\n<custom-container custom-attrs=\\\\\\"require('\\\\\\"./\\\\\\\\\\\\\\"foo\\\\\\\\\\\\\\"\\\\\\"')\\\\\\"></custom-container>\\\\\\\\n\\\\\\";\\");"`;
10+
11+
exports[`HTML Loader Should process HTML with options.interpolate {Boolean} 1`] = `"throw new Error(\\"Module parse failed: /Users/Cini/Github/Webpack/webpack-loaders/html-loader/src/cjs.js??ref--0-0!/Users/Cini/Github/Webpack/webpack-loaders/html-loader/test/fixtures/interpolate.html Unexpected token (1:15)\\\\nYou may need an appropriate loader to handle this file type.\\\\n| export default <img src=\\\\\\"require('\\\\\\"./\\\\\\\\\\\\\\"\${img.src}\\\\\\\\\\\\\\"\\\\\\"')\\\\\\" alt=\\\\\\"\${img.alt}\\\\\\">\\\\n| <img src=\\\\\\"require('\\\\\\"./\\\\\\\\\\\\\\"\${require(./image.png)}\\\\\\\\\\\\\\"\\\\\\"')\\\\\\">\\\\n| ;\\");"`;
12+
13+
exports[`HTML Loader Should process HTML with options.interpolate {String} 1`] = `"throw new Error(\\"Module parse failed: /Users/Cini/Github/Webpack/webpack-loaders/html-loader/src/cjs.js??ref--0-0!/Users/Cini/Github/Webpack/webpack-loaders/html-loader/test/fixtures/interpolate.html Unexpected token (1:26)\\\\nYou may need an appropriate loader to handle this file type.\\\\n| export default \\\\\\"<img src=\\\\\\"require('\\\\\\"./\\\\\\\\\\\\\\"\${img.src}\\\\\\\\\\\\\\"\\\\\\"')\\\\\\" alt=\\\\\\\\\\\\\\"\${img.alt}\\\\\\\\\\\\\\">\\\\\\\\n<img src=\\\\\\"require('\\\\\\"./\\\\\\\\\\\\\\"\${require(./image.png)}\\\\\\\\\\\\\\"\\\\\\"')\\\\\\">\\\\\\\\n\\\\\\";\\");"`;
14+
15+
exports[`HTML Loader Should process HTML with options.minimize {Boolean} 1`] = `"export default \\"<!DOCTYPE html> <html lang=en> <head> <meta charset=utf-8> <title>HTML Loader</title> </head> <body> <div id=app></div> </body> </html> \\";"`;
16+
17+
exports[`HTML Loader Should process HTML with options.minimize {Object} 1`] = `"export default \\"<!DOCTYPE html> <html lang=en> <head> <meta charset=utf-8> <title>HTML Loader</title> </head> <body> <!-- I'm a comment --> <div id=app></div> </body> </html> \\";"`;
18+
19+
exports[`HTML Loader Should process HTML with options.root 1`] = `"throw new Error(\\"Module parse failed: /Users/Cini/Github/Webpack/webpack-loaders/html-loader/src/cjs.js??ref--0-0!/Users/Cini/Github/Webpack/webpack-loaders/html-loader/test/fixtures/root.html Unexpected token (1:26)\\\\nYou may need an appropriate loader to handle this file type.\\\\n| export default \\\\\\"<img src=\\\\\\"require('\\\\\\"./\\\\\\\\\\\\\\"./image.png\\\\\\\\\\\\\\"\\\\\\"')\\\\\\">\\\\\\\\n\\\\\\";\\");"`;

‎test/fixtures/attrs.html‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
<div data-attrs="foo"></div>
2+
3+
<custom-element custom-attrs="foo"></custom-element>
4+
<custom-container custom-attrs="foo"></custom-container>

‎test/fixtures/attrs.js‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
import html from './attrs.html';
2+
3+
console.log(html);

‎test/fixtures/index.html‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
<title>HTML Loader</title>
66
</head>
77
<body>
8+
<!-- I'm a comment -->
89
<div id="app"></div>
910
</body>
1011
</html>

‎test/fixtures/interpolate.html‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
<img src="${img.src}" alt="${img.alt}">
2+
<img src="${require(./image.png)}">

‎test/fixtures/interpolate.js‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
import html from './interpolate.html';
2+
3+
console.log(html);

‎test/fixtures/root.html‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
<img src="./image.png">

0 commit comments

Comments
 (0)