eslint-plugin-crisp 1.5.13 → 1.6.0

package/tests/rules/one-space-after-operator.test.js

@@ -0,0 +1,69 @@
+import rule from "../../rules/one-space-after-operator.js";
+import createRuleTester from "../rule-tester.js";
+
+const ruleTester = createRuleTester();
+
+ruleTester.run("one-space-after-operator", rule, {
+  valid: [
+    "let value = 1; value = 2;",
+    "let value = 1; value += 2;",
+    "const object = { key : value };",
+    "const object = { method() {} };",
+    {
+      code: "const object = { key:  value };",
+      options: [
+        {
+          checkColon: false
+        }
+      ]
+    }
+  ],
+
+  invalid: [
+    {
+      code: "const value=  1;",
+      output: "const value = 1;",
+      errors: [
+        {
+          message: "There should be at least one space before and exactly one space after '='"
+        }
+      ]
+    },
+    {
+      code: "let value = 1; value=  2;",
+      output: "let value = 1; value = 2;",
+      errors: [
+        {
+          message: "There should be at least one space before and exactly one space after '='"
+        }
+      ]
+    },
+    {
+      code: "let value = 1; value+=2;",
+      output: "let value = 1; value += 2;",
+      errors: [
+        {
+          message: "There should be at least one space before and exactly one space after '+='"
+        }
+      ]
+    },
+    {
+      code: "const value =1;",
+      output: "const value = 1;",
+      errors: [
+        {
+          message: "There should be at least one space before and exactly one space after '='"
+        }
+      ]
+    },
+    {
+      code: "const object = { key:  value };",
+      output: "const object = { key : value };",
+      errors: [
+        {
+          message: "There should be at least one space before and exactly one space after ':'"
+        }
+      ]
+    }
+  ]
+});

package/tests/rules/regex-in-constructor.test.js

@@ -0,0 +1,40 @@
+import rule from "../../rules/regex-in-constructor.js";
+import createRuleTester from "../rule-tester.js";
+
+const ruleTester = createRuleTester();
+
+ruleTester.run("regex-in-constructor", rule, {
+  valid: [
+    "class Matcher { constructor() { this.pattern = /value/i; } }",
+    "function matches(value) { return /value/.test(value); }",
+    "class Matcher { matches(value) { return new RegExp(value).test(value); } }",
+    "class Matcher { pattern = /value/; }"
+  ],
+
+  invalid: [
+    {
+      code: "class Matcher { matches(value) { return /value/i.test(value); } }",
+      errors: [
+        {
+          message: "Regular expressions should be defined in the constructor."
+        }
+      ]
+    },
+    {
+      code: "class Matcher { static matches(value) { return /value/.test(value); } }",
+      errors: [
+        {
+          message: "Regular expressions should be defined in the constructor."
+        }
+      ]
+    },
+    {
+      code: "class Matcher { matches(value) { return /first/.test(value) || /second/.test(value); } }",
+      errors: [
+        {
+          message: "Regular expressions should be defined in the constructor."
+        }
+      ]
+    }
+  ]
+});

package/tests/rules/ternary-parenthesis.test.js

@@ -0,0 +1,34 @@
+import rule from "../../rules/ternary-parenthesis.js";
+import createRuleTester from "../rule-tester.js";
+
+const ruleTester = createRuleTester();
+
+ruleTester.run("ternary-parenthesis", rule, {
+  valid: [
+    "const result = ready ? first : second;",
+    "const result = !ready ? first : second;",
+    "const result = (count > 0) ? first : second;",
+    "const result = (ready && enabled) ? first : second;"
+  ],
+
+  invalid: [
+    {
+      code: "const result = count > 0 ? first : second;",
+      output: "const result = (count > 0) ? first : second;",
+      errors: [
+        {
+          message: "The condition in ternary expressions with an operator should be wrapped in parentheses"
+        }
+      ]
+    },
+    {
+      code: "const result = ready && enabled ? first : second;",
+      output: "const result = (ready && enabled) ? first : second;",
+      errors: [
+        {
+          message: "The condition in ternary expressions with an operator should be wrapped in parentheses"
+        }
+      ]
+    }
+  ]
+});

