fix: fixed Class action

packages/svelte-materialify/src/internal/Class/index.js

@@ -1,17 +1,21 @@
+const filter = (classes) => classes.filter((x) => !!x);
+const format = (classes) => classes.split(' ').filter((x) => !!x);
+
 /**
  * @param node {Element}
  * @param classes {Array<string>}
  */
-export default (node, classes) => {
-  classes.forEach((klass) => {
-    if (klass) node.classList.add(klass);
-  });
+export default (node, _classes) => {
+  let classes = _classes;
+  node.classList.add(...format(filter(classes).join(' ')));
   return {
-    update(newClasses) {
-      newClasses.forEach((klass) => {
-        if (klass) node.classList.add(klass);
-        else node.classList.remove(klass);
+    update(_newClasses) {
+      const newClasses = _newClasses;
+      newClasses.forEach((klass, i) => {
+        if (klass) node.classList.add(...format(klass));
+        else if (classes[i]) node.classList.remove(...format(classes[i]));
       });
+      classes = newClasses;
     },
   };
 };

packages/svelte-materialify/tests/internal/spec.js

@@ -0,0 +1,26 @@
+import Class from '@s/internal/Class';
+
+describe('internal actions', () => {
+  test('Class', () => {
+    document.body.appendChild(document.createElement('div'));
+
+    const div = document.querySelector('div');
+    const { classList } = div;
+
+    const initial = [true && 'a', false && 'b'];
+    const { update } = Class(div, initial);
+    expect(classList.contains('a')).toBe(true);
+
+    update([true && 'a', true && 'b']);
+    expect(classList.contains('a')).toBe(true);
+    expect(classList.contains('b')).toBe(true);
+
+    update([false && 'a', true && 'b']);
+    expect(classList.contains('b')).toBe(true);
+    expect(classList.contains('a')).toBe(false);
+
+    update([false && 'a', false && 'b']);
+    expect(classList.contains('a')).toBe(false);
+    expect(classList.contains('b')).toBe(false);
+  });
+});