rm ArangoDatabase.getDocument()

Author: rashtao

core/src/main/java/com/arangodb/ArangoDatabase.java

@@ -579,29 +579,6 @@ public interface ArangoDatabase extends ArangoSerdeAccessor {
      */
     DatabaseEntity getInfo();
 
-    /**
-     * Reads a single document
-     *
-     * @param id   The id of the document
-     * @param type The type of the document (POJO or {@link com.arangodb.util.RawData})
-     * @return the document identified by the id
-     * @see <a href="https://www.arangodb.com/docs/stable/http/document-working-with-documents.html#read-document">API
-     * Documentation</a>
-     */
-    <T> T getDocument(String id, Class<T> type);
-
-    /**
-     * Reads a single document
-     *
-     * @param id      The id of the document
-     * @param type    The type of the document (POJO or {@link com.arangodb.util.RawData})
-     * @param options Additional options, can be null
-     * @return the document identified by the id
-     * @see <a href="https://www.arangodb.com/docs/stable/http/document-working-with-documents.html#read-document">API
-     * Documentation</a>
-     */
-    <T> T getDocument(String id, Class<T> type, DocumentReadOptions options);
-
     /**
      * Reload the routing table.
      *

core/src/main/java/com/arangodb/internal/ArangoDatabaseImpl.java

@@ -344,20 +344,6 @@ public DatabaseEntity getInfo() {
         return executor.execute(getInfoRequest(), getInfoResponseDeserializer());
     }
 
-    @Override
-    public <T> T getDocument(final String id, final Class<T> type) {
-        DocumentUtil.validateDocumentId(id);
-        final String[] split = id.split("/");
-        return collection(split[0]).getDocument(split[1], type);
-    }
-
-    @Override
-    public <T> T getDocument(final String id, final Class<T> type, final DocumentReadOptions options) {
-        DocumentUtil.validateDocumentId(id);
-        final String[] split = id.split("/");
-        return collection(split[0]).getDocument(split[1], type, options);
-    }
-
     @Override
     public void reloadRouting() {
         executor.execute(reloadRoutingRequest(), Void.class);

driver/src/test/java/com/arangodb/ArangoDatabaseTest.java

@@ -1363,17 +1363,6 @@ void getInfo(ArangoDatabase db) {
         }
     }
 
-    @ParameterizedTest(name = "{index}")
-    @MethodSource("dbs")
-    void getDocument(ArangoDatabase db) {
-        String key = "key-" + rnd();
-        final BaseDocument value = new BaseDocument(key);
-        db.collection(CNAME1).insertDocument(value);
-        final BaseDocument document = db.getDocument(CNAME1 + "/" + key, BaseDocument.class);
-        assertThat(document).isNotNull();
-        assertThat(document.getKey()).isEqualTo(key);
-    }
-
     @ParameterizedTest(name = "{index}")
     @MethodSource("dbs")
     void shouldIncludeExceptionMessage(ArangoDatabase db) {
@@ -1389,13 +1378,6 @@ void shouldIncludeExceptionMessage(ArangoDatabase db) {
         }
     }
 
-    @ParameterizedTest(name = "{index}")
-    @MethodSource("dbs")
-    void getDocumentWrongId(ArangoDatabase db) {
-        Throwable thrown = catchThrowable(() -> db.getDocument("123", BaseDocument.class));
-        assertThat(thrown).isInstanceOf(ArangoDBException.class);
-    }
-
     @ParameterizedTest(name = "{index}")
     @MethodSource("dbs")
     void reloadRouting(ArangoDatabase db) {