diff --git a/src/androidTest/java/com/nutomic/syncthingandroid/test/syncthing/PollWebGuiAvailableTaskTest.java b/src/androidTest/java/com/nutomic/syncthingandroid/test/syncthing/PollWebGuiAvailableTaskTest.java index 2473f0cd..2d77e43f 100644 --- a/src/androidTest/java/com/nutomic/syncthingandroid/test/syncthing/PollWebGuiAvailableTaskTest.java +++ b/src/androidTest/java/com/nutomic/syncthingandroid/test/syncthing/PollWebGuiAvailableTaskTest.java @@ -43,6 +43,6 @@ public class PollWebGuiAvailableTaskTest extends AndroidTestCase { }.execute(mConfig.getWebGuiUrl()); latch.await(1, TimeUnit.SECONDS); - // TODO? SyncthingRunnable.killSyncthing(); + // TODO: Unit tests fail when Syncthing is killed SyncthingRunnable.killSyncthing(); } } diff --git a/src/androidTest/java/com/nutomic/syncthingandroid/test/syncthing/RestApiTest.java b/src/androidTest/java/com/nutomic/syncthingandroid/test/syncthing/RestApiTest.java index 9cf345bf..39860cbc 100644 --- a/src/androidTest/java/com/nutomic/syncthingandroid/test/syncthing/RestApiTest.java +++ b/src/androidTest/java/com/nutomic/syncthingandroid/test/syncthing/RestApiTest.java @@ -39,7 +39,7 @@ public class RestApiTest extends AndroidTestCase { } }.execute(config.getWebGuiUrl()); mApi = new RestApi(getContext(), config.getWebGuiUrl(), config.getApiKey(), - null, null, // TODO + null, null, new RestApi.OnApiAvailableListener() { @Override public void onApiAvailable() { @@ -52,7 +52,7 @@ public class RestApiTest extends AndroidTestCase { @Override protected void tearDown() throws Exception { super.tearDown(); - // TODO? SyncthingRunnable.killSyncthing(); + // TODO: Unit tests fail when Syncthing is killed SyncthingRunnable.killSyncthing(); ConfigXml.getConfigFile(new MockContext(getContext())).delete(); } diff --git a/src/main/java/com/nutomic/syncthingandroid/syncthing/RestApi.java b/src/main/java/com/nutomic/syncthingandroid/syncthing/RestApi.java index 0826c364..f380e180 100644 --- a/src/main/java/com/nutomic/syncthingandroid/syncthing/RestApi.java +++ b/src/main/java/com/nutomic/syncthingandroid/syncthing/RestApi.java @@ -356,7 +356,7 @@ public class RestApi implements SyncthingService.OnWebGuiAvailableListener, new PostTask(mHttpsCertPath) .execute(mUrl, PostTask.URI_CONFIG, mApiKey, mConfig.toString()); } - // TODO Should wait for completion... + // TODO Should wait until PostTask is completed, see #398 if (mRestartPostponed) return;