added ArangoCollection.drop(isSystem)

Author: Mark

ChangeLog

@@ -4,6 +4,7 @@ v4.1.8 (2017-02-xx)
 * changed java.sql.Date serialization from VPack.date to VPack.string (ISO 8601)
 * changed java.sql.Timestamp serialization from VPack.date to VPack.string (ISO 8601)
 * added byte[] de-/serialization from/to VPack.string (Base64)
+* added ArangoCollection.drop(isSystem)
 
 v4.1.7 (2017-01-26)
 ---------------------------

src/main/java/com/arangodb/ArangoCollection.java

@@ -658,7 +658,21 @@ public CollectionPropertiesEntity count() throws ArangoDBException {
 	 * @throws ArangoDBException
 	 */
 	public void drop() throws ArangoDBException {
-		executor.execute(dropRequest(), Void.class);
+		executor.execute(dropRequest(null), Void.class);
+	}
+
+	/**
+	 * Drops the collection
+	 * 
+	 * @see <a href="https://docs.arangodb.com/current/HTTP/Collection/Creating.html#drops-collection">API
+	 *      Documentation</a>
+	 * @param isSystem
+	 *            Whether or not the collection to drop is a system collection. This parameter must be set to true in
+	 *            order to drop a system collection.
+	 * @throws ArangoDBException
+	 */
+	public void drop(final boolean isSystem) throws ArangoDBException {
+		executor.execute(dropRequest(isSystem), Void.class);
 	}
 
 	/**

src/main/java/com/arangodb/internal/ArangoDBConstants.java

@@ -109,5 +109,6 @@ public class ArangoDBConstants {
 	public static final String COMPLETE = "complete";
 	public static final String DETAILS = "details";
 	public static final String TYPE = "type";
+	public static final String IS_SYSTEM = "isSystem";
 
 }

src/main/java/com/arangodb/internal/InternalArangoCollection.java

@@ -535,8 +535,9 @@ protected Request createFulltextIndexRequest(final Collection<String> fields, fi
 		return request;
 	}
 
-	protected Request dropRequest() {
-		return new Request(db, RequestType.DELETE, executor.createPath(ArangoDBConstants.PATH_API_COLLECTION, name));
+	protected Request dropRequest(final Boolean isSystem) {
+		return new Request(db, RequestType.DELETE, executor.createPath(ArangoDBConstants.PATH_API_COLLECTION, name))
+				.putQueryParam(ArangoDBConstants.IS_SYSTEM, isSystem);
 	}
 
 	protected Request loadRequest() {

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

@@ -104,6 +104,35 @@ public void deleteCollection() {
 		}
 	}
 
+	@Test
+	public void deleteSystemCollection() {
+		final String name = "_system_test";
+		db.createCollection(name, new CollectionCreateOptions().isSystem(true));
+		db.collection(name).drop(true);
+		try {
+			db.collection(name).getInfo();
+			fail();
+		} catch (final ArangoDBException e) {
+		}
+	}
+
+	@Test
+	public void deleteSystemCollectionFail() {
+		final String name = "_system_test";
+		db.createCollection(name, new CollectionCreateOptions().isSystem(true));
+		try {
+			db.collection(name).drop();
+			fail();
+		} catch (final ArangoDBException e) {
+		}
+		db.collection(name).drop(true);
+		try {
+			db.collection(name).getInfo();
+			fail();
+		} catch (final ArangoDBException e) {
+		}
+	}
+
 	@Test
 	public void getIndex() {
 		try {