Skip to content

Commit 533ba5b

Browse files
NullVoxPopuliNullVoxPopuli
committed
Extract rule: template-no-trailing-spaces
1 parent 8c8af7c commit 533ba5b

4 files changed

Lines changed: 345 additions & 0 deletions

File tree

‎README.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -256,6 +256,7 @@ rules in templates can be disabled with eslint directives with mustache or html
256256
| [template-no-obsolete-elements](docs/rules/template-no-obsolete-elements.md) | disallow obsolete HTML elements | | | |
257257
| [template-no-outlet-outside-routes](docs/rules/template-no-outlet-outside-routes.md) | disallow {{outlet}} outside of route templates | | | |
258258
| [template-no-page-title-component](docs/rules/template-no-page-title-component.md) | disallow usage of ember-page-title component | | | |
259+
| [template-no-trailing-spaces](docs/rules/template-no-trailing-spaces.md) | disallow trailing whitespace at the end of lines in templates | | 🔧 | |
259260
| [template-no-unavailable-this](docs/rules/template-no-unavailable-this.md) | disallow `this` in templates that are not inside a class or function | | | |
260261
| [template-no-unnecessary-component-helper](docs/rules/template-no-unnecessary-component-helper.md) | disallow unnecessary component helper | | 🔧 | |
261262
| [template-no-unnecessary-concat](docs/rules/template-no-unnecessary-concat.md) | disallow unnecessary string concatenation | | 🔧 | |

‎docs/rules/template-no-trailing-spaces.md‎

