From bc15a1f1a754912cd64d3526c1f3dd84cf3264d2 Mon Sep 17 00:00:00 2001
From: tyiuhc <137842098+tyiuhc@users.noreply.github.com>
Date: Fri, 20 Oct 2023 18:07:46 -0700
Subject: [PATCH] fix: start() always sets user (#41)

Co-authored-by: Tim Yiu <tim.yiu@Tims-MacBook-Pro.local>
---
 .../java/com/amplitude/experiment/DefaultExperimentClient.kt    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sdk/src/main/java/com/amplitude/experiment/DefaultExperimentClient.kt b/sdk/src/main/java/com/amplitude/experiment/DefaultExperimentClient.kt
index 8529d3a..339c66d 100644
--- a/sdk/src/main/java/com/amplitude/experiment/DefaultExperimentClient.kt
+++ b/sdk/src/main/java/com/amplitude/experiment/DefaultExperimentClient.kt
@@ -117,6 +117,7 @@ internal class DefaultExperimentClient internal constructor(
 
     override fun start(user: ExperimentUser?): Future<ExperimentClient> {
         synchronized(isRunningLock) {
+            this.user = user
             if (isRunning) {
                 val future = AsyncFuture<ExperimentClient>()
                 future.complete(this)
@@ -128,7 +129,6 @@ internal class DefaultExperimentClient internal constructor(
                 this.poller.start()
             }
         }
-        this.user = user
         return this.executorService.submit(
             Callable {
                 val flagsFuture = doFlags()