@adyen/eslint-plugin-bento 2.9.0 → 2.10.0

package/rules/deprecation-components-has-error-prop/deprecation-components-has-error-prop.test.ts

@@ -0,0 +1,133 @@
+import { RuleTester } from 'eslint';
+import vueParser from 'vue-eslint-parser';
+import tsParser from '@typescript-eslint/parser';
+import deprecationComponentsHasErrorProp from './deprecation-components-has-error-prop';
+
+const ruleTester = new RuleTester({
+    languageOptions: {
+        ecmaVersion: 'latest',
+        sourceType: 'module',
+        parser: vueParser,
+        parserOptions: {
+            parser: tsParser,
+        },
+    },
+});
+
+describe('@adyen/bento/deprecation-components-has-error-prop', () => {
+    ruleTester.run('deprecation-components-has-error-prop', deprecationComponentsHasErrorProp, {
+        valid: [
+            {
+                name: 'Valid usage for bento-checkbox-group',
+                filename: 'test.vue',
+                code: `
+                <template>
+                    <bento-checkbox-group errorMessage="val" />
+                </template>
+                `,
+            },
+            {
+                name: 'Valid usage for bento-radio-group',
+                filename: 'test.vue',
+                code: `
+                <template>
+                    <bento-radio-group errorMessage="val" />
+                </template>
+                `,
+            },
+        ],
+        invalid: [
+            // bento-checkbox-group: Static
+            {
+                name: 'Invalid: Static prop on bento-checkbox-group',
+                filename: 'test.vue',
+                code: `<template><bento-checkbox-group has-error="val" /></template>`,
+                errors: [{ messageId: 'deprecation' }],
+            },
+            // bento-checkbox-group: Shorthand
+            {
+                name: 'Invalid: Dynamic prop on bento-checkbox-group',
+                filename: 'test.vue',
+                code: `<template><bento-checkbox-group :has-error="val" /></template>`,
+                errors: [{ messageId: 'deprecation' }],
+            },
+            // bento-radio-group: Static
+            {
+                name: 'Invalid: Static prop on bento-radio-group',
+                filename: 'test.vue',
+                code: `<template><bento-radio-group has-error="val" /></template>`,
+                errors: [{ messageId: 'deprecation' }],
+            },
+            // bento-radio-group: Shorthand
+            {
+                name: 'Invalid: Dynamic prop on bento-radio-group',
+                filename: 'test.vue',
+                code: `<template><bento-radio-group :has-error="val" /></template>`,
+                errors: [{ messageId: 'deprecation' }],
+            },
+        ],
+    });
+
+    /* ------------------------------------------------ */
+    /* AUTOFIX TESTS                                    */
+    /* ------------------------------------------------ */
+    ruleTester.run('deprecation-components-has-error-prop (autofix enabled)', deprecationComponentsHasErrorProp, {
+        valid: [
+            {
+                name: 'Valid: error-message prop on bento-checkbox-group (autofix enabled)',
+                filename: 'test.vue',
+                options: [{ autofix: true }],
+                code: `<template><bento-checkbox-group error-message="val" /></template>`,
+            },
+        ],
+        invalid: [
+            {
+                name: 'Autofix: static prop on bento-checkbox-group',
+                filename: 'test.vue',
+                options: [{ autofix: true }],
+                code: `<template><bento-checkbox-group has-error="val" /></template>`,
+                output: `<template><bento-checkbox-group error-message="val" /></template>`,
+                errors: [{ messageId: 'deprecation' }],
+            },
+            {
+                name: 'Autofix: dynamic prop on bento-checkbox-group',
+                filename: 'test.vue',
+                options: [{ autofix: true }],
+                code: `<template><bento-checkbox-group :has-error="val" /></template>`,
+                output: `<template><bento-checkbox-group :error-message="val" /></template>`,
+                errors: [{ messageId: 'deprecation' }],
+            },
+            {
+                name: 'Autofix: static prop on bento-radio-group',
+                filename: 'test.vue',
+                options: [{ autofix: true }],
+                code: `<template><bento-radio-group has-error="val" /></template>`,
+                output: `<template><bento-radio-group error-message="val" /></template>`,
+                errors: [{ messageId: 'deprecation' }],
+            },
+            {
+                name: 'Autofix: dynamic prop on bento-radio-group',
+                filename: 'test.vue',
+                options: [{ autofix: true }],
+                code: `<template><bento-radio-group :has-error="val" /></template>`,
+                output: `<template><bento-radio-group :error-message="val" /></template>`,
+                errors: [{ messageId: 'deprecation' }],
+            },
+            {
+                name: 'Autofix: v-bind longhand on bento-radio-group',
+                filename: 'test.vue',
+                options: [{ autofix: true }],
+                code: `<template><bento-radio-group v-bind:has-error="val" /></template>`,
+                output: `<template><bento-radio-group v-bind:error-message="val" /></template>`,
+                errors: [{ messageId: 'deprecation' }],
+            },
+            {
+                name: 'Autofix: does not fix when autofix is false',
+                filename: 'test.vue',
+                options: [{ autofix: false }],
+                code: `<template><bento-checkbox-group has-error="val" /></template>`,
+                errors: [{ messageId: 'deprecation' }],
+            },
+        ],
+    });
+});