Lines changed: 45 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,45 @@
1+
# ember/template-no-trailing-spaces
2+
3+
🔧 This rule is automatically fixable by the [`--fix` CLI option](https://eslint.org/docs/latest/user-guide/command-line-interface#--fix).
4+
5+
<!-- end auto-generated rule header -->
6+
7+
Disallows trailing whitespace at the end of lines.
8+
9+
Trailing whitespace is unnecessary and can cause issues with version control systems, where it may be flagged as changes.
10+
11+
## Rule Details
12+
13+
This rule detects and removes trailing spaces or tabs at the end of lines.
14+
15+
## Examples
16+
17+
Examples of **incorrect** code for this rule:
18+
19+
```gjs
20+
<template>
21+
<div>Hello</div>
22+
</template>
23+
```
24+
25+
```gjs
26+
<template>
27+
<div>Hello</div>
28+
</template>
29+
```
30+
31+
Examples of **correct** code for this rule:
32+
33+
```gjs
34+
<template>
35+
<div>Hello</div>
36+
</template>
37+
```
38+
39+
## Related Rules
40+
41+
- [no-trailing-spaces](https://eslint.org/docs/rules/no-trailing-spaces) from eslint
42+
43+
## References
44+
45+
- [eslint-plugin-ember template-no-trailing-spaces](https://github.com/ember-cli/eslint-plugin-ember/blob/master/docs/rules/template-no-trailing-spaces.md)

‎lib/rules/template-no-trailing-spaces.js‎

Lines changed: 67 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,67 @@
1+
/** @type {import('eslint').Rule.RuleModule} */
2+
module.exports = {
3+
meta: {
4+
type: 'suggestion',
5+
docs: {
6+
description: 'disallow trailing whitespace at the end of lines in templates',
7+
category: 'Best Practices',
8+
url: 'https://github.com/ember-cli/eslint-plugin-ember/tree/master/docs/rules/template-no-trailing-spaces.md',
9+
templateMode: 'both',
10+
},
11+
fixable: 'whitespace',
12+
schema: [],
13+
messages: {
14+
unexpected: 'Trailing whitespace detected.',
15+
},
16+
originallyFrom: {
17+
name: 'ember-template-lint',
18+
rule: 'lib/rules/no-trailing-spaces.js',
19+
docs: 'docs/rule/no-trailing-spaces.md',
20+
tests: 'test/unit/rules/no-trailing-spaces-test.js',
21+
},
22+
},
23+
24+
create(context) {
25+
const sourceCode = context.sourceCode || context.getSourceCode();
26+
const glimmerTemplateRanges = [];
27+
28+
return {
29+
GlimmerTemplate(node) {
30+
glimmerTemplateRanges.push(node.range);
31+
},
32+
33+
'Program:exit'() {
34+
const text = sourceCode.getText();
35+
const lines = text.split('\n');
36+
let lineOffset = 0;
37+
38+
for (const [index, line] of lines.entries()) {
39+
if (line.endsWith(' ') || line.endsWith('\t')) {
40+
const trimmedLength = line.trimEnd().length;
41+
const trailingStart = lineOffset + trimmedLength;
42+
const lineEnd = lineOffset + line.length;
43+
44+
const isInTemplate = glimmerTemplateRanges.some(
45+
([start, end]) => trailingStart >= start && lineEnd <= end
46+
);
47+
48+
if (isInTemplate) {
49+
context.report({
50+
loc: {
51+
start: { line: index + 1, column: trimmedLength },
52+
end: { line: index + 1, column: line.length },
53+
},
54+
messageId: 'unexpected',
55+
fix(fixer) {
56+
return fixer.removeRange([trailingStart, lineEnd]);
57+
},
58+
});
59+
}
60+
}
61+
62+
lineOffset += line.length + 1; // +1 for the \n
63+
}
64+
},
65+
};
66+
},
67+
};

‎tests/lib/rules/template-no-trailing-spaces.js‎

Lines changed: 232 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,232 @@
1+
//------------------------------------------------------------------------------
2+
// Requirements
3+
//------------------------------------------------------------------------------
4+
5+
const rule = require('../../../lib/rules/template-no-trailing-spaces');
6+
const RuleTester = require('eslint').RuleTester;
7+
8+
//------------------------------------------------------------------------------
9+
// Tests
10+
//------------------------------------------------------------------------------
11+
12+
const ruleTester = new RuleTester({
13+
parser: require.resolve('ember-eslint-parser'),
14+
parserOptions: { ecmaVersion: 2022, sourceType: 'module' },
15+
});
16+
17+
ruleTester.run('template-no-trailing-spaces', rule, {
18+
valid: [
19+
`<template>
20+
<div>Hello World</div>
21+
</template>`,
22+
`<template>
23+
<div>
24+
Content
25+
</div>
26+
</template>`,
27+
28+
'<template>test</template>',
29+
'<template> test</template>',
30+
`<template>test
31+
</template>`,
32+
`<template>{{#my-component}}
33+
{{/my-component}}</template>`,
34+
`<template> test
35+
</template>`,
36+
37+
// JS code with trailing spaces outside <template> must NOT be flagged
38+
'const foo = "bar"; \n\n<template><div>Hello</div></template>',
39+
],
40+
41+
invalid: [
42+
{
43+
code: `<template>
44+
<div>Hello</div>
45+
</template>`,
46+
output: `<template>
47+
<div>Hello</div>
48+
</template>`,
49+
errors: [
50+
{
51+
message: 'Trailing whitespace detected.',
52+
},
53+
],
54+
},
55+
{
56+
code: `<template>
57+
<div>Hello</div>
58+
</template>`,
59+
output: `<template>
60+
<div>Hello</div>
61+
</template>`,
62+
errors: [
63+
{
64+
message: 'Trailing whitespace detected.',
65+
},
66+
],
67+
},
68+
69+
{
70+
code: `<template>test
71+
</template>`,
72+
output: `<template>test
73+
</template>`,
74+
errors: [{ message: 'Trailing whitespace detected.' }],
75+
},
76+
{
77+
code: `<template>import { hbs } from 'ember-cli-htmlbars';
78+
79+
test('it renders', async (assert) => {
80+
await render(hbs\`
81+
<div class="parent">
82+
<div class="child"></div>
83+
</div>
84+
\`);
85+
});</template>`,
86+
output: `<template>import { hbs } from 'ember-cli-htmlbars';
87+
88+
test('it renders', async (assert) => {
89+
await render(hbs\`
90+
<div class="parent">
91+
<div class="child"></div>
92+
</div>
93+
\`);
94+
});</template>`,
95+
errors: [{ message: 'Trailing whitespace detected.' }],
96+
},
97+
{
98+
code: `<template>import { hbs } from 'ember-cli-htmlbars';
99+
100+
test('it renders', async (assert) => {
101+
await render(hbs\`
102+
<div></div>
103+
104+
<div></div>
105+
\`);
106+
});</template>`,
107+
output: `<template>import { hbs } from 'ember-cli-htmlbars';
108+
109+
test('it renders', async (assert) => {
110+
await render(hbs\`
111+
<div></div>
112+
113+
<div></div>
114+
\`);
115+
});</template>`,
116+
errors: [{ message: 'Trailing whitespace detected.' }],
117+
},
118+
{
119+
code: '<template>test\n \n</template>',
120+
output: '<template>test\n\n</template>',
121+
errors: [{ message: 'Trailing whitespace detected.' }],
122+
},
123+
{
124+
code: `<template>{{#my-component}}
125+
test
126+
{{/my-component}}</template>`,
127+
output: `<template>{{#my-component}}
128+
test
129+
{{/my-component}}</template>`,
130+
errors: [{ message: 'Trailing whitespace detected.' }],
131+
},
132+
],
133+
});
134+
135+
const hbsRuleTester = new RuleTester({
136+
parser: require.resolve('ember-eslint-parser/hbs'),
137+
parserOptions: {
138+
ecmaVersion: 2022,
139+
sourceType: 'module',
140+
},
141+
});
142+
143+
hbsRuleTester.run('template-no-trailing-spaces', rule, {
144+
valid: [
145+
'test',
146+
' test',
147+
`test
148+
`,
149+
`
150+
`,
151+
` test
152+
`,
153+
`{{#my-component}}
154+
test
155+
{{/my-component}}`,
156+
`import { hbs } from 'ember-cli-htmlbars';
157+
158+
test('it renders', async (assert) => {
159+
await render(hbs\`
160+
<div class="parent">
161+
<div class="child"></div>
162+
</div>
163+
\`);
164+
});`,
165+
],
166+
invalid: [
167+
{
168+
code: 'test ',
169+
output: 'test',
170+
errors: [{ message: 'Trailing whitespace detected.' }],
171+
},
172+
{
173+
code: `test
174+
`,
175+
output: `test
176+
`,
177+
errors: [{ message: 'Trailing whitespace detected.' }],
178+
},
179+
{
180+
code: `import { hbs } from 'ember-cli-htmlbars';
181+
182+
test('it renders', async (assert) => {
183+
await render(hbs\`
184+
<div class="parent">
185+
<div class="child"></div>
186+
</div>
187+
\`);
188+
});`,
189+
output: `import { hbs } from 'ember-cli-htmlbars';
190+
191+
test('it renders', async (assert) => {
192+
await render(hbs\`
193+
<div class="parent">
194+
<div class="child"></div>
195+
</div>
196+
\`);
197+
});`,
198+
errors: [{ message: 'Trailing whitespace detected.' }],
199+
},
200+
{
201+
code: `import { hbs } from 'ember-cli-htmlbars';
202+
203+
test('it renders', async (assert) => {
204+
await render(hbs\`
205+
<div></div>
206+
207+
<div></div>
208+
\`);
209+
});`,
210+
output: `import { hbs } from 'ember-cli-htmlbars';
211+
212+
test('it renders', async (assert) => {
213+
await render(hbs\`
214+
<div></div>
215+
216+
<div></div>
217+
\`);
218+
});`,
219+
errors: [{ message: 'Trailing whitespace detected.' }],
220+
},
221+
{
222+
code: 'test\n \n',
223+
output: 'test\n\n',
224+
errors: [{ message: 'Trailing whitespace detected.' }],
225+
},
226+
{
227+
code: '{{#my-component}}\n test \n{{/my-component}}',
228+
output: '{{#my-component}}\n test\n{{/my-component}}',
229+
errors: [{ message: 'Trailing whitespace detected.' }],
230+
},
231+
],
232+
});

0 commit comments

Comments
 (0)