fix: address claude review nits

Author: ben-kaufman

app/src/main/java/to/bitkit/ui/components/ProfileEditForm.kt

@@ -155,7 +155,7 @@ fun ProfileEditForm(
                             painter = painterResource(R.drawable.ic_pencil_simple),
                             contentDescription = null,
                             tint = Colors.White64,
-                            modifier = Modifier.size(16.dp),
+                            modifier = Modifier.size(16.dp)
                         )
                         IconButton(onClick = { onRemoveLink(index) }) {
                             Icon(
@@ -195,7 +195,7 @@ fun ProfileEditForm(
                         modifier = Modifier.size(16.dp)
                     )
                 },
-                modifier = Modifier.testTag("ProfileEditAddLink"),
+                modifier = Modifier.testTag("ProfileEditAddLink")
             )
         }
 
@@ -237,7 +237,7 @@ fun ProfileEditForm(
                         modifier = Modifier.size(16.dp)
                     )
                 },
-                modifier = Modifier.testTag("ProfileEditAddTag"),
+                modifier = Modifier.testTag("ProfileEditAddTag")
             )
         }
 

app/src/main/java/to/bitkit/ui/components/Tag.kt

@@ -50,7 +50,7 @@ fun TagButton(
             color = textColor,
             maxLines = 1,
             overflow = TextOverflow.Ellipsis,
-            modifier = Modifier.testTag("Tag-$text"),
+            modifier = Modifier.testTag("Tag-$text")
         )
 
         if (displayIconClose) {

app/src/test/java/to/bitkit/ui/WalletViewModelTest.kt

@@ -5,7 +5,6 @@ import kotlinx.coroutines.ExperimentalCoroutinesApi
 import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.runBlocking
 import kotlinx.coroutines.test.advanceUntilIdle
-import org.junit.Assert.assertEquals
 import org.junit.Before
 import org.junit.Test
 import org.lightningdevkit.ldknode.PeerDetails
@@ -33,6 +32,7 @@ import to.bitkit.services.MigrationService
 import to.bitkit.test.BaseUnitTest
 import to.bitkit.viewmodels.RestoreState
 import to.bitkit.viewmodels.WalletViewModel
+import kotlin.test.assertEquals
 
 @OptIn(ExperimentalCoroutinesApi::class)
 class WalletViewModelTest : BaseUnitTest() {
@@ -62,7 +62,7 @@ class WalletViewModelTest : BaseUnitTest() {
         whenever(migrationService.isMigrationChecked()).thenReturn(true)
         whenever(migrationService.isChannelRecoveryChecked()).thenReturn(true)
         whenever(migrationService.tryFetchMigrationPeersFromBackup()).thenReturn(emptyList())
-        whenever(migrationService.getRNRemoteBackupTimestamp()).thenReturn(null)
+        whenever { migrationService.getRNRemoteBackupTimestamp() }.thenReturn(null)
         whenever(connectivityRepo.isOnline).thenReturn(isOnline)
 
         sut = WalletViewModel(

app/src/test/java/to/bitkit/ui/screens/contacts/ContactImportFlowTest.kt

@@ -7,6 +7,11 @@ import kotlin.test.assertEquals
 import kotlin.test.assertNull
 
 class ContactImportFlowTest {
+    private companion object {
+        const val VALID_PUBLIC_KEY = "pubkyybndrfg8ejkmcpqxot1uwisza345h769ybndrfg8ejkmcpqxot1u"
+        const val OTHER_VALID_PUBLIC_KEY = "pubkya345h769ybndrfg8ejkmcpqxot1uwiszybndrfg8ejkmcpqxot1u"
+    }
+
     @Test
     fun `resolveAddContactValidation returns empty for blank input`() {
         assertEquals(
@@ -87,9 +92,4 @@ class ContactImportFlowTest {
             ),
         )
     }
-
-    private companion object {
-        const val VALID_PUBLIC_KEY = "pubkyybndrfg8ejkmcpqxot1uwisza345h769ybndrfg8ejkmcpqxot1u"
-        const val OTHER_VALID_PUBLIC_KEY = "pubkya345h769ybndrfg8ejkmcpqxot1uwiszybndrfg8ejkmcpqxot1u"
-    }
 }