Tunnel: Require passing a state to the constructor

Signed-off-by: Samuel Holland <samuel@sholland.org>

1 files changed, 5 insertions, 4 deletions

diff --git a/app/src/main/java/com/wireguard/android/model/TunnelManager.java b/app/src/main/java/com/wireguard/android/model/TunnelManager.java
index 53322ab4..eaf8c0d6 100644
--- a/app/src/main/java/com/wireguard/android/model/TunnelManager.java
+++ b/app/src/main/java/com/wireguard/android/model/TunnelManager.java
@@ -48,14 +48,14 @@ public final class TunnelManager {
         this.preferences = preferences;
     }
 
-    private Tunnel add(final String name, final Config config) {
-        final Tunnel tunnel = new Tunnel(backend, configStore, name, config);
+    private Tunnel add(final String name, final Config config, final State state) {
+        final Tunnel tunnel = new Tunnel(backend, configStore, name, config, state);
         tunnels.add(tunnel);
         return tunnel;
     }
 
     private Tunnel add(final String name) {
-        return add(name, null);
+        return add(name, null, State.UNKNOWN);
     }
 
     public CompletionStage<Tunnel> create(final String name, final Config config) {
@@ -66,7 +66,8 @@ public final class TunnelManager {
             final String message = "Tunnel " + name + " already exists";
             return CompletableFuture.failedFuture(new IllegalArgumentException(message));
         }
-        return configStore.create(name, config).thenApply(savedConfig -> add(name, savedConfig));
+        return configStore.create(name, config)
+                .thenApply(savedConfig -> add(name, savedConfig, State.DOWN));
     }
 
     public CompletionStage<Void> delete(final Tunnel tunnel) {