From 00eb8432cc1c4062b535aada6575a6166e9d4811 Mon Sep 17 00:00:00 2001
From: jmvan <thelys-3cv@axonium.eu>
Date: Sat, 6 Feb 2016 09:35:49 +0100
Subject: [PATCH 1/2] doc: added statement about direct access to _aqlfunctions

---
 .../Books/Users/AqlExtending/Conventions.mdpp          | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/Documentation/Books/Users/AqlExtending/Conventions.mdpp b/Documentation/Books/Users/AqlExtending/Conventions.mdpp
index 60630a38c6..8ee543b0d3 100644
--- a/Documentation/Books/Users/AqlExtending/Conventions.mdpp
+++ b/Documentation/Books/Users/AqlExtending/Conventions.mdpp
@@ -117,7 +117,9 @@ Any violation of the strict mode will trigger a runtime error.
 
 Internally, user functions are stored in a system collection named
 *_aqlfunctions*  of the selected database. 
-That means that by default they are excluded from dumps
-created with [arangodump](../HttpBulkImports/Arangodump.md).
-To include AQL user functions in a dump, the dump should be started
-with the option *--include-system-collections true*.
+Documents in this collection should not be accessed directly, but 
+only via the dedicated interfaces.
+Also system collection are excluded from dumps created with 
+[arangodump](../HttpBulkImports/Arangodump.md). To include AQL user 
+functions in a dump, the dump should be started with the 
+option *--include-system-collections true*.

From 91f5eda895f9dcb9e4d57294f7ef124ac349b4f1 Mon Sep 17 00:00:00 2001
From: Frank Celler <frank@arangodb.com>
Date: Sun, 7 Feb 2016 19:53:36 +0100
Subject: [PATCH 2/2] fixed result

---
 js/client/modules/@arangodb/testing.js | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/js/client/modules/@arangodb/testing.js b/js/client/modules/@arangodb/testing.js
index 5d0f458d21..7a09d8d71f 100644
--- a/js/client/modules/@arangodb/testing.js
+++ b/js/client/modules/@arangodb/testing.js
@@ -2996,6 +2996,7 @@ const recoveryTests = [
 
 testFuncs.recovery = function(options) {
   let results = {};
+  let status = true;
 
   for (let i = 0; i < recoveryTests.length; ++i) {
     let test = recoveryTests[i];
@@ -3010,9 +3011,17 @@ testFuncs.recovery = function(options) {
     }
 
     results[test] = instanceInfo.pid;
+
+    if (!results[test].status) {
+      status = false;
+    }
   }
 
-  return results;
+  results.status = status;
+
+  return {
+    recovery: results
+  };
 };
 
 ////////////////////////////////////////////////////////////////////////////////
@@ -3852,7 +3861,6 @@ function unitTest(which, options) {
       if (thisReply.hasOwnProperty(i)) {
         if (thisReply[i].status !== true) {
           ok = false;
-          allok = false;
         }
       }
     }
@@ -3861,7 +3869,7 @@ function unitTest(which, options) {
     results.status = ok;
     results.crashed = serverCrashed;
 
-    if (allok) {
+    if (ok) {
       cleanupDBDirectories(options);
     } else {
       print("since some tests weren't successfully, not cleaning up: \n" +
@@ -3873,7 +3881,7 @@ function unitTest(which, options) {
     if (jsonReply === true) {
       return results;
     } else {
-      return allok;
+      return ok;
     }
   }