Discover the login url for self-hosted sites and log it (#21851)

* Adding WordPress RS library to the login lib

* Injecting LoginSiteAddressViewModel

* Discovering the url

* Injecting WpLoginClient

* style and detekt

* Adding test class

* Explanatory comment about the inclusion of the APi discovery

* Fixing a warning

Author: adalpari

WordPress/src/main/java/org/wordpress/android/modules/ApplicationModule.java

@@ -40,6 +40,7 @@
 import org.wordpress.android.viewmodel.helpers.ConnectionStatusLiveData;
 
 import javax.inject.Named;
+import javax.inject.Singleton;
 
 import dagger.Binds;
 import dagger.Module;
@@ -49,6 +50,7 @@
 import dagger.hilt.android.qualifiers.ApplicationContext;
 import dagger.hilt.components.SingletonComponent;
 import kotlinx.coroutines.CoroutineScope;
+import rs.wordpress.api.kotlin.WpLoginClient;
 
 import static org.wordpress.android.modules.ThreadModuleKt.APPLICATION_SCOPE;
 
@@ -148,4 +150,10 @@ public static IAudioRecorder provideAudioRecorder(
     public static RecordingStrategy provideVoiceToContentRecordingStrategy() {
         return new VoiceToContentRecordingStrategy();
     }
+
+    @Provides
+    @Singleton
+    public static WpLoginClient provideWpLoginClient() {
+        return new WpLoginClient();
+    }
 }

WordPress/src/main/java/org/wordpress/android/modules/ViewModelModule.java

@@ -3,6 +3,7 @@
 import androidx.lifecycle.ViewModel;
 import androidx.lifecycle.ViewModelProvider;
 
+import org.wordpress.android.login.viewmodel.LoginSiteAddressViewModel;
 import org.wordpress.android.ui.accounts.LoginEpilogueViewModel;
 import org.wordpress.android.ui.accounts.LoginViewModel;
 import org.wordpress.android.ui.activitylog.list.filter.ActivityLogTypeFilterViewModel;
@@ -450,6 +451,11 @@ abstract class ViewModelModule {
     @ViewModelKey(LoginViewModel.class)
     abstract ViewModel loginViewModel(LoginViewModel viewModel);
 
+    @Binds
+    @IntoMap
+    @ViewModelKey(LoginSiteAddressViewModel.class)
+    abstract ViewModel loginSiteAddressViewModel(LoginSiteAddressViewModel viewModel);
+
     @Binds
     @IntoMap
     @ViewModelKey(StorageUtilsViewModel.class)

WordPress/src/test/java/org/wordpress/android/login/viewmodel/LoginSiteAddressViewModelTest.kt

@@ -0,0 +1,21 @@
+package org.wordpress.android.login.viewmodel
+
+import kotlinx.coroutines.ExperimentalCoroutinesApi
+import org.junit.Before
+import org.mockito.Mock
+import rs.wordpress.api.kotlin.WpLoginClient
+
+@ExperimentalCoroutinesApi
+ class LoginSiteAddressViewModelTest {
+    @Mock
+    lateinit var wpLoginClient: WpLoginClient
+
+    private lateinit var viewModel: LoginSiteAddressViewModel
+
+    @Before
+    fun setUp() {
+        viewModel = LoginSiteAddressViewModel(wpLoginClient)
+    }
+
+    // NOTE: there's actually nothing to test yet, since the initial function is just logging the server call
+ }

libs/login/build.gradle

@@ -56,6 +56,8 @@ dependencies {
     implementation libs.androidx.appcompat.main
 
     implementation libs.androidx.constraintlayout.main
+    implementation(libs.androidx.lifecycle.viewmodel.main)
+    implementation(libs.androidx.lifecycle.viewmodel.compose)
     implementation libs.google.material
 
     implementation libs.androidx.core.main
@@ -70,6 +72,8 @@ dependencies {
     implementation libs.androidx.credentials.main
     implementation libs.androidx.credentials.play.service.auth
 
+    implementation(libs.wordpress.rs.android)
+
     // Dagger
     implementation libs.google.dagger
     ksp libs.google.dagger.compiler

libs/login/src/main/java/org/wordpress/android/login/LoginSiteAddressFragment.kt

@@ -37,6 +37,8 @@ import org.wordpress.android.util.NetworkUtils
 import org.wordpress.android.util.UrlUtils
 import javax.inject.Inject
 import androidx.core.net.toUri
+import androidx.lifecycle.ViewModelProvider
+import org.wordpress.android.login.viewmodel.LoginSiteAddressViewModel
 
 class LoginSiteAddressFragment : LoginBaseDiscoveryFragment(), TextWatcher, OnEditorCommitListener,
     LoginBaseDiscoveryListener {
@@ -50,6 +52,10 @@ class LoginSiteAddressFragment : LoginBaseDiscoveryFragment(), TextWatcher, OnEd
 
     private var loginSiteAddressValidator: LoginSiteAddressValidator? = null
 
+    @Inject
+    lateinit var viewModelFactory: ViewModelProvider.Factory
+    private lateinit var viewModel: LoginSiteAddressViewModel
+
     @JvmField
     @Inject
     var accountStore: AccountStore? = null
@@ -128,6 +134,8 @@ class LoginSiteAddressFragment : LoginBaseDiscoveryFragment(), TextWatcher, OnEd
     override fun onActivityCreated(savedInstanceState: Bundle?) {
         super.onActivityCreated(savedInstanceState)
 
+        viewModel = ViewModelProvider(this, viewModelFactory)[LoginSiteAddressViewModel::class.java]
+
         if (savedInstanceState != null) {
             requestedSiteAddress = savedInstanceState.getString(KEY_REQUESTED_SITE_ADDRESS)
             connectSiteInfoUrl = savedInstanceState.getString(KEY_SITE_INFO_URL)
@@ -190,6 +198,10 @@ class LoginSiteAddressFragment : LoginBaseDiscoveryFragment(), TextWatcher, OnEd
 
         val cleanedUrl = stripKnownPaths(requestedSiteAddress.orEmpty())
 
+        // This work is in progress as right now we are just testing the API discovery through the RS library
+        // No further actions are taken
+        viewModel.runApiDiscovery(cleanedUrl)
+
         mAnalyticsListener.trackConnectedSiteInfoRequested(cleanedUrl)
         dispatcher?.dispatch(SiteActionBuilder.newFetchConnectSiteInfoAction(cleanedUrl))
 

libs/login/src/main/java/org/wordpress/android/login/viewmodel/LoginSiteAddressViewModel.kt

@@ -0,0 +1,25 @@
+package org.wordpress.android.login.viewmodel
+
+import android.util.Log
+import androidx.lifecycle.ViewModel
+import androidx.lifecycle.viewModelScope
+import kotlinx.coroutines.launch
+import rs.wordpress.api.kotlin.WpLoginClient
+import javax.inject.Inject
+
+@Suppress("TooGenericExceptionCaught")
+class LoginSiteAddressViewModel @Inject constructor(
+    private val wpLoginClient: WpLoginClient
+) : ViewModel() {
+    fun runApiDiscovery(url: String) {
+        viewModelScope.launch {
+            try {
+                val urlDiscovery = wpLoginClient.apiDiscovery(url)
+                val authorizationUrl = urlDiscovery.apiDetails.findApplicationPasswordsAuthenticationUrl()
+                Log.d("WP_RS", "VM: Found authorization URL: $authorizationUrl")
+            } catch (throwable: Throwable) {
+                Log.e("WP_RS", "VM: Error during API discovery", throwable)
+            }
+        }
+    }
+}