Rename FailFastOperationQueue to StressTesterOperationQueue

We are no longer using this queue to fail fast, so the name seems misleading.

Author: ahoppen

SourceKitStressTester/Sources/SwiftCWrapper/FailFastOperationQueue.swift renamed to SourceKitStressTester/Sources/SwiftCWrapper/StressTesterOperationQueue.swift

@@ -14,11 +14,11 @@ import Foundation
 
 /// An operation queue that stops scheduling new operations as soon as the the
 /// completion handler returns `false`.
-public final class FailFastOperationQueue<Item: Operation> {
-  private let serialQueue = DispatchQueue(label: "\(FailFastOperationQueue.self)")
+public final class StressTesterOperationQueue<Item: Operation> {
+  private let serialQueue = DispatchQueue(label: "\(StressTesterOperationQueue.self)")
   private let queue = OperationQueue()
   private let operations: [Item]
-  private let completionHandler: (Int, Item, Int, Int) -> Bool
+  private let completionHandler: (_ index: Int, _ operation: Item, _ operationsCompleted: Int, _ totalOperationCount: Int) -> Bool
 
   public init(operations: [Item], maxWorkers: Int? = nil,
        completionHandler: @escaping (Int, Item, Int, Int) -> Bool) {

SourceKitStressTester/Sources/SwiftCWrapper/SwiftCWrapper.swift

@@ -129,7 +129,7 @@ public struct SwiftCWrapper {
       }
     }
 
-    let queue = FailFastOperationQueue(operations: operations, maxWorkers: maxJobs) { index, operation, completed, total -> Bool in
+    let queue = StressTesterOperationQueue(operations: operations, maxWorkers: maxJobs) { index, operation, completed, total -> Bool in
       let message = "\(operation.file) (\(operation.summary)): \(operation.status.name)"
       progress?.update(step: completed, total: total, text: message)
       orderingHandler?.complete(operation.responses, at: index, setLast: !operation.status.isPassed)

SourceKitStressTester/Tests/SwiftCWrapperToolTests/SwiftCWrapperToolTests.swift

@@ -75,7 +75,7 @@ class SwiftCWrapperToolTests: XCTestCase {
     }
   }
 
-  func testFailFastOperationQueue() throws {
+  func testStressTesterOperationQueue() throws {
     class TestOperation: Operation {
       var waitCount: Int
 
@@ -96,13 +96,13 @@ class SwiftCWrapperToolTests: XCTestCase {
     let third = TestOperation(waitCount: 20)
     let fourth = TestOperation(waitCount: 30)
 
-    FailFastOperationQueue(operations: [first, second, third, fourth], maxWorkers: 2, completionHandler: { _, finishedOp, _, _ in
+    StressTesterOperationQueue(operations: [first, second, third, fourth], maxWorkers: 2, completionHandler: { _, finishedOp, _, _ in
       // cancel later operations when the second operation completes
       return finishedOp !== second
     }).waitUntilFinished()
 
-    XCTAssertTrue(!first.isCancelled, "first was cancelled")
-    XCTAssertTrue(!second.isCancelled, "second was cancelled")
+    XCTAssertFalse(first.isCancelled, "first was cancelled")
+    XCTAssertFalse(second.isCancelled, "second was cancelled")
     XCTAssertTrue(third.isCancelled, "third wasn't cancelled")
     XCTAssertTrue(fourth.isCancelled, "fourth wasn't cancelled")
   }