OAK-12113: MongoDocumentStore: BsonException should be wrapped as DocumentStoreException

Author: reschke

oak-store-document/src/main/java/org/apache/jackrabbit/oak/plugins/document/mongo/MongoDocumentStore.java

@@ -1675,7 +1675,7 @@ private <T extends Document> BulkRequestResult sendBulkRequest(final Collection<
                     updateOps.size(), bsonException.getMessage(), idOfbiggestUpdate, estimatedSizeOfBiggestUpdate,
                     bsonException);
             // rethrow
-            throw bsonException;
+            throw new DocumentStoreException(bsonException);
         }
 
         for (BulkWriteUpsert upsert : bulkResult.getUpserts()) {

oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/mongo/MongoDBExceptionTest.java

@@ -201,8 +201,11 @@ public void createOrUpdate16MBBatchWithMultiDocs() {
             try {
                 store.createOrUpdate(Collection.NODES, updateOps);
                 fail("createOrUpdate(many with one >16MB) should have failed");
-            } catch (BSONException expected) {
-                // currently expected but incorrect -> OAK-12113
+            } catch (DocumentStoreException expected) {
+                assertTrue("DocumentStoreException should wrap BSONException, but got: "+
+                        expected.getCause(),
+                        expected.getCause() instanceof BSONException);
+
                 List<String> messages = log.getLogs();
                 assertEquals("only 1 message expected, but got: " + messages.size(),
                         1, messages.size());