Add detailed logging in AdbWrapperImpl to identify why isReady() returns false

Author: Grigory-Rylov

src/main/kotlin/com/github/grishberg/androidstudio/plugins/AdbWrapper.kt

@@ -11,29 +11,49 @@ interface AdbWrapper {
 }
 
 class AdbWrapperImpl(private val logger: AppLogger, project: Project) : AdbWrapper {
-    val androidBridge = AndroidSdkUtils.getDebugBridge(project)
+    private val androidBridge: com.android.ddmlib.AndroidDebugBridge?
 
     init {
-        logger.d("AdbWrapperImpl created, androidBridge = ${if (androidBridge != null) "NOT NULL" else "NULL"}")
+        logger.d("AdbWrapperImpl.init - calling AndroidSdkUtils.getDebugBridge(project)...")
+        androidBridge = AndroidSdkUtils.getDebugBridge(project)
+        logger.d("AdbWrapperImpl.init - AndroidSdkUtils.getDebugBridge() returned: ${if (androidBridge != null) "NOT NULL (hash=${androidBridge.hashCode()})" else "NULL"}")
+        if (androidBridge != null) {
+            logger.d("AdbWrapperImpl.init - bridge initial state: isConnected=${androidBridge.isConnected}, hasInitialDeviceList=${androidBridge.hasInitialDeviceList()}")
+        }
     }
 
     override fun isReady(): Boolean {
         return try {
             logger.d("AdbWrapperImpl.isReady() - checking bridge: ${if (androidBridge != null) "NOT NULL" else "NULL"}")
             if (androidBridge == null) {
                 logger.d("AdbWrapperImpl.isReady() - bridge is NULL, returning false")
+                logger.d("AdbWrapperImpl.isReady() - FAILURE REASON: androidBridge is null from AndroidSdkUtils.getDebugBridge()")
                 return false
             }
 
+            logger.d("AdbWrapperImpl.isReady() - bridge available, checking isConnected...")
             val isConnected = androidBridge.isConnected
+            logger.d("AdbWrapperImpl.isReady() - isConnected=$isConnected")
+            if (!isConnected) {
+                logger.d("AdbWrapperImpl.isReady() - FAILURE REASON: androidBridge.isConnected is false")
+                logger.d("AdbWrapperImpl.isReady() - bridge state details: ${androidBridge}")
+            }
+
+            logger.d("AdbWrapperImpl.isReady() - checking hasInitialDeviceList()...")
             val hasInitialList = androidBridge.hasInitialDeviceList()
-            logger.d("AdbWrapperImpl.isReady() - isConnected=$isConnected, hasInitialDeviceList=$hasInitialList")
+            logger.d("AdbWrapperImpl.isReady() - hasInitialDeviceList=$hasInitialList")
+            if (!hasInitialList) {
+                logger.d("AdbWrapperImpl.isReady() - FAILURE REASON: hasInitialDeviceList() is false")
+            }
+
             val result = isConnected && hasInitialList
-            logger.d("AdbWrapperImpl.isReady() - FINAL result: $result")
+            logger.d("AdbWrapperImpl.isReady() - FINAL result: $result (isConnected=$isConnected && hasInitialList=$hasInitialList)")
             result
         } catch (t: Throwable) {
             logger.e("AdbWrapperImpl.isReady() FAILED with exception", t)
             logger.e("  androidBridge is null: ${androidBridge == null}")
+            logger.e("  exception type: ${t.javaClass.simpleName}")
+            logger.e("  exception message: ${t.message}")
             false
         }
     }