Merge branch '86-sync-is-available' into sync

Author: greenrobot-team

objectbox-java/src/main/java/io/objectbox/sync/Sync.java

@@ -12,6 +12,13 @@
 @Experimental
 public final class Sync {
 
+    /**
+     * Returns true if the included native (JNI) ObjectBox library supports sync.
+     */
+    public static boolean isAvailable() {
+        return BoxStore.isSyncAvailable();
+    }
+
     /**
      * Start building a sync client. Requires the BoxStore that should be synced with the server,
      * the URL and port of the server to connect to and credentials to authenticate against the server.

tests/objectbox-java-test/src/test/java/io/objectbox/BoxStoreTest.java

@@ -240,14 +240,4 @@ public void testIsObjectBrowserAvailable() {
         assertFalse(BoxStore.isObjectBrowserAvailable());
     }
 
-    @Test
-    public void testIsSyncAvailable() {
-        // The individual values don't matter; basically just ensure the methods are available and don't crash...
-        if(BoxStore.isSyncServerAvailable()) {
-            assertTrue(BoxStore.isSyncAvailable());
-        } else {
-            BoxStore.isSyncAvailable();
-        }
-    }
-
 }

tests/objectbox-java-test/src/test/java/io/objectbox/FunctionalTestSuite.java

@@ -29,6 +29,7 @@
 import io.objectbox.relation.ToOneTest;
 import io.objectbox.sync.ConnectivityMonitorTest;
 import io.objectbox.sync.PlatformTest;
+import io.objectbox.sync.SyncTest;
 
 import org.junit.runner.RunWith;
 import org.junit.runners.Suite;
@@ -55,6 +56,7 @@
         QueryTest.class,
         RelationTest.class,
         RelationEagerTest.class,
+        SyncTest.class,
         ToManyStandaloneTest.class,
         ToManyTest.class,
         ToOneTest.class,

tests/objectbox-java-test/src/test/java/io/objectbox/sync/SyncTest.java

@@ -0,0 +1,43 @@
+package io.objectbox.sync;
+
+import org.junit.Test;
+
+import io.objectbox.AbstractObjectBoxTest;
+import io.objectbox.BoxStore;
+
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertThrows;
+
+public class SyncTest extends AbstractObjectBoxTest {
+
+    /**
+     * Ensure that non-sync native library correctly reports sync client availability.
+     *
+     * Note: this test is mirrored in objectbox-integration-test sync tests, where sync is available.
+     */
+    @Test
+    public void clientIsNotAvailable() {
+        assertFalse(Sync.isAvailable());
+    }
+
+    /**
+     * Ensure that non-sync native library correctly reports sync server availability.
+     *
+     * Note: this test is mirrored in objectbox-integration-test sync tests, where sync is available.
+     */
+    @Test
+    public void serverIsNotAvailable() {
+        assertFalse(BoxStore.isSyncServerAvailable());
+    }
+
+    @Test
+    public void creatingSyncClient_throws() {
+        IllegalStateException exception = assertThrows(
+                IllegalStateException.class,
+                () -> Sync.client(store, "wss://127.0.0.1", SyncCredentials.none())
+        );
+        assertEquals("This ObjectBox library (JNI) does not include sync. Please update your dependencies.", exception.getMessage());
+    }
+}