From 119812507accc6dfa27ea1460f1f67d7273f88be Mon Sep 17 00:00:00 2001 From: David Perez Date: Wed, 14 May 2025 12:50:55 -0500 Subject: [PATCH] Remove logging from tests (#5192) --- .../ui/platform/feature/settings/other/OtherScreenTest.kt | 3 --- .../movetoorganization/VaultMoveToOrganizationScreenTest.kt | 4 ---- .../ui/platform/feature/debugmenu/DebugMenuScreenTest.kt | 5 ----- 3 files changed, 12 deletions(-) diff --git a/app/src/test/java/com/x8bit/bitwarden/ui/platform/feature/settings/other/OtherScreenTest.kt b/app/src/test/java/com/x8bit/bitwarden/ui/platform/feature/settings/other/OtherScreenTest.kt index 29af2e29ec..0e42083d4f 100644 --- a/app/src/test/java/com/x8bit/bitwarden/ui/platform/feature/settings/other/OtherScreenTest.kt +++ b/app/src/test/java/com/x8bit/bitwarden/ui/platform/feature/settings/other/OtherScreenTest.kt @@ -9,10 +9,8 @@ import androidx.compose.ui.test.isPopup import androidx.compose.ui.test.onAllNodesWithText import androidx.compose.ui.test.onNodeWithContentDescription import androidx.compose.ui.test.onNodeWithText -import androidx.compose.ui.test.onRoot import androidx.compose.ui.test.performClick import androidx.compose.ui.test.performScrollTo -import androidx.compose.ui.test.printToLog import com.bitwarden.core.data.repository.util.bufferedMutableSharedFlow import com.bitwarden.ui.util.asText import com.x8bit.bitwarden.data.platform.repository.model.ClearClipboardFrequency @@ -171,7 +169,6 @@ class OtherScreenTest : BaseComposeTest() { @Test fun `should display correct items according to state`() { mutableStateFlow.update { it.copy(isPreAuth = false) } - composeTestRule.onRoot().printToLog(tag = "BRIAN") composeTestRule .onNodeWithText(text = "Allow sync on refresh") .assertExists() diff --git a/app/src/test/java/com/x8bit/bitwarden/ui/vault/feature/movetoorganization/VaultMoveToOrganizationScreenTest.kt b/app/src/test/java/com/x8bit/bitwarden/ui/vault/feature/movetoorganization/VaultMoveToOrganizationScreenTest.kt index 022e239381..89d727f1f0 100644 --- a/app/src/test/java/com/x8bit/bitwarden/ui/vault/feature/movetoorganization/VaultMoveToOrganizationScreenTest.kt +++ b/app/src/test/java/com/x8bit/bitwarden/ui/vault/feature/movetoorganization/VaultMoveToOrganizationScreenTest.kt @@ -12,10 +12,8 @@ import androidx.compose.ui.test.onAllNodesWithText import androidx.compose.ui.test.onLast import androidx.compose.ui.test.onNodeWithContentDescription import androidx.compose.ui.test.onNodeWithText -import androidx.compose.ui.test.onRoot import androidx.compose.ui.test.performClick import androidx.compose.ui.test.performScrollTo -import androidx.compose.ui.test.printToLog import com.bitwarden.core.data.repository.util.bufferedMutableSharedFlow import com.x8bit.bitwarden.ui.platform.base.BaseComposeTest import com.bitwarden.ui.util.asText @@ -123,7 +121,6 @@ class VaultMoveToOrganizationScreenTest : BaseComposeTest() { @Test fun `the organization option field description should update according to state`() { - composeTestRule.onRoot().printToLog("Brian") composeTestRule .onNodeWithContentDescription(label = "Choose an organization that", substring = true) .assertIsDisplayed() @@ -172,7 +169,6 @@ class VaultMoveToOrganizationScreenTest : BaseComposeTest() { @Test fun `selecting an organization should send OrganizationSelect action`() { - composeTestRule.onRoot().printToLog("Brian") composeTestRule .onNodeWithContentDescriptionAfterScroll( label = "mockOrganizationName-1. Organization. " + diff --git a/authenticator/src/test/java/com/bitwarden/authenticator/ui/platform/feature/debugmenu/DebugMenuScreenTest.kt b/authenticator/src/test/java/com/bitwarden/authenticator/ui/platform/feature/debugmenu/DebugMenuScreenTest.kt index 357298e00f..307591cdec 100644 --- a/authenticator/src/test/java/com/bitwarden/authenticator/ui/platform/feature/debugmenu/DebugMenuScreenTest.kt +++ b/authenticator/src/test/java/com/bitwarden/authenticator/ui/platform/feature/debugmenu/DebugMenuScreenTest.kt @@ -2,10 +2,8 @@ package com.bitwarden.authenticator.ui.platform.feature.debugmenu import androidx.compose.ui.test.onNodeWithContentDescription import androidx.compose.ui.test.onNodeWithText -import androidx.compose.ui.test.onRoot import androidx.compose.ui.test.performClick import androidx.compose.ui.test.performScrollTo -import androidx.compose.ui.test.printToLog import com.bitwarden.authenticator.data.platform.manager.model.FlagKey import com.bitwarden.authenticator.ui.platform.base.BaseComposeTest import com.bitwarden.core.data.repository.util.bufferedMutableSharedFlow @@ -44,9 +42,6 @@ class DebugMenuScreenTest : BaseComposeTest() { @Test fun `onNavigateBack should send action to viewModel`() { - composeTestRule - .onRoot() - .printToLog("djf") composeTestRule .onNodeWithContentDescription("Back") .performClick()