Merge pull request #221 from ReactiveCocoa/as-fix-action-replay-loop

Fix infinite feedback loop between Action states

Sources/Action.swift

@@ -102,8 +102,8 @@ public final class Action<Input, Output, Error: Swift.Error> {
 				}
 			}
 
-		self.isEnabled = state.map { $0.isEnabled }
-		self.isExecuting = state.map { $0.isExecuting }
+		self.isEnabled = state.map { $0.isEnabled }.skipRepeats()
+		self.isExecuting = state.map { $0.isExecuting }.skipRepeats()
 	}
 
 	/// Initializes an action that will be conditionally enabled, and creates a

Tests/ReactiveSwiftTests/ActionSpec.swift

@@ -7,7 +7,7 @@
 //
 
 import Foundation
-
+import Dispatch
 import Result
 import Nimble
 import Quick
@@ -114,6 +114,36 @@ class ActionSpec: QuickSpec {
 				expect(action.isExecuting.value) == false
 			}
 
+			if #available(macOS 10.10, *) {
+				it("should not loop indefinitely") {
+					let condition = MutableProperty(1)
+
+					let action = Action<Void, Void, NoError>(state: condition, enabledIf: { $0 == 0 }) { _ in
+						return .empty
+					}
+
+					let disposable = CompositeDisposable()
+
+					waitUntil(timeout: 0.01) { done in
+						let target = DispatchQueue(label: "test target queue")
+						let highPriority = QueueScheduler(qos: .userInitiated, targeting: target)
+						let lowPriority = QueueScheduler(qos: .default, targeting: target)
+
+						disposable += action.isExecuting.producer
+							.observe(on: highPriority)
+							.startWithValues { _ in
+								condition.value = 10
+							}
+
+						disposable += lowPriority.schedule {
+							done()
+						}
+					}
+
+					disposable.dispose()
+				}
+			}
+
 			describe("completed") {
 				beforeEach {
 					enabled.value = true