package/tests/rules/two-lines-between-class-members.test.js

@@ -0,0 +1,62 @@
+import rule from "../../rules/two-lines-between-class-members.js";
+import createRuleTester from "../rule-tester.js";
+
+const ruleTester = createRuleTester();
+
+ruleTester.run("two-lines-between-class-members", rule, {
+  valid: [
+    "class Worker { run() {} }",
+    `class Worker {
+  start() {}
+
+
+  stop() {}
+}`,
+    `class Worker {
+  active = true;
+
+
+  run() {}
+}`
+  ],
+
+  invalid: [
+    {
+      code: `class Worker {
+  start() {}
+  stop() {}
+}`,
+      errors: [
+        {
+          message: "Expected exactly two line breaks between class methods."
+        }
+      ]
+    },
+    {
+      code: `class Worker {
+  start() {}
+
+  stop() {}
+}`,
+      errors: [
+        {
+          message: "Expected exactly two line breaks between class methods."
+        }
+      ]
+    },
+    {
+      code: `class Worker {
+  start() {}
+
+
+
+  stop() {}
+}`,
+      errors: [
+        {
+          message: "Expected exactly two line breaks between class methods."
+        }
+      ]
+    }
+  ]
+});

package/tests/rules/variable-names.test.js

@@ -0,0 +1,60 @@
+import rule from "../../rules/variable-names.js";
+import createRuleTester from "../rule-tester.js";
+
+const ruleTester = createRuleTester();
+
+ruleTester.run("variable-names", rule, {
+  valid: [
+    "let topLevel = 1; var otherTopLevel = 2;",
+    "function calculate() { const result = 1; let _temporary = result; }",
+    "const calculate = () => { let { value } = object; let [first] = values; };",
+    {
+      code: "function calculate() { let result = 1; }",
+      options: [
+        {
+          variableExceptions: ["result"]
+        }
+      ]
+    }
+  ],
+
+  invalid: [
+    {
+      code: "function calculate() { let result = 1; }",
+      errors: [
+        {
+          message: "Local variables should start with \"_\" ({result})"
+        }
+      ]
+    },
+    {
+      code: "const calculate = () => { var result; };",
+      errors: [
+        {
+          message: "Local variables should start with \"_\" ({result})"
+        }
+      ]
+    },
+    {
+      code: "class Calculator { calculate() { let result = 1; } }",
+      errors: [
+        {
+          message: "Local variables should start with \"_\" ({result})"
+        }
+      ]
+    },
+    {
+      code: "function calculate() { let allowed = 1; let rejected = 2; }",
+      options: [
+        {
+          variableExceptions: ["allowed"]
+        }
+      ],
+      errors: [
+        {
+          message: "Local variables should start with \"_\" ({rejected})"
+        }
+      ]
+    }
+  ]
+});

package/tests/rules/vue-attribute-comma.test.js

@@ -0,0 +1,43 @@
+import rule from "../../rules/vue-attribute-comma.js";
+import { createVueRuleTester } from "../rule-tester.js";
+
+const ruleTester = createVueRuleTester();
+const filename = "TestComponent.vue";
+
+ruleTester.run("vue-attribute-comma", rule, {
+  valid: [
+    {
+      filename,
+      code: `<template>
+  <button type="button" :aria-label="label">Save, now</button>
+</template>
+
+<script setup>
+const label = "Save";
+</script>`
+    },
+    {
+      filename,
+      code: `<template>
+  <Widget disabled />
+</template>`
+    }
+  ],
+
+  invalid: [
+    {
+      filename,
+      code: `<template>
+  <button type="button", disabled>Save</button>
+</template>`,
+      output: `<template>
+  <button type="button" disabled>Save</button>
+</template>`,
+      errors: [
+        {
+          message: "Comma after attribute value is not allowed."
+        }
+      ]
+    }
+  ]
+});

package/tests/rules/vue-attribute-linebreak.test.js

