Satisfy detekt and checkstyle

- @Suppress("ReturnCount") on ensureAuth — its five returns are each a distinct
  auth outcome; a single-return rewrite would read worse.
- Drop a stray blank line before a brace left from removing a test region.

Author: jkmassel

WordPress/src/main/java/org/wordpress/android/repositories/SiteProvisioningSource.kt

@@ -159,6 +159,9 @@ class SiteProvisioningSource @Inject constructor(
      * confirmed rejection wipes them and mints fresh ones via the FluxC Jetpack
      * tunnel. The mint persists the credentials, so later stages read them back.
      */
+    // Each return is a distinct auth outcome (missing site, valid, transient, minted, failed);
+    // collapsing to one return would thread a result through nested branches and read worse.
+    @Suppress("ReturnCount")
     private suspend fun ensureAuth(siteLocalId: Int): SiteAuthState {
         val site = siteStore.getSiteByLocalId(siteLocalId)
             ?: return SiteAuthState.Unprovisionable(hadCredentials = false)

WordPress/src/test/java/org/wordpress/android/ui/posts/GutenbergEditorPreloaderTest.kt

@@ -481,5 +481,4 @@ class GutenbergEditorPreloaderTest :
     }
 
     // endregion
-
 }