fixing lint issues

Author: cmonfortep

app/src/androidTest/java/com/duckduckgo/app/email/EmailInjectorJsTest.kt

@@ -16,6 +16,7 @@
 
 package com.duckduckgo.app.email
 
+import android.annotation.SuppressLint
 import android.webkit.WebView
 import androidx.test.annotation.UiThreadTest
 import androidx.test.filters.SdkSuppress
@@ -59,25 +60,27 @@ class EmailInjectorJsTest {
         whenever(mockAutofill.isAnException(any())).thenReturn(false)
     }
 
+    @SuppressLint("DenyListedApi")
     @UiThreadTest
     @Test
     @SdkSuppress(minSdkVersion = 24)
     fun whenInjectAddressThenInjectJsCodeReplacingTheAlias() {
         val address = "address"
         val jsToEvaluate = getAliasJsToEvaluate().replace("%s", address)
         val webView = spy(WebView(InstrumentationRegistry.getInstrumentation().targetContext))
-        autofillFeature.self().setEnabled(Toggle.State(enable = true))
+        autofillFeature.self().setRawStoredState(Toggle.State(enable = true))
 
         testee.injectAddressInEmailField(webView, address, "https://example.com")
 
         verify(webView).evaluateJavascript(jsToEvaluate, null)
     }
 
+    @SuppressLint("DenyListedApi")
     @UiThreadTest
     @Test
     @SdkSuppress(minSdkVersion = 24)
     fun whenInjectAddressAndFeatureIsDisabledThenJsCodeNotInjected() {
-        autofillFeature.self().setEnabled(Toggle.State(enable = true))
+        autofillFeature.self().setRawStoredState(Toggle.State(enable = true))
 
         val address = "address"
         val webView = spy(WebView(InstrumentationRegistry.getInstrumentation().targetContext))
@@ -127,12 +130,13 @@ class EmailInjectorJsTest {
         verify(webView, never()).evaluateJavascript(jsToEvaluate, null)
     }
 
+    @SuppressLint("DenyListedApi")
     @UiThreadTest
     @Test
     @SdkSuppress(minSdkVersion = 24)
     fun whenNotifyWebAppSignEventAndUrlIsFromDuckDuckGoAndFeatureIsDisabledAndEmailIsNotSignedInThenDoNotEvaluateJsCode() {
         whenever(mockEmailManager.isSignedIn()).thenReturn(false)
-        autofillFeature.self().setEnabled(Toggle.State(enable = false))
+        autofillFeature.self().setRawStoredState(Toggle.State(enable = false))
 
         val jsToEvaluate = getNotifySignOutJsToEvaluate()
         val webView = spy(WebView(InstrumentationRegistry.getInstrumentation().targetContext))
@@ -142,12 +146,13 @@ class EmailInjectorJsTest {
         verify(webView, never()).evaluateJavascript(jsToEvaluate, null)
     }
 
+    @SuppressLint("DenyListedApi")
     @UiThreadTest
     @Test
     @SdkSuppress(minSdkVersion = 24)
     fun whenNotifyWebAppSignEventAndUrlIsFromDuckDuckGoAndFeatureIsEnabledAndEmailIsNotSignedInThenEvaluateJsCode() {
         whenever(mockEmailManager.isSignedIn()).thenReturn(false)
-        autofillFeature.self().setEnabled(Toggle.State(enable = true))
+        autofillFeature.self().setRawStoredState(Toggle.State(enable = true))
 
         val jsToEvaluate = getNotifySignOutJsToEvaluate()
         val webView = spy(WebView(InstrumentationRegistry.getInstrumentation().targetContext))

autofill/autofill-impl/src/main/java/com/duckduckgo/autofill/impl/email/ResultHandlerEmailProtectionChooseEmail.kt

@@ -133,7 +133,7 @@ class ResultHandlerEmailProtectionChooseEmail @Inject constructor(
     @Suppress("DEPRECATION")
     @SuppressLint("NewApi")
     private inline fun <reified T : Parcelable> Bundle.safeGetParcelable(key: String) =
-        if (appBuildConfig.sdkInt >= Build.VERSION_CODES.TIRAMISU) {
+        if (appBuildConfig.sdkInt >= 33) {
             getParcelable(key, T::class.java)
         } else {
             getParcelable(key)

autofill/autofill-impl/src/main/java/com/duckduckgo/autofill/impl/email/incontext/ResultHandlerInContextEmailProtectionPrompt.kt

@@ -90,7 +90,7 @@ class ResultHandlerInContextEmailProtectionPrompt @Inject constructor(
     @Suppress("DEPRECATION")
     @SuppressLint("NewApi")
     private inline fun <reified T : Parcelable> Bundle.safeGetParcelable(key: String) =
-        if (appBuildConfig.sdkInt >= Build.VERSION_CODES.TIRAMISU) {
+        if (appBuildConfig.sdkInt >= 33) {
             getParcelable(key, T::class.java)
         } else {
             getParcelable(key)

autofill/autofill-impl/src/main/java/com/duckduckgo/autofill/impl/ui/credential/saving/ResultHandlerSaveLoginCredentials.kt

@@ -80,7 +80,7 @@ class ResultHandlerSaveLoginCredentials @Inject constructor(
     @Suppress("DEPRECATION")
     @SuppressLint("NewApi")
     private inline fun <reified T : Parcelable> Bundle.safeGetParcelable(key: String) =
-        if (appBuildConfig.sdkInt >= Build.VERSION_CODES.TIRAMISU) {
+        if (appBuildConfig.sdkInt >= 33) {
             getParcelable(key, T::class.java)
         } else {
             getParcelable(key)

autofill/autofill-impl/src/main/java/com/duckduckgo/autofill/impl/ui/credential/selecting/ResultHandlerCredentialSelection.kt

@@ -138,7 +138,7 @@ class ResultHandlerCredentialSelection @Inject constructor(
     @Suppress("DEPRECATION")
     @SuppressLint("NewApi")
     private inline fun <reified T : Parcelable> Bundle.safeGetParcelable(key: String) =
-        if (appBuildConfig.sdkInt >= Build.VERSION_CODES.TIRAMISU) {
+        if (appBuildConfig.sdkInt >= 33) {
             getParcelable(key, T::class.java)
         } else {
             getParcelable(key)

autofill/autofill-impl/src/main/java/com/duckduckgo/autofill/impl/ui/credential/updating/ResultHandlerUpdateLoginCredentials.kt

@@ -78,7 +78,7 @@ class ResultHandlerUpdateLoginCredentials @Inject constructor(
     @Suppress("DEPRECATION")
     @SuppressLint("NewApi")
     private inline fun <reified T : Parcelable> Bundle.safeGetParcelable(key: String) =
-        if (appBuildConfig.sdkInt >= Build.VERSION_CODES.TIRAMISU) {
+        if (appBuildConfig.sdkInt >= 33) {
             getParcelable(key, T::class.java)
         } else {
             getParcelable(key)