Fixed compareAndSwap

Author: wkornewald

CHANGELOG.md

@@ -11,6 +11,7 @@
 * Removed `ValueFlow`, `MutableValueFlow` and `SuspendMutableValueFlow`.
 * Renamed `launcherScope` to `scope` on `CoroutineLauncher` and all subtypes like `BaseReactiveState`.
 * Renamed modules: `reactivestate` -> `reactivestate-android`, `reactivestate-test` -> `reactivestate-android-test`. You might only want to use `reactivestate-compose` in case you don't need to add ViewModels to Activities/Fragments.
+* Fixed `toMutable` and other interceptors' `compareAndSwap`.
 
 ## 5.13.0
 

reactivestate-core/src/commonMain/kotlin/com/ensody/reactivestate/StateFlowInterceptor.kt

@@ -87,7 +87,7 @@ private class StateFlowInterceptor<T>(
 
     override fun compareAndSet(expect: T, update: T): Boolean =
         mutex.withSpinLock {
-            if (value != expect) {
+            if (value == expect) {
                 value = update
                 true
             } else {

reactivestate-core/src/commonTest/kotlin/com/ensody/reactivestate/InterceptMutableStateFlowTest.kt

@@ -4,6 +4,7 @@ import com.ensody.reactivestate.test.CoroutineTest
 import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.StateFlow
 import kotlinx.coroutines.flow.asStateFlow
+import kotlinx.coroutines.flow.update
 import kotlin.test.Test
 import kotlin.test.assertEquals
 
@@ -17,6 +18,10 @@ internal class InterceptMutableStateFlowTest : CoroutineTest() {
         intercepted.value = 1
         assertEquals(1, base.value)
         assertEquals(1, intercepted.value)
+        // Tests compareAndSwap
+        intercepted.update { 2 }
+        assertEquals(2, base.value)
+        assertEquals(2, intercepted.value)
     }
 
     @Test