Debug tests

Author: ASalavei

compose/ui/ui-uikit/src/iosMain/objc/CMPUIKitUtils/CMPUIKitUtils/CMPEditMenuView.h

@@ -47,5 +47,5 @@
 - (void)activateTextInputInteractionIfNeeded;
 
 - (void)deactivateTextInputInteractionIfNeeded;
-- (void)qweqwqwe;
+
 @end

compose/ui/ui-uikit/src/iosMain/objc/CMPUIKitUtils/CMPUIKitUtils/CMPEditMenuView.m

@@ -123,8 +123,6 @@ - (void)showEditMenuAtRect:(CGRect)targetRect
     self.selectAllBlock = selectAllBlock;
     self.customActions = customActions;
 
-    NSLog(@">>>>> OBJ-C SHOW EDIT MENU: %@", self);
-
     if (@available(iOS 16, *)) {
         [[CMPEditMenuViewRegister shared] hideAllMenusSkipping:self];
         if (self.editInteraction == nil || contextMenuItemsChanged || !self.isEditMenuShown) {
@@ -244,15 +242,8 @@ - (void)setEditInteraction:(UIEditMenuInteraction *)editInteraction API_AVAILABL
 - (void)presentEditMenuInteraction API_AVAILABLE(ios(16.0)) {
     NSAssert(self.editInteraction != nil, @"Edit Interaction must be initialized");
 
-    if (self.window == nil) {
-        NSLog(@"[CMPEditMenuView] presentEditMenuInteraction called with nil window. Stack trace:\n%@",
-              [NSThread callStackSymbols]);
-    }
-
     UIEditMenuConfiguration *config = [UIEditMenuConfiguration configurationWithIdentifier:nil
                                                                                sourcePoint:self.targetRect.origin];
-    // hideEditMenu
-    NSLog(@"[CMPEditMenuView] presentEditMenuInteraction called with config: %@ || view: %@", config, self);
     [self.editInteraction presentEditMenuWithConfiguration:config];
 }
 

compose/ui/ui/src/iosMain/kotlin/androidx/compose/ui/platform/UIKitTextInputService.ios.kt

@@ -416,8 +416,6 @@ internal class UIKitTextInputService(
                     val density = view.density
                     val offset = textUIView.frame.useContents { origin.asDpOffset().toOffset(density) }
                     val target = rect.translate(-offset).toDpRect(density).asCGRect()
-                    println(">>>> SHOW MENU AT $target | copy: ${onCopyRequested != null}, paste: ${onPasteRequested != null}, cut: ${onCutRequested != null}, selectAll: ${onSelectAllRequested != null} ")
-                    println(">>>>>>>>>> window? ${textUIView.window} | $textUIView")
                     textUIView.showEditMenuAtRect(
                         targetRect = target,
                         copy = onCopyRequested,
@@ -436,7 +434,6 @@ internal class UIKitTextInputService(
     override fun hide() {
         showMenuOrUpdatePosition = {}
         textUIView?.let {
-            println(">>>> HIDE")
             it.hideTextMenu()
             textMenuAppearanceChanged()
         }

compose/ui/ui/src/iosMain/kotlin/androidx/compose/ui/window/IntermediateTextInputUIView.ios.kt

@@ -156,7 +156,6 @@ internal class IntermediateTextInputUIView(
     override fun didMoveToWindow() {
         super.didMoveToWindow()
 
-        println(">>>>>>>>>> sis move to? ${window} | $this")
         if (usingNativeTextInput) {
             if (window != null && !selectionInteractionAttached) {
                 // Ensure UIKit text interaction is attached early so that cursor and selection

compose/ui/ui/src/uikitInstrumentedTest/kotlin/androidx/compose/ui/Configuration.kt

@@ -16,17 +16,13 @@
 
 package androidx.compose.ui
 
-import androidx.compose.ui.interaction.HapticFeedbackSelectionTest
-import androidx.compose.ui.interaction.TextFieldEditMenuTest
 import androidx.compose.xctest.setupXCTestSuite
 import kotlinx.cinterop.ExperimentalForeignApi
 import platform.XCTest.XCTestSuite
 
 @Suppress("unused")
 @OptIn(ExperimentalForeignApi::class)
 fun testSuite(): XCTestSuite = setupXCTestSuite(
-    HapticFeedbackSelectionTest::class,
-    TextFieldEditMenuTest::class,
     // Run all test cases from the tests
     // BasicInteractionTest::class,
     // LayersAccessibilityTest::class,

compose/ui/ui/src/uikitInstrumentedTest/kotlin/androidx/compose/ui/interaction/TextFieldEditMenuTest.kt

@@ -49,7 +49,6 @@ import androidx.compose.ui.test.assertVisibleInContainer
 import androidx.compose.ui.test.findNodeWithLabel
 import androidx.compose.ui.test.findNodeWithTag
 import androidx.compose.ui.test.firstNodeOrNull
-import androidx.compose.ui.test.getAccessibilityTree
 import androidx.compose.ui.test.runUIKitInstrumentedTest
 import androidx.compose.ui.test.utils.hold
 import androidx.compose.ui.test.utils.up
@@ -432,13 +431,9 @@ class TextFieldEditMenuTest {
     }
 
     private fun UIKitInstrumentedTest.openToolbar(textFieldTag: String) {
-        delay(50)
-        println(">> ---------------- tap $textFieldTag")
         findNodeWithTag(textFieldTag).tap()
         delay(500)
-        println(">> ---------------- double tap $textFieldTag")
         findNodeWithTag(textFieldTag).doubleTap()
-        println(">> ---------------- double tap done $textFieldTag")
         waitForContextMenu()
     }
 
@@ -499,18 +494,12 @@ class TextFieldEditMenuTest {
         } else {
             "UICalloutBar"
         }
-        println(">> ---------------- waitForContextMenu")
+
         waitForIdle()
-        try {
-            UIKitInstrumentedTest.waitUntil {
-                firstNodeOrNull { node ->
-                    node.element?.let { it::class.simpleName } == menuClassName
-                } != null
-            }
-        } catch (e: Throwable) {
-            println("Actual accessibility tree:")
-            println(getAccessibilityTree().printTree())
-            throw e
+        waitUntil {
+            firstNodeOrNull { node ->
+                node.element?.let { it::class.simpleName } == menuClassName
+            } != null
         }
         // Additional delay to wait until toolbar animation ends
         delay(500)

compose/ui/ui/src/uikitInstrumentedTest/kotlin/androidx/compose/ui/test/UIKitInstrumentedTest.kt

@@ -417,7 +417,7 @@ internal class UIKitInstrumentedTest(
     fun AccessibilityTestNode.doubleTap() {
         val frame = frame ?: error("Internal error. Frame is missing.")
         tap(frame.center())
-        delay(100)
+        delay(50)
         return tap(frame.center())
     }