package/rules/deprecation-input-filter-dropdown-options/README.md

@@ -0,0 +1,44 @@
+# deprecation-input-filter-dropdown-options
+
+Deprecates usage of `dropdown`, `dropdownPosition`, and `variant` properties inside `BentoInputFilterOptions`.
+
+These properties were used to configure the input filter as a dropdown variant. They should be replaced by using
+`BentoFilterItemType.INPUT_DROPDOWN` instead of `BentoFilterItemType.INPUT`.
+
+## Examples
+
+### Invalid
+
+```vue
+<script setup>
+    const config = [
+        {
+            field: 'name',
+            label: 'Name',
+            type: BentoFilterItemType.INPUT,
+            options: { dropdown: { items: [] }, dropdownPosition: 'end', variant: 'dropdown' },
+        },
+    ];
+</script>
+<template>
+    <bento-filter-bar :config="config" />
+</template>
+```
+
+### Valid
+
+```vue
+<script setup>
+    const config = [
+        {
+            field: 'name',
+            label: 'Name',
+            type: BentoFilterItemType.INPUT_DROPDOWN,
+            options: { dropdown: { items: [] }, dropdownPosition: 'end' },
+        },
+    ];
+</script>
+<template>
+    <bento-filter-bar :config="config" />
+</template>
+```

package/rules/deprecation-input-filter-dropdown-options/__tests__/deprecation-input-filter-dropdown-options.vue

@@ -0,0 +1,46 @@
+<template>
+    <div>
+        <bento-filter-bar :config="filterBarConfig" />
+        <bento-data-grid :filters="dataGridFilters" />
+        <bento-filter-bar
+            :config="[
+                {
+                    field: 'inline',
+                    label: 'Inline',
+                    type: BentoFilterItemType.INPUT,
+                    options: { dropdown: { items: [] } },
+                },
+            ]"
+        />
+        <bento-data-grid
+            :filters="[
+                {
+                    field: 'inlineGrid',
+                    label: 'Inline Grid',
+                    type: BentoFilterItemType.INPUT,
+                    options: { variant: 'dropdown' },
+                },
+            ]"
+        />
+    </div>
+</template>
+
+<script setup>
+    const filterBarConfig = [
+        {
+            field: 'name',
+            label: 'Name',
+            type: BentoFilterItemType.INPUT,
+            options: { dropdown: { items: [] }, dropdownPosition: 'end', variant: 'dropdown' },
+        },
+    ];
+
+    const dataGridFilters = [
+        {
+            field: 'status',
+            label: 'Status',
+            type: BentoFilterItemType.INPUT,
+            options: { dropdown: { items: [] } },
+        },
+    ];
+</script>

package/rules/deprecation-input-filter-dropdown-options/__tests__/eslint.config.js

@@ -0,0 +1,26 @@
+// eslint.config.js
+const { defineConfig } = require('eslint/config');
+const vueParser = require('vue-eslint-parser');
+const tsParser = require('@typescript-eslint/parser');
+const adyenBento = require('@adyen/eslint-plugin-bento');
+
+module.exports = defineConfig([
+    {
+        languageOptions: {
+            ecmaVersion: 'latest',
+            sourceType: 'module',
+            parser: vueParser,
+            parserOptions: {
+                parser: tsParser,
+            },
+        },
+        plugins: {
+            '@adyen/bento': adyenBento,
+        },
+        rules: {
+            // Register the specific rule we are testing
+            '@adyen/bento/deprecation-input-filter-dropdown-options': 'warn',
+        },
+        files: ['**/*.vue'],
+    },
+]);

package/rules/deprecation-input-filter-dropdown-options/deprecation-input-filter-dropdown-options.integration.test.ts

@@ -0,0 +1,27 @@
+import path from 'node:path';
+import { execFileSyncEslintVueFiles } from '../../utils/exec-file-sync-eslint-vue-files';
+
+describe('@adyen/bento/deprecation-input-filter-dropdown-options', () => {
+    it('should lint with warning', () => {
+        const eslintrc = path.resolve(__dirname, '__tests__', 'eslint.config.js');
+        const vueFile = path.resolve(__dirname, '__tests__', 'deprecation-input-filter-dropdown-options.vue');
+
+        const result = JSON.parse(execFileSyncEslintVueFiles(eslintrc, vueFile));
+
+        // 3 from filterBarConfig (dropdown, dropdownPosition, variant)
+        // + 1 from dataGridFilters (dropdown)
+        // + 1 from inline bento-filter-bar (dropdown)
+        // + 1 from inline bento-data-grid (variant)
+        expect(result[0].warningCount).toBe(6);
+
+        expect(result[0].messages).toEqual(
+            expect.arrayContaining([
+                expect.objectContaining({
+                    ruleId: '@adyen/bento/deprecation-input-filter-dropdown-options',
+                    severity: 1,
+                    messageId: 'deprecation',
+                }),
+            ])
+        );
+    });
+});

