Skip to content

Commit eb2ad67

Browse files
committed
test to fix autofix
1 parent bf08eea commit eb2ad67

File tree

1 file changed

+15
-10
lines changed

1 file changed

+15
-10
lines changed

src/rules/new-color-css-vars.js

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -125,17 +125,22 @@ module.exports = {
125125
Object.keys(cssVars).forEach(cssVar => {
126126
if (Array.isArray(cssVars[cssVar])) {
127127
cssVars[cssVar].forEach(cssVarObject => {
128-
const regex = new RegExp(`var\\(${cssVar}\\)`, 'g')
129-
if (cssVarObject.props.some(prop => rawText.includes(prop)) && regex.test(rawText)) {
130-
const fixedString = rawText.replace(regex, `var(${cssVarObject.replacement}, var(${cssVar}))`)
131-
context.report({
132-
node,
133-
message: `Replace var(${cssVar}) with var(${cssVarObject.replacement}, var(${cssVar}))`,
134-
fix: function(fixer) {
135-
return fixer.replaceText(node, node.type === 'Literal' ? `"${fixedString}"` : fixedString)
128+
cssVarObject.props.forEach(prop => {
129+
const regex = new RegExp(`var\\(${cssVar}\\)`, 'g')
130+
if (rawText.includes(prop) && regex.test(rawText)) {
131+
const fixedString = rawText.replace(regex, `var(${cssVarObject.replacement}, var(${cssVar}))`)
132+
if (!rawText.includes(fixedString)) {
133+
// Check if the autofix has already been applied
134+
context.report({
135+
node,
136+
message: `Replace var(${cssVar}) with var(${cssVarObject.replacement}, var(${cssVar}))`,
137+
fix: function(fixer) {
138+
return fixer.replaceText(node, node.type === 'Literal' ? `"${fixedString}"` : fixedString)
139+
}
140+
})
136141
}
137-
})
138-
}
142+
}
143+
})
139144
})
140145
}
141146
})

0 commit comments

Comments
 (0)