@@ -0,0 +1,54 @@
+import rule from "../../rules/vue-attribute-linebreak.js";
+import { createVueRuleTester } from "../rule-tester.js";
+
+const ruleTester = createVueRuleTester();
+const filename = "TestComponent.vue";
+
+ruleTester.run("vue-attribute-linebreak", rule, {
+  valid: [
+    {
+      filename,
+      code: `<template>
+  <section>
+    <button
+      type="button"
+      disabled
+    >
+      Save
+    </button>
+  </section>
+</template>`
+    }
+  ],
+
+  invalid: [
+    {
+      filename,
+      code: `<template>
+  <button type="button">Save</button>
+</template>`,
+      errors: [
+        {
+          message: "Expected a linebreak above this attribute."
+        },
+        {
+          message: "Expected a linebreak below this attribute."
+        }
+      ]
+    },
+    {
+      filename,
+      code: `<template>
+  <button type="button"
+  >
+    Save
+  </button>
+</template>`,
+      errors: [
+        {
+          message: "Expected a linebreak above this attribute."
+        }
+      ]
+    }
+  ]
+});

package/tests/rules/vue-class-attribute-last.test.js

@@ -0,0 +1,59 @@
+import rule from "../../rules/vue-class-attribute-last.js";
+import { createVueRuleTester } from "../rule-tester.js";
+
+const ruleTester = createVueRuleTester();
+const filename = "TestComponent.vue";
+
+ruleTester.run("vue-class-attribute-last", rule, {
+  valid: [
+    {
+      filename,
+      code: `<template>
+  <button
+    type="button"
+    :aria-label="label"
+    :class="{ active }"
+    class="button"
+    disabled
+  >
+    Save
+  </button>
+</template>
+
+<script setup>
+const active = true;
+const label = "Save";
+</script>`
+    }
+  ],
+
+  invalid: [
+    {
+      filename,
+      code: `<template>
+  <button class="button" type="button">Save</button>
+</template>`,
+      errors: [
+        {
+          message: "Attribute \"type\" must be declared before the \"class\" attribute."
+        }
+      ]
+    },
+    {
+      filename,
+      code: `<template>
+  <button :class="{ active }" :aria-label="label">Save</button>
+</template>
+
+<script setup>
+const active = true;
+const label = "Save";
+</script>`,
+      errors: [
+        {
+          message: "Attribute \":aria-label\" must be declared before the \":class\" attribute."
+        }
+      ]
+    }
+  ]
+});

package/tests/rules/vue-class-shorthand-max-len.test.js

@@ -0,0 +1,67 @@
+import rule from "../../rules/vue-class-shorthand-max-len.js";
+import { createPugRuleTester } from "../rule-tester.js";
+
+const ruleTester = createPugRuleTester();
+const filename = "TestComponent.vue";
+
+ruleTester.run("vue-class-shorthand-max-len", rule, {
+  valid: [
+    {
+      filename,
+      options: [
+        {
+          max: 9
+        }
+      ],
+      code: `<template lang="pug">
+  div.short
+</template>`
+    },
+    {
+      filename,
+      options: [
+        {
+          max: 5
+        }
+      ],
+      code: `<template lang="pug">
+  UserCard.very-long-class
+  user-card.very-long-class
+</template>`
+    }
+  ],
+
+  invalid: [
+    {
+      filename,
+      options: [
+        {
+          max: 8
+        }
+      ],
+      code: `<template lang="pug">
+  div.short
+</template>`,
+      errors: [
+        {
+          messageId: "tooLong",
+          data: {
+            length: 9,
+            max: 8
+          }
+        }
+      ]
+    },
+    {
+      filename,
+      code: `<template lang="pug">
+  div.this-class-name-is-deliberately-long-to-exceed-the-default-maximum-length-without-options-and-report
+</template>`,
+      errors: [
+        {
+          messageId: "tooLong"
+        }
+      ]
+    }
+  ]
+});

package/tests/rules/vue-computed-order.test.js

