fix some lint issues

flutter/android/src/main/kotlin/io/sentry/flutter/SentryFlutter.kt

@@ -17,34 +17,64 @@ class SentryFlutter(
     data.getIfNotNull<String>("dsn") {
       options.dsn = it
     }
-    data.getIfNotNull<Boolean>("debug") { options.isDebug = it }
-    data.getIfNotNull<String>("environment") { options.environment = it }
-    data.getIfNotNull<String>("release") { options.release = it }
-    data.getIfNotNull<String>("dist") { options.dist = it }
-    data.getIfNotNull<Boolean>("enableAutoSessionTracking") { options.isEnableAutoSessionTracking = it }
-    data.getIfNotNull<Long>("autoSessionTrackingIntervalMillis") { options.sessionTrackingIntervalMillis = it }
-    data.getIfNotNull<Long>("anrTimeoutIntervalMillis") { options.anrTimeoutIntervalMillis = it }
-    data.getIfNotNull<Boolean>("attachThreads") { options.isAttachThreads = it }
-    data.getIfNotNull<Boolean>("attachStacktrace") { options.isAttachStacktrace = it }
+    data.getIfNotNull<Boolean>("debug") {
+      options.isDebug = it
+    }
+    data.getIfNotNull<String>("environment") {
+      options.environment = it
+    }
+    data.getIfNotNull<String>("release") {
+      options.release = it
+    }
+    data.getIfNotNull<String>("dist") {
+      options.dist = it
+    }
+    data.getIfNotNull<Boolean>("enableAutoSessionTracking") {
+      options.isEnableAutoSessionTracking = it
+    }
+    data.getIfNotNull<Long>("autoSessionTrackingIntervalMillis") {
+      options.sessionTrackingIntervalMillis = it
+    }
+    data.getIfNotNull<Long>("anrTimeoutIntervalMillis") {
+      options.anrTimeoutIntervalMillis = it
+    }
+    data.getIfNotNull<Boolean>("attachThreads") {
+      options.isAttachThreads = it
+    }
+    data.getIfNotNull<Boolean>("attachStacktrace") {
+      options.isAttachStacktrace = it
+    }
     data.getIfNotNull<Boolean>("enableAutoNativeBreadcrumbs") {
       options.isEnableActivityLifecycleBreadcrumbs = it
       options.isEnableAppLifecycleBreadcrumbs = it
       options.isEnableSystemEventBreadcrumbs = it
       options.isEnableAppComponentBreadcrumbs = it
       options.isEnableUserInteractionBreadcrumbs = it
     }
-    data.getIfNotNull<Int>("maxBreadcrumbs") { options.maxBreadcrumbs = it }
-    data.getIfNotNull<Int>("maxCacheItems") { options.maxCacheItems = it }
+    data.getIfNotNull<Int>("maxBreadcrumbs") {
+      options.maxBreadcrumbs = it
+    }
+    data.getIfNotNull<Int>("maxCacheItems") {
+      options.maxCacheItems = it
+    }
     data.getIfNotNull<String>("diagnosticLevel") {
       if (options.isDebug) {
         val sentryLevel = SentryLevel.valueOf(it.toUpperCase(Locale.ROOT))
         options.setDiagnosticLevel(sentryLevel)
       }
     }
-    data.getIfNotNull<Boolean>("anrEnabled") { options.isAnrEnabled = it }
-    data.getIfNotNull<Boolean>("sendDefaultPii") { options.isSendDefaultPii = it }
-    data.getIfNotNull<Boolean>("enableNdkScopeSync") { options.isEnableScopeSync = it }
-    data.getIfNotNull<String>("proguardUuid") { options.proguardUuid = it }
+    data.getIfNotNull<Boolean>("anrEnabled") {
+      options.isAnrEnabled = it
+    }
+    data.getIfNotNull<Boolean>("sendDefaultPii") {
+      options.isSendDefaultPii = it
+    }
+    data.getIfNotNull<Boolean>("enableNdkScopeSync") {
+      options.isEnableScopeSync = it
+    }
+    data.getIfNotNull<String>("proguardUuid") {
+      options.proguardUuid = it
+    }
 
     val nativeCrashHandling = (data["enableNativeCrashHandling"] as? Boolean) ?: true
     // nativeCrashHandling has priority over anrEnabled
@@ -62,9 +92,13 @@ class SentryFlutter(
       }
     }
 
-    data.getIfNotNull<Boolean>("sendClientReports") { options.isSendClientReports = it }
+    data.getIfNotNull<Boolean>("sendClientReports") {
+      options.isSendClientReports = it
+    }
 