package/rules/deprecation-input-filter-dropdown-options/deprecation-input-filter-dropdown-options.js

@@ -0,0 +1,135 @@
+/**
+ * @typedef {import('eslint').Rule.RuleContext} RuleContext
+ */
+/**
+ * @typedef {import('eslint-plugin-vue/lib/utils').RuleModule} RuleModule
+ */
+
+const VueUtils = require('eslint-plugin-vue/lib/utils');
+
+const DEPRECATED_KEYS = ['dropdown', 'dropdownPosition', 'variant'];
+
+/** @type {RuleModule} */
+module.exports = {
+    meta: {
+        type: 'suggestion',
+        docs: {
+            description:
+                'Deprecates usage of "dropdown", "dropdownPosition", and "variant" properties inside BentoInputFilterOptions. Use BentoFilterItemType.INPUT_DROPDOWN instead.',
+            category: 'Deprecations',
+            url: 'https://github.com/Adyen/bento/blob/main/packages/eslint/rules/deprecation-input-filter-dropdown-options/README.md',
+            recommended: true,
+        },
+        messages: {
+            deprecation:
+                'The "{{ key }}" property in BentoInputFilterOptions is deprecated. Use "BentoFilterItemType.INPUT_DROPDOWN" instead of "BentoFilterItemType.INPUT".',
+        },
+        fixable: 'code',
+        schema: [
+            {
+                type: 'object',
+                properties: {
+                    autofix: {
+                        type: 'boolean',
+                        default: false,
+                    },
+                },
+                additionalProperties: false,
+            },
+        ],
+    },
+
+    /** @param {RuleContext} context */
+    create(context) {
+        const options = context.options[0] || {};
+        const autofix = options.autofix === true;
+
+        function getStaticPropertyName(prop) {
+            if (prop.type !== 'Property' || prop.computed) {
+                return null;
+            }
+            return prop.key.type === 'Identifier'
+                ? prop.key.name
+                : prop.key.type === 'Literal'
+                  ? String(prop.key.value)
+                  : null;
+        }
+
+        function reportDeprecatedKeys(node, typeValueNode) {
+            if (!node || node.type !== 'ObjectExpression') {
+                return;
+            }
+
+            for (const prop of node.properties) {
+                const keyName = getStaticPropertyName(prop);
+
+                if (keyName && DEPRECATED_KEYS.includes(keyName)) {
+                    context.report({
+                        node: prop.key,
+                        messageId: 'deprecation',
+                        data: { key: keyName },
+                        ...(autofix && {
+                            fix(fixer) {
+                                if (typeValueNode.type === 'MemberExpression') {
+                                    return fixer.replaceText(typeValueNode.property, 'INPUT_DROPDOWN');
+                                }
+                                return fixer.replaceText(typeValueNode, "'BentoInputWithDropdownFilter'");
+                            },
+                        }),
+                    });
+                }
+            }
+        }
+
+        function isInputFilterType(valueNode) {
+            if (
+                valueNode.type === 'MemberExpression' &&
+                valueNode.object.type === 'Identifier' &&
+                valueNode.object.name === 'BentoFilterItemType' &&
+                valueNode.property.type === 'Identifier' &&
+                valueNode.property.name === 'INPUT'
+            ) {
+                return true;
+            }
+
+            if (valueNode.type === 'Literal' && valueNode.value === 'BentoInputFilter') {
+                return true;
+            }
+
+            return false;
+        }
+
+        function checkFilterConfigObject(node) {
+            if (node.type !== 'ObjectExpression') {
+                return;
+            }
+
+            let typeValueNode = null;
+            let optionsNode = null;
+
+            for (const prop of node.properties) {
+                const keyName = getStaticPropertyName(prop);
+
+                if (keyName === 'type' && isInputFilterType(prop.value)) {
+                    typeValueNode = prop.value;
+                }
+
+                if (keyName === 'options' && prop.value.type === 'ObjectExpression') {
+                    optionsNode = prop.value;
+                }
+            }
+
+            if (typeValueNode && optionsNode) {
+                reportDeprecatedKeys(optionsNode, typeValueNode);
+            }
+        }
+
+        const visitor = {
+            ObjectExpression(node) {
+                checkFilterConfigObject(node);
+            },
+        };
+
+        return VueUtils.defineTemplateBodyVisitor(context, visitor, visitor);
+    },
+};