fix: @putout/operator-jsx: removeClassName: middle

Author: coderaiser

packages/operator-jsx/lib/jsx.js

@@ -91,8 +91,13 @@ function removeAttributeValue(path, name, attributeValue) {
 
     const {value} = classAttribute.value;
 
-    if (value.includes(attributeValue))
-        setLiteralValue(classAttribute.value, value.replace(RegExp(`\\s?${attributeValue}\\s?`), ''));
+    if (!value.includes(attributeValue))
+        return;
+    
+    const newValue = value
+        .replace(RegExp(`\\s?${attributeValue}`), '')
+        .trim();
+    setLiteralValue(classAttribute.value, newValue);
 }
 
 module.exports.setAttributeValue = (path, name, value) => {

packages/operator-jsx/lib/jsx.spec.js

@@ -187,6 +187,17 @@ test('putout: operator: jsx: removeAttributeValue', (t) => {
     t.end();
 });
 
+test('putout: operator: jsx: removeAttributeValue: no', (t) => {
+    const node = template.ast.fresh('<hello className="hello world"/>');
+    removeAttributeValue(node, 'className', 'abc');
+    
+    const result = print(node);
+    const expected = `<hello className="hello world"/>;\n`;
+    
+    t.equal(result, expected);
+    t.end();
+});
+
 test('putout: operator: jsx: setAttributeValue', (t) => {
     const node = template.ast.fresh('<hello className="hello"/>');
     setAttributeValue(node, 'className', 'world');
@@ -250,6 +261,17 @@ test('putout: operator: jsx: removeClassName: first', (t) => {
     t.end();
 });
 
+test('putout: operator: jsx: removeClassName: middle', (t) => {
+    const node = template.ast.fresh('<hello className="hello world abc"/>');
+    removeClassName(node, 'world');
+    
+    const result = print(node);
+    const expected = `<hello className="hello abc"/>;\n`;
+    
+    t.equal(result, expected);
+    t.end();
+});
+
 test('putout: operator: jsx: containsClassName', (t) => {
     const node = template.ast.fresh('<hello className="hello world"/>');
     const result = containsClassName(node, 'world');