-    data.getIfNotNull<Long>("maxAttachmentSize") { options.maxAttachmentSize = it }
+    data.getIfNotNull<Long>("maxAttachmentSize") {
+      options.maxAttachmentSize = it
+    }
 
     var sdkVersion = options.sdkVersion
     if (sdkVersion == null) {
@@ -77,8 +111,12 @@ class SentryFlutter(
     options.sentryClientName = "$androidSdk/${BuildConfig.VERSION_NAME}"
     options.nativeSdkName = nativeSdk
 
-    data.getIfNotNull<Int>("connectionTimeoutMillis") { options.connectionTimeoutMillis = it }
-    data.getIfNotNull<Int>("readTimeoutMillis") { options.readTimeoutMillis = it }
+    data.getIfNotNull<Int>("connectionTimeoutMillis") {
+      options.connectionTimeoutMillis = it
+    }
+    data.getIfNotNull<Int>("readTimeoutMillis") {
+      options.readTimeoutMillis = it
+    }
   }
 }
 

flutter/android/src/test/kotlin/io/sentry/flutter/SentryFlutterTest.kt

@@ -22,31 +22,34 @@ class SentryFlutterTest {
     val sut = fixture.getSut()
 
     // When
-    sut.updateOptions(fixture.options, mapOf(
-      "dsn" to "fixture-dsn",
-      "debug" to true,
-      "environment" to "fixture-environment",
-      "release" to "fixture-release",
-      "dist" to "fixture-dist",
-      "enableAutoSessionTracking" to false,
-      "autoSessionTrackingIntervalMillis" to 9001L,
-      "anrTimeoutIntervalMillis" to 9002L,
-      "attachThreads" to true,
-      "attachStacktrace" to false,
-      "enableAutoNativeBreadcrumbs" to false,
-      "maxBreadcrumbs" to 9003,
-      "maxCacheItems" to 9004,
-      "anrEnabled" to false,
-      "sendDefaultPii" to true,
-      "enableNdkScopeSync" to false,
-      "proguardUuid" to "fixture-proguardUuid",
-      "enableNativeCrashHandling" to false,
-      "sendClientReports" to false,
-      "maxAttachmentSize" to 9005L,
-      "enableAutoPerformanceTracing" to true,
-      "connectionTimeoutMillis" to 9006,
-      "readTimeoutMillis" to 9007,
-    ))
+    sut.updateOptions(
+      fixture.options,
+      mapOf(
+        "dsn" to "fixture-dsn",
+        "debug" to true,
+        "environment" to "fixture-environment",
+        "release" to "fixture-release",
+        "dist" to "fixture-dist",
+        "enableAutoSessionTracking" to false,
+        "autoSessionTrackingIntervalMillis" to 9001L,
+        "anrTimeoutIntervalMillis" to 9002L,
+        "attachThreads" to true,
+        "attachStacktrace" to false,
+        "enableAutoNativeBreadcrumbs" to false,
+        "maxBreadcrumbs" to 9003,
+        "maxCacheItems" to 9004,
+        "anrEnabled" to false,
+        "sendDefaultPii" to true,
+        "enableNdkScopeSync" to false,
+        "proguardUuid" to "fixture-proguardUuid",
+        "enableNativeCrashHandling" to false,
+        "sendClientReports" to false,
+        "maxAttachmentSize" to 9005L,
+        "enableAutoPerformanceTracing" to true,
+        "connectionTimeoutMillis" to 9006,
+        "readTimeoutMillis" to 9007
+      )
+    )
 
     // Then
     assertEquals("fixture-dsn", fixture.options.dsn)
@@ -91,9 +94,12 @@ class SentryFlutterTest {
     fixture.options.isDebug = true
 
     // When
-    sut.updateOptions(fixture.options, mapOf(
-      "diagnosticLevel" to "warning",
-    ))
+    sut.updateOptions(
+      fixture.options,
+      mapOf(
+        "diagnosticLevel" to "warning",
+      )
+    )
 
     // Then
     assertEquals(SentryLevel.WARNING, fixture.options.diagnosticLevel)
@@ -105,9 +111,12 @@ class SentryFlutterTest {
     val sut = fixture.getSut()
 
     // When
-    sut.updateOptions(fixture.options, mapOf(
-      "enableNativeCrashHandling" to false,
-    ))
+    sut.updateOptions(
+      fixture.options,
+      mapOf(
+        "enableNativeCrashHandling" to false,
+      )
+    )
 
     // Then
     assertEquals(false, fixture.options.isEnableUncaughtExceptionHandler)