Skip to content

Commit f2e53ca

Browse files
committed
add tests
1 parent a0c8738 commit f2e53ca

File tree

3 files changed

+64
-4
lines changed

3 files changed

+64
-4
lines changed

packages/eslint-plugin/lib/rules/use-standard-html/check-content-model.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,6 @@ function checkContentModel(context, spec, node, children) {
6262
};
6363
let result = CONTINUE;
6464
while (result && state.contentModels && !!getContentModel(state)) {
65-
debugger;
6665
const contentModel = getContentModel(state);
6766
if (!contentModel) {
6867
return;
@@ -129,7 +128,7 @@ function required(model, context, state, node) {
129128
}
130129
context.report({
131130
node,
132-
messageId: MESSAGE_IDS.REQUIRED,
131+
messageId: MESSAGE_IDS.NOT_ALLOWED,
133132
});
134133
return EXIT;
135134
}

packages/eslint-plugin/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141
"@eslint/plugin-kit": "0.2.8",
4242
"@html-eslint/template-parser": "^0.40.0",
4343
"@html-eslint/template-syntax-parser": "^0.40.0",
44-
"html-standard": "^0.0.6"
44+
"html-standard": "file:../../../html-standard"
4545
},
4646
"devDependencies": {
4747
"@eslint/core": "0.13.0",

packages/eslint-plugin/tests/rules/use-standard-html.test.js

Lines changed: 62 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,9 @@ ruleTester.run("use-standard-html", rule, {
2929
{
3030
code: `<mark>text<br/></mark>`,
3131
},
32+
{
33+
code: `<fieldset></fieldset>`,
34+
},
3235
{
3336
code: `<fieldset><legend></legend></fieldset>`,
3437
},
@@ -38,14 +41,34 @@ ruleTester.run("use-standard-html", rule, {
3841
<legend></legend>
3942
</fieldset>`,
4043
},
44+
{
45+
code: '<template id="template"><p>Smile!</p></template>',
46+
},
47+
{
48+
code: "<custom-element>content</custom-element>",
49+
},
50+
{
51+
code: "<head><style> div {} </style> <script> console.log('hello'); </script></head>",
52+
},
4153
],
4254
invalid: [
4355
// required
4456
{
4557
code: `<html><div></div></html>`,
4658
errors: [
4759
{
48-
messageId: "required",
60+
messageId: "notAllowed",
61+
},
62+
],
63+
},
64+
{
65+
code: `<html>
66+
<!--comment-->
67+
<div></div>
68+
</html>`,
69+
errors: [
70+
{
71+
messageId: "notAllowed",
4972
},
5073
],
5174
},
@@ -57,6 +80,14 @@ ruleTester.run("use-standard-html", rule, {
5780
},
5881
],
5982
},
83+
{
84+
code: `<html><script></script></html>`,
85+
errors: [
86+
{
87+
messageId: "notAllowed",
88+
},
89+
],
90+
},
6091
// zeroOrMore
6192
{
6293
code: `<menu><div></div></menu>`,
@@ -66,6 +97,17 @@ ruleTester.run("use-standard-html", rule, {
6697
},
6798
],
6899
},
100+
{
101+
code: `<menu>
102+
<div></div>
103+
<!-- comment -->
104+
</menu>`,
105+
errors: [
106+
{
107+
messageId: "notAllowed",
108+
},
109+
],
110+
},
69111
// oneOreMore
70112
{
71113
code: `<mark><div></div></mark>`,
@@ -75,6 +117,25 @@ ruleTester.run("use-standard-html", rule, {
75117
},
76118
],
77119
},
120+
{
121+
code: `<mark>
122+
<div></div>
123+
</mark>`,
124+
errors: [
125+
{
126+
messageId: "notAllowed",
127+
},
128+
],
129+
},
130+
{
131+
code: `<head>
132+
</head>`,
133+
errors: [
134+
{
135+
messageId: "required",
136+
},
137+
],
138+
},
78139
{
79140
code: `<fieldset><base></base></fieldset>`,
80141
errors: [

0 commit comments

Comments
 (0)