@@ -0,0 +1,27 @@
+import rule from "../../rules/vue-computed-order.js";
+import createRuleTester from "../rule-tester.js";
+
+const ruleTester = createRuleTester();
+
+ruleTester.run("vue-computed-order", rule, {
+  valid: [
+    {
+      code: "export default { computed: { alpha() {}, beta() {} } };"
+    },
+    {
+      code: "const options = { computed: { beta() {}, alpha() {} } };"
+    }
+  ],
+
+  invalid: [
+    {
+      code: "export default { computed: { beta() {}, alpha() {} } };",
+      output: "export default { computed: { alpha() {}, beta() {} } };",
+      errors: [
+        {
+          message: "Computed properties should be in alphabetical order."
+        }
+      ]
+    }
+  ]
+});

package/tests/rules/vue-data-comment.test.js

@@ -0,0 +1,61 @@
+import rule from "../../rules/vue-data-comment.js";
+import createRuleTester from "../rule-tester.js";
+
+const ruleTester = createRuleTester();
+
+ruleTester.run("vue-data-comment", rule, {
+  valid: [
+    {
+      code: `
+        export default {
+          data() {
+            return {
+              // --> STATE <--
+              ready: false
+            };
+          }
+        };
+      `
+    },
+    {
+      code: "export default { data() { return {}; } };"
+    }
+  ],
+
+  invalid: [
+    {
+      code: "export default { data() { return { ready: false }; } };",
+      errors: [
+        {
+          message:
+            "data() return object must start with a comment: " +
+            "'// --> STATE <--', '// --> DATA <--', " +
+            "'// --> ROUTE PARAMS <--', '// --> ROUTE QUERY <--', or " +
+            "'// --> COMPONENTS <--'."
+        }
+      ]
+    },
+    {
+      code: `
+        export default {
+          data() {
+            return {
+              // --> DATA <--
+              value: 1,
+              // --> UNKNOWN <--
+              other: 2
+            };
+          }
+        };
+      `,
+      errors: [
+        {
+          message:
+            "Invalid comment. Must be '// --> STATE <--', " +
+            "'// --> DATA <--', '// --> ROUTE PARAMS <--', " +
+            "'// --> ROUTE QUERY <--', or '// --> COMPONENTS <--'."
+        }
+      ]
+    }
+  ]
+});

package/tests/rules/vue-define-component.test.js

@@ -0,0 +1,45 @@
+import rule from "../../rules/vue-define-component.js";
+import createRuleTester from "../rule-tester.js";
+
+const ruleTester = createRuleTester();
+
+ruleTester.run("vue-define-component", rule, {
+  valid: [
+    {
+      code: "export default { name: \"Utility\" };",
+      filename: "/project/utility.js"
+    },
+    {
+      code: "export default defineComponent({});",
+      filename: "/project/Panel.vue"
+    },
+    {
+      code: "export default { name: \"Index\" };",
+      filename: "/project/mixins/index.js"
+    }
+  ],
+
+  invalid: [
+    {
+      code: "import { ref } from \"vue\";\nexport default { setup() { return { ref }; } };",
+      filename: "/project/Panel.vue",
+      output:
+        "import { ref, defineComponent } from \"vue\";\n" +
+        "export default defineComponent({ setup() { return { ref }; } });",
+      errors: [
+        {
+          message: "Mixins and components must use the `defineComponent` wrapper."
+        }
+      ]
+    },
+    {
+      code: "export default createOptions();",
+      filename: "/project/mixins/shared.js",
+      errors: [
+        {
+          message: "Mixins and components must use the `defineComponent` wrapper."
+        }
+      ]
+    }
+  ]
+});

package/tests/rules/vue-emits-order.test.js

@@ -0,0 +1,27 @@
+import rule from "../../rules/vue-emits-order.js";
+import createRuleTester from "../rule-tester.js";
+
+const ruleTester = createRuleTester();
+
+ruleTester.run("vue-emits-order", rule, {
+  valid: [
+    {
+      code: "export default { emits: [\"close\", \"open\"] };"
+    },
+    {
+      code: "const options = { emits: [\"open\", \"close\"] };"
+    }
+  ],
+
+  invalid: [
+    {
+      code: "export default { emits: [\"open\", \"close\"] };",
+      output: "export default { emits: [\"close\", \"open\"] };",
+      errors: [
+        {
+          message: "Emits should be in alphabetical order."
+        }
+      ]
+    }
+  ]
+});