detekt

samgst-amazon · samgst-amazon · commit c78a536f30c8 · 2024-12-09T11:30:40.000-08:00
diff --git a/plugins/toolkit/jetbrains-core/tst/software/aws/toolkits/jetbrains/core/region/AwsRegionProviderTest.kt b/plugins/toolkit/jetbrains-core/tst/software/aws/toolkits/jetbrains/core/region/AwsRegionProviderTest.kt
@@ -167,14 +167,13 @@ class AwsRegionProviderTest {
                 override fun resolve(resource: RemoteResource): CompletionStage<Path> = CompletableFuture<Path>().apply {
                     complete(file)
                 }
+
                 // following two methods are not used in this test, make compiler happy
-                override fun checkForUpdates(endpoint: String, eTagProvider: ETagProvider): UpdateCheckResult {
-                    return UpdateCheckResult.NoUpdates
-                }
+                override fun checkForUpdates(endpoint: String, eTagProvider: ETagProvider): UpdateCheckResult =
+                    UpdateCheckResult.NoUpdates
 
-                override fun getLocalResourcePath(filename: String): Path? {
-                    return null
-                }
+                override fun getLocalResourcePath(filename: String): Path? =
+                    null
             }
         }
         ApplicationManager.getApplication().replaceService(RemoteResourceResolverProvider::class.java, mockRemoteResource, disposableRule.disposable)

Original file line number	Diff line number	Diff line change
`@@ -167,14 +167,13 @@ class AwsRegionProviderTest {`
`167`	`167`	`override fun resolve(resource: RemoteResource): CompletionStage<Path> = CompletableFuture<Path>().apply {`
`168`	`168`	`complete(file)`
`169`	`169`	`}`
	`170`	`+`
`170`	`171`	`// following two methods are not used in this test, make compiler happy`
`171`		`- override fun checkForUpdates(endpoint: String, eTagProvider: ETagProvider): UpdateCheckResult {`
`172`		`- return UpdateCheckResult.NoUpdates`
`173`		`- }`
	`172`	`+ override fun checkForUpdates(endpoint: String, eTagProvider: ETagProvider): UpdateCheckResult =`
	`173`	`+ UpdateCheckResult.NoUpdates`
`174`	`174`
`175`		`- override fun getLocalResourcePath(filename: String): Path? {`
`176`		`- return null`
`177`		`- }`
	`175`	`+ override fun getLocalResourcePath(filename: String): Path? =`
	`176`	`+ null`
`178`	`177`	`}`
`179`	`178`	`}`
`180`	`179`	`ApplicationManager.getApplication().replaceService(RemoteResourceResolverProvider::class.java, mockRemoteResource, disposableRule.disposable)`