refactor(Bazel)!: Rename the Bazel registry service

In an upcoming change, a local registry service will be introduced, requiring this renaming. Signed-off-by: Nicolas Nobelis <[email protected]>
oss-review-toolkit · Jun 24, 2024 · 50c0512 · 50c0512
1 parent 72c1a14
commit 50c0512
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/clients/bazel-module-registry/src/funTest/kotlin/BazelModuleRegistryClientFunTest.kt b/clients/bazel-module-registry/src/funTest/kotlin/BazelModuleRegistryClientFunTest.kt
@@ -25,7 +25,7 @@ import io.kotest.matchers.comparables.shouldBeGreaterThan
 import io.kotest.matchers.shouldBe
 
 class BazelModuleRegistryClientFunTest : WordSpec({
-    val client = BazelModuleRegistryService.create()
+    val client = RemoteBazelModuleRegistryService.create()
     val repoUrl = "https://github.com/google/glog"
 
     "getModuleMetadata" should {

diff --git a/...main/kotlin/BazelModuleRegistryService.kt → ...otlin/RemoteBazelModuleRegistryService.kt b/...main/kotlin/BazelModuleRegistryService.kt → ...otlin/RemoteBazelModuleRegistryService.kt
@@ -47,13 +47,13 @@ private const val DEFAULT_URL = "https://bcr.bazel.build"
  * Interface for a Bazel Module Registry, based on the directory structure of https://bcr.bazel.build/ and the Git
  * repository it is based on (https://github.com/bazelbuild/bazel-central-registry/).
  */
-interface BazelModuleRegistryService {
+interface RemoteBazelModuleRegistryService {
     companion object {
         /**
          * Create a Bazel Module Registry client instance for communicating with a server running at the given [url],
          * defaulting to the Bazel Central Registry, optionally with a pre-built OkHttp [client].
          */
-        fun create(url: String? = null, client: OkHttpClient? = null): BazelModuleRegistryService {
+        fun create(url: String? = null, client: OkHttpClient? = null): RemoteBazelModuleRegistryService {
             val bmrClient = client ?: OkHttpClient()
 
             val contentType = "application/json".toMediaType()
@@ -63,7 +63,7 @@ interface BazelModuleRegistryService {
                 .addConverterFactory(JSON.asConverterFactory(contentType))
                 .build()
 
-            return retrofit.create(BazelModuleRegistryService::class.java)
+            return retrofit.create(RemoteBazelModuleRegistryService::class.java)
         }
     }
 

diff --git a/plugins/package-managers/bazel/src/main/kotlin/Bazel.kt b/plugins/package-managers/bazel/src/main/kotlin/Bazel.kt
@@ -29,7 +29,7 @@ import org.apache.logging.log4j.kotlin.logger
 
 import org.ossreviewtoolkit.analyzer.AbstractPackageManagerFactory
 import org.ossreviewtoolkit.analyzer.PackageManager
-import org.ossreviewtoolkit.clients.bazelmoduleregistry.BazelModuleRegistryService
+import org.ossreviewtoolkit.clients.bazelmoduleregistry.RemoteBazelModuleRegistryService
 import org.ossreviewtoolkit.clients.bazelmoduleregistry.ModuleMetadata
 import org.ossreviewtoolkit.clients.bazelmoduleregistry.ModuleSourceInfo
 import org.ossreviewtoolkit.downloader.VersionControlSystem
@@ -125,7 +125,7 @@ class Bazel(
     }
 
     private fun getPackages(scopes: Set<Scope>, registryUrl: String?): Set<Package> {
-        val registry = BazelModuleRegistryService.create(registryUrl)
+        val registry = RemoteBazelModuleRegistryService.create(registryUrl)
         val ids = scopes.flatMapTo(mutableSetOf()) { it.collectDependencies() }
         val moduleMetadataForId = ids.associateWith { getModuleMetadata(it, registry) }
         val moduleSourceInfoForId = ids.associateWith { getModuleSourceInfo(it, registry) }
@@ -143,14 +143,14 @@ class Bazel(
             vcs = metadata?.vcsInfo().orEmpty()
         )
 
-    private fun getModuleMetadata(id: Identifier, registry: BazelModuleRegistryService): ModuleMetadata? =
+    private fun getModuleMetadata(id: Identifier, registry: RemoteBazelModuleRegistryService): ModuleMetadata? =
         runCatching {
             runBlocking { registry.getModuleMetadata(id.name) }
         }.onFailure {
             logger.warn { "Failed to fetch metadata for Bazel module '${id.name}': ${it.collectMessages()}" }
         }.getOrNull()
 
-    private fun getModuleSourceInfo(id: Identifier, registry: BazelModuleRegistryService): ModuleSourceInfo? =
+    private fun getModuleSourceInfo(id: Identifier, registry: RemoteBazelModuleRegistryService): ModuleSourceInfo? =
         runCatching {
             runBlocking { registry.getModuleSourceInfo(id.name, id.version) }
         }.onFailure {