Adding factories

google · Sep 15, 2023 · 8942479 · 8942479
1 parent 11d3bb3
commit 8942479
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 7 deletions.
diff --git a/engine/src/main/java/com/google/android/fhir/sync/upload/patch/PatchGenerator.kt b/engine/src/main/java/com/google/android/fhir/sync/upload/patch/PatchGenerator.kt
@@ -34,13 +34,14 @@ internal interface PatchGenerator {
    * certain numbers of [Patch]es.
    */
   fun generate(localChanges: List<LocalChange>): List<Patch>
+}
 
-  companion object {
-    internal fun byMode(mode: PatchGeneratorMode): PatchGenerator {
-      return when (mode) {
-        is PatchGeneratorMode.PerChange -> PerChangePatchGenerator
-        is PatchGeneratorMode.PerResource -> PerResourcePatchGenerator
-      }
+internal object PatchGeneratorFactory {
+  fun byMode(
+    mode: PatchGeneratorMode,
+  ): PatchGenerator =
+    when (mode) {
+      is PatchGeneratorMode.PerChange -> PerChangePatchGenerator
+      is PatchGeneratorMode.PerResource -> PerResourcePatchGenerator
     }
-  }
 }
diff --git a/engine/src/main/java/com/google/android/fhir/sync/upload/patch/PatchGeneratorMode.kt b/engine/src/main/java/com/google/android/fhir/sync/upload/patch/PatchGeneratorMode.kt
@@ -16,6 +16,9 @@
 
 package com.google.android.fhir.sync.upload.patch
 
+/**
+ * Mode to decide the type of [PatchGenerator] that needs to be used to upload the [LocalChange]s
+ */
 internal sealed class PatchGeneratorMode {
   object PerResource : PatchGeneratorMode()