Merge pull request #366 from ml054/v5.2

RDBC-693 Node.js client should be able to handle 404 in GetDatabaseRe…

Author: ml054

src/ServerWide/Operations/GetDatabaseRecordOperation.ts

@@ -51,7 +51,7 @@ export class GetDatabaseRecordCommand extends RavenCommand<DatabaseRecordWithEta
 
     public async setResponseAsync(bodyStream: stream.Stream, fromCache: boolean): Promise<string> {
         if (!bodyStream) {
-            this._throwInvalidResponse();
+            return null;
         }
 
         let body: string = null;

test/Ported/Issues/RDBC_693.ts

@@ -0,0 +1,27 @@
+import { testContext, disposeTestDocumentStore } from "../../Utils/TestUtil";
+
+import {
+    IDocumentStore, DocumentChange, GetDatabaseRecordOperation,
+} from "../../../src";
+import { User } from "../../Assets/Entities";
+import { AsyncQueue } from "../../Utils/AsyncQueue";
+import { throwError } from "../../../src/Exceptions";
+import { assertThat, assertThrows } from "../../Utils/AssertExtensions";
+
+describe("RDBC_693", function () {
+
+    let store: IDocumentStore;
+
+    beforeEach(async function () {
+        store = await testContext.getDocumentStore();
+    });
+
+    afterEach(async () =>
+        await disposeTestDocumentStore(store));
+
+    it("can handle 404 in GetDatabaseRecordOperation", async () => {
+        const dbRecord = await store.maintenance.server.send(new GetDatabaseRecordOperation("DB_WHICH_DOESNT_EXIST"));
+        assertThat(dbRecord)
+            .isNull();
+    });
+});