[Test] minor cleanup

Author: honghaoz

ComposeUI/Sources/ComposeUI/CrossPlatform/ScrollView/ScrollViewType.swift

@@ -241,4 +241,11 @@ public extension ScrollViewType {
   var canScrollToBottom: Bool {
     contentOffset().y < maxOffsetY
   }
+
+  #if canImport(UIKit)
+  /// Stops any ongoing scroll deceleration, freezing the scroll view at its current position.
+  func stopDecelerating() {
+    setContentOffset(contentOffset(), animated: false)
+  }
+  #endif
 }

ComposeUI/Tests/ComposeUITests/Animations/CALayer+DisableActionsTests.swift

@@ -77,7 +77,7 @@ class CALayer_DisableActionsTests: XCTestCase {
     window.layer.addSublayer(layer)
 
     // wait for the layer to have a presentation layer
-    wait(timeout: 0.05)
+    expect(layer.presentation()).toEventuallyNot(beNil())
 
     // without delegate
     do {
@@ -108,7 +108,7 @@ class CALayer_DisableActionsTests: XCTestCase {
     window.layer.addSublayer(layer)
 
     // wait for the layer to have a presentation layer
-    wait(timeout: 0.05)
+    expect(layer.presentation()).toEventuallyNot(beNil())
 
     // with delegate - should trigger assertion when called from background thread
     do {

ComposeUI/Tests/ComposeUITests/ComposeNodes/LayoutCacheNodeTests.swift

@@ -64,7 +64,7 @@ class LayoutCacheNodeTests: XCTestCase {
     expect(state.layoutCount) == 2
 
     expect(state.renderCount) == 0
-    _ = cachedNode.renderableItems(in: CGRect(0, 0, 100, 100))
+    _ = cachedNode.renderableItems(in: CGRect(x: 0, y: 0, width: 100, height: 100))
     expect(state.renderCount) == 1
   }
 }

ComposeUI/Tests/ComposeUITests/ComposeNodes/ModifierNodeTests.swift

@@ -275,14 +275,8 @@ class ModifierNodeTests: XCTestCase {
       contentView.frame = CGRect(x: 0, y: 0, width: 100, height: 50)
       contentView.refresh(animated: true)
 
-      // Layer should eventually have full opacity after transition
-      let expectation = expectation(description: "opacity transition")
-      DispatchQueue.main.asyncAfter(deadline: .now() + 0.2) {
-        expect(layer?.opacity) == 1.0
-        expectation.fulfill()
-      }
-
-      wait(for: [expectation], timeout: 1)
+      // layer should eventually have full opacity after transition
+      expect(layer?.opacity).toEventually(beEqual(to: 1.0))
     }
   }
 

ComposeUI/Tests/ComposeUITests/ComposeView/ComposeView+RefreshTests.swift

@@ -112,7 +112,7 @@ class ComposeView_RefreshTests: XCTestCase {
     isAnimated = nil
 
     // wait for next run loop iteration
-    wait(timeout: 0.01)
+    wait(timeout: 1e-3)
 
     // then: expect the refresh is performed
     expect(DynamicLookup(view).property("pendingRefresh")) == nil

ComposeUI/Tests/ComposeUITests/ComposeView/TestNode.swift

@@ -46,7 +46,7 @@ struct TestNode: ComposeNode {
 
   var id: ComposeNodeId = .custom("test", isFixed: false)
 
-  var size: CGSize = .zero
+  var size: CGSize = CGSize(width: 0, height: 0)
 
   mutating func layout(containerSize: CGSize, context: ComposeNodeLayoutContext) -> ComposeNodeSizing {
     state.layoutCount += 1