@webda/core 2.0.0-beta.3 → 2.1.0

package/lib/services/binary.spec.js

@@ -1,34 +1,21 @@
-"use strict";
 var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
     var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
     if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
     else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
     return c > 3 && r && Object.defineProperty(target, key, r), r;
 };
-var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
-    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
-    return new (P || (P = Promise))(function (resolve, reject) {
-        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
-        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
-        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
-        step((generator = generator.apply(thisArg, _arguments || [])).next());
-    });
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.BinaryTest = exports.ImageUser = void 0;
-const mocha_1 = require("@testdeck/mocha");
-const assert = require("assert");
-const axios_1 = require("axios");
-const events_1 = require("events");
-const fs = require("fs");
-const sinon = require("sinon");
-const __1 = require("..");
-const test_1 = require("../test");
-const binary_1 = require("./binary");
-class ImageUser extends __1.User {
+import { suite, test } from "@testdeck/mocha";
+import * as assert from "assert";
+import axios from "axios";
+import { EventEmitter } from "events";
+import * as fs from "fs";
+import * as sinon from "sinon";
+import { Binary, User } from "../index.js";
+import { WebdaTest } from "../test.js";
+import { BinaryNotFoundError, LocalBinaryFile, MemoryBinaryFile } from "./binary.js";
+export class ImageUser extends User {
 }
-exports.ImageUser = ImageUser;
-class TestBinaryService extends __1.Binary {
+class TestBinaryService extends Binary {
     store(object, property, file, metadatas, index) {
         throw new Error("Method not implemented.");
     }
@@ -47,11 +34,9 @@ class TestBinaryService extends __1.Binary {
     _get() {
         return null;
     }
-    cascadeDelete() {
-        return __awaiter(this, void 0, void 0, function* () { });
-    }
+    async cascadeDelete() { }
 }
-class BinaryTest extends test_1.WebdaTest {
+class BinaryTest extends WebdaTest {
     getUserStore() {
         return this.getService("Users");
     }
@@ -67,202 +52,183 @@ class BinaryTest extends test_1.WebdaTest {
     getTestFile() {
         return process.cwd() + "/test/Dockerfile.txt";
     }
-    before(init = true) {
-        const _super = Object.create(null, {
-            before: { get: () => super.before }
-        });
-        return __awaiter(this, void 0, void 0, function* () {
-            yield _super.before.call(this, init);
-            let userStore = this.getUserStore();
-            let binary = this.getBinary();
-            assert.notStrictEqual(userStore, undefined);
-            assert.notStrictEqual(binary, undefined);
-            yield userStore.__clean();
-            yield binary.__clean();
-        });
+    async before(init = true) {
+        await super.before(init);
+        let userStore = this.getUserStore();
+        let binary = this.getBinary();
+        assert.notStrictEqual(userStore, undefined);
+        assert.notStrictEqual(binary, undefined);
+        await userStore.__clean();
+        await binary.__clean();
     }
-    normal() {
-        return __awaiter(this, void 0, void 0, function* () {
-            let userStore = this.getUserStore();
-            let binary = this.getBinary();
-            let map = this.getMap();
-            let exposePath = this.getExposePath();
-            var eventFired = 0;
-            var events = ["Binary.Get", "Binary.Create", "Binary.Delete"];
-            for (let evt in events) {
-                binary.on(events[evt], function () {
-                    eventFired++;
-                });
-            }
-            // Check CREATE - READ
-            var hash;
-            var user1;
-            var user2;
-            var user;
-            var ctx;
-            this.log("DEBUG", "Initializing users");
-            user1 = yield userStore.save({
-                test: "plop"
-            });
-            user2 = yield userStore.save({
-                test: "plop"
-            });
-            this.log("DEBUG", "Store a local file");
-            yield binary.store(user1, map, new binary_1.LocalBinaryFile(this.getTestFile()), {});
-            user1 = yield userStore.get(user1.uuid);
-            assert.notStrictEqual(user1[map], undefined);
-            assert.strictEqual(user1[map].length, 1);
-            hash = user1[map][0].hash;
-            this.log("DEBUG", "Get usage count");
-            let value = yield binary.getUsageCount(hash);
-            assert.strictEqual(value, 1);
-            this.log("DEBUG", "Store same local file on separate object");
-            yield binary.store(user2, map, new binary_1.LocalBinaryFile(this.getTestFile()), {});
-            user = yield userStore.get(user2.uuid);
-            assert.notStrictEqual(user[map], undefined);
-            assert.strictEqual(user[map].length, 1);
-            assert.strictEqual(user[map][0].constructor.name, "BinaryMap");
-            hash = user[map][0].hash;
-            this.log("DEBUG", "Verify usage count");
-            value = yield binary.getUsageCount(hash);
-            assert.strictEqual(value, 2);
-            this.log("DEBUG", "Delete one attached object");
-            yield binary.delete(user, map, 0);
-            user = yield userStore.get(user2.uuid);
-            assert.strictEqual(user[map].length, 0);
-            this.log("DEBUG", "Verify usage count decreased");
-            value = yield binary.getUsageCount(hash);
-            assert.strictEqual(value, 1);
-            // Try to get images on user1 as user2
-            ctx = yield this.newContext({
-                type: "CRUD",
-                uuid: "PLOP"
+    async normal() {
+        let userStore = this.getUserStore();
+        let binary = this.getBinary();
+        let map = this.getMap();
+        let exposePath = this.getExposePath();
+        var eventFired = 0;
+        var events = ["Binary.Get", "Binary.Create", "Binary.Delete"];
+        for (let evt in events) {
+            binary.on(events[evt], function () {
+                eventFired++;
             });
-            ctx.session.userId = user2.uuid;
-            let executor = this.getExecutor(ctx, "test.webda.io", "GET", exposePath + "/users/" + user1.uuid + "/" + map + "/0");
-            this.log("DEBUG", "Verify permission check on HTTP GET");
-            yield assert.rejects(executor.execute(ctx), res => res == 403);
-            ctx.session.userId = user1.uuid;
-            this.log("DEBUG", "Verify valid permission check on HTTP GET");
-            executor = this.getExecutor(ctx, "test.webda.io", "GET", exposePath + "/users/" + user1.uuid + "/" + map + "/0");
-            yield executor.execute(ctx);
-            // We dont check for result as FileBinary will return datas and S3 a redirect
-            if (fs.existsSync("./downloadTo.tmp")) {
-                fs.unlinkSync("./downloadTo.tmp");
-            }
-            this.log("DEBUG", "BinaryMap downloadTo");
-            yield binary.downloadTo(user1[map][0], "./downloadTo.tmp");
-            // Check the result is the same
-            assert.strictEqual(fs.readFileSync("./downloadTo.tmp").toString(), fs.readFileSync(this.getTestFile()).toString());
-            fs.unlinkSync("./downloadTo.tmp");
-            this.log("DEBUG", "BinaryMap downloadTo from CoreModel");
-            yield user1[map][0].downloadTo("./downloadTo.tmp");
-            assert.strictEqual(fs.readFileSync("./downloadTo.tmp").toString(), fs.readFileSync(this.getTestFile()).toString());
+        }
+        // Check CREATE - READ
+        var hash;
+        var user1;
+        var user2;
+        var user;
+        var ctx;
+        this.log("DEBUG", "Initializing users");
+        user1 = await userStore.save({
+            test: "plop"
+        });
+        user2 = await userStore.save({
+            test: "plop"
+        });
+        this.log("DEBUG", "Store a local file");
+        await binary.store(user1, map, new LocalBinaryFile(this.getTestFile()), {});
+        user1 = await userStore.get(user1.uuid);
+        assert.notStrictEqual(user1[map], undefined);
+        assert.strictEqual(user1[map].length, 1);
+        hash = user1[map][0].hash;
+        this.log("DEBUG", "Get usage count");
+        let value = await binary.getUsageCount(hash);
+        assert.strictEqual(value, 1);
+        this.log("DEBUG", "Store same local file on separate object");
+        await binary.store(user2, map, new LocalBinaryFile(this.getTestFile()), {});
+        user = await userStore.get(user2.uuid);
+        assert.notStrictEqual(user[map], undefined);
+        assert.strictEqual(user[map].length, 1);
+        assert.strictEqual(user[map][0].constructor.name, "BinaryMap");
+        hash = user[map][0].hash;
+        this.log("DEBUG", "Verify usage count");
+        value = await binary.getUsageCount(hash);
+        assert.strictEqual(value, 2);
+        this.log("DEBUG", "Delete one attached object");
+        await binary.delete(user, map, 0);
+        user = await userStore.get(user2.uuid);
+        assert.strictEqual(user[map].length, 0);
+        this.log("DEBUG", "Verify usage count decreased");
+        value = await binary.getUsageCount(hash);
+        assert.strictEqual(value, 1);
+        // Try to get images on user1 as user2
+        ctx = await this.newContext({
+            type: "CRUD",
+            uuid: "PLOP"
+        });
+        ctx.session.userId = user2.uuid;
+        let executor = this.getExecutor(ctx, "test.webda.io", "GET", exposePath + "/users/" + user1.uuid + "/" + map + "/0");
+        this.log("DEBUG", "Verify permission check on HTTP GET");
+        await assert.rejects(executor.execute(ctx), res => res == 403);
+        ctx.session.userId = user1.uuid;
+        this.log("DEBUG", "Verify valid permission check on HTTP GET");
+        executor = this.getExecutor(ctx, "test.webda.io", "GET", exposePath + "/users/" + user1.uuid + "/" + map + "/0");
+        await executor.execute(ctx);
+        // We dont check for result as FileBinary will return datas and S3 a redirect
+        if (fs.existsSync("./downloadTo.tmp")) {
             fs.unlinkSync("./downloadTo.tmp");
-            // Fake IO issue
-            let stub = sinon.stub(binary, "_get").callsFake(() => {
-                return {
-                    pipe: stream => {
-                        stream.emit("error", "bad read");
-                    }
-                };
-            });
-            let stub2;
-            try {
-                this.log("DEBUG", "BinaryMap downloadTo with I/O issues");
-                yield assert.rejects(() => binary.downloadTo(user1[map][0], "./downloadTo.tmp"));
-                assert.ok(!fs.existsSync("./downloadTo.tmp"));
-                stub2 = sinon.stub(fs, "unlinkSync").callsFake(() => {
-                    throw new Error();
-                });
-                yield assert.rejects(() => binary.downloadTo(user1[map][0], "./downloadTo.tmp"));
-            }
-            finally {
-                stub.restore();
-                if (stub2) {
-                    stub2.restore();
+        }
+        this.log("DEBUG", "BinaryMap downloadTo");
+        await binary.downloadTo(user1[map][0], "./downloadTo.tmp");
+        // Check the result is the same
+        assert.strictEqual(fs.readFileSync("./downloadTo.tmp").toString(), fs.readFileSync(this.getTestFile()).toString());
+        fs.unlinkSync("./downloadTo.tmp");
+        this.log("DEBUG", "BinaryMap downloadTo from CoreModel");
+        await user1[map][0].downloadTo("./downloadTo.tmp");
+        assert.strictEqual(fs.readFileSync("./downloadTo.tmp").toString(), fs.readFileSync(this.getTestFile()).toString());
+        fs.unlinkSync("./downloadTo.tmp");
+        // Fake IO issue
+        let stub = sinon.stub(binary, "_get").callsFake(() => {
+            return {
+                pipe: stream => {
+                    stream.emit("error", "bad read");
                 }
-            }
-            this.log("DEBUG", "Check get stream");
-            let buf1 = yield __1.Binary.streamToBuffer(yield user1[map][0].get());
-            // cov
-            let buf2 = yield __1.Binary.streamToBuffer(yield binary._get(user1[map][0]));
-            assert.strictEqual(buf1.toString(), buf2.toString());
-            this.log("DEBUG", "Delete CoreModel and ensure usage count");
-            yield userStore.delete(user1.uuid);
-            value = yield binary.getUsageCount(hash);
-            assert.strictEqual(value, 0);
+            };
         });
+        let stub2;
+        try {
+            this.log("DEBUG", "BinaryMap downloadTo with I/O issues");
+            await assert.rejects(() => binary.downloadTo(user1[map][0], "./downloadTo.tmp"));
+            assert.ok(!fs.existsSync("./downloadTo.tmp"));
+        }
+        finally {
+            stub.restore();
+            if (stub2) {
+                stub2.restore();
+            }
+        }
+        this.log("DEBUG", "Check get stream");
+        let buf1 = await Binary.streamToBuffer(await user1[map][0].get());
+        // cov
+        let buf2 = await Binary.streamToBuffer(await binary._get(user1[map][0]));
+        assert.strictEqual(buf1.toString(), buf2.toString());
+        this.log("DEBUG", "Delete CoreModel and ensure usage count");
+        await userStore.delete(user1.uuid);
+        value = await binary.getUsageCount(hash);
+        assert.strictEqual(value, 0);
     }
-    notMapped() {
-        return __awaiter(this, void 0, void 0, function* () {
-            let userStore = this.getUserStore();
-            let binary = this.getBinary();
-            let user1 = yield userStore.save({
-                test: "plop"
-            });
-            yield assert.rejects(() => binary.store(user1, "images2", new binary_1.LocalBinaryFile(this.getTestFile()), {}), err => true);
+    async notMapped() {
+        let userStore = this.getUserStore();
+        let binary = this.getBinary();
+        let user1 = await userStore.save({
+            test: "plop"
         });
+        await assert.rejects(() => binary.store(user1, "images2", new LocalBinaryFile(this.getTestFile()), {}), err => true);
     }
-    update() {
-        return __awaiter(this, void 0, void 0, function* () {
-            let userStore = this.getUserStore();
-            let binary = this.getBinary();
-            let map = this.getMap();
-            var user1;
-            var user;
-            let hash;
-            user1 = yield userStore.save({
-                test: "plop"
-            });
-            yield binary.store(user1, map, new binary_1.LocalBinaryFile(this.getTestFile()), {});
-            user = yield userStore.get(user1.uuid);
-            assert.notStrictEqual(user[map], undefined);
-            assert.strictEqual(user[map].length, 1);
-            let value = yield binary.getUsageCount(user[map][0].hash);
-            assert.strictEqual(value, 1);
+    async update() {
+        let userStore = this.getUserStore();
+        let binary = this.getBinary();
+        let map = this.getMap();
+        var user1;
+        var user;
+        let hash;
+        user1 = await userStore.save({
+            test: "plop"
         });
+        await binary.store(user1, map, new LocalBinaryFile(this.getTestFile()), {});
+        user = await userStore.get(user1.uuid);
+        assert.notStrictEqual(user[map], undefined);
+        assert.strictEqual(user[map].length, 1);
+        let value = await binary.getUsageCount(user[map][0].hash);
+        assert.strictEqual(value, 1);
     }
     checkMap() {
         let binary = this.getBinary();
         // @ts-ignore
         assert.throws(() => binary.checkMap("plop", "pouf"), /Unknown mapping/);
     }
-    httpCreate() {
-        return __awaiter(this, void 0, void 0, function* () {
-            let userStore = this.getUserStore();
-            let user1 = yield userStore.save({
-                test: "plop"
-            });
-            let ctx = yield this.newContext();
-            let executor = this.getExecutor(ctx, "test.webda.io", "POST", `/binary/users/${user1.getUuid()}/images`, {});
-            yield assert.rejects(() => executor.execute(ctx), /403/);
-            /**
-             * Two modes should be handled
-             *
-             * POST body + X-Webda-Metadatas header
-             * POST multipart
-             */
-            ctx.getHttpContext().setBody("MY_IMAGE");
-            ctx.getSession().login(user1.getUuid(), "fake");
-            // Need to verify
-            yield executor.execute(ctx);
+    async httpCreate() {
+        let userStore = this.getUserStore();
+        let user1 = await userStore.save({
+            test: "plop"
         });
+        let ctx = await this.newContext();
+        let executor = this.getExecutor(ctx, "test.webda.io", "POST", `/binary/users/${user1.getUuid()}/images`, {});
+        await assert.rejects(() => executor.execute(ctx), /403/);
+        /**
+         * Two modes should be handled
+         *
+         * POST body + X-Webda-Metadatas header
+         * POST multipart
+         */
+        ctx.getHttpContext().setBody("MY_IMAGE");
+        ctx.getSession().login(user1.getUuid(), "fake");
+        // Need to verify
+        await executor.execute(ctx);
     }
-    httpDelete() {
-        return __awaiter(this, void 0, void 0, function* () {
-            let { user1, ctx } = yield this.setupDefault(false);
-            let executor = this.getExecutor(ctx, "test.webda.io", "DELETE", `/binary/users/${user1.getUuid()}/images/0/myhash`, {});
-            yield assert.rejects(() => executor.execute(ctx), /403/, "DELETE binary w/o permission");
-            // Now login
-            ctx.getSession().login(user1.getUuid(), "fake");
-            executor = this.getExecutor(ctx, "test.webda.io", "DELETE", `/binary/users/${user1.getUuid()}/images/0/myhash`, {});
-            yield assert.rejects(() => executor.execute(ctx), /412/);
-            executor = this.getExecutor(ctx, "test.webda.io", "DELETE", `/binary/users/${user1.getUuid()}/images/0/${user1.images[0].hash}`, {});
-            yield executor.execute(ctx);
-            yield user1.refresh();
-            assert.ok(user1.images.length === 0);
-        });
+    async httpDelete() {
+        let { user1, ctx } = await this.setupDefault(false);
+        let executor = this.getExecutor(ctx, "test.webda.io", "DELETE", `/binary/users/${user1.getUuid()}/images/0/myhash`, {});
+        await assert.rejects(() => executor.execute(ctx), /403/, "DELETE binary w/o permission");
+        // Now login
+        ctx.getSession().login(user1.getUuid(), "fake");
+        executor = this.getExecutor(ctx, "test.webda.io", "DELETE", `/binary/users/${user1.getUuid()}/images/0/myhash`, {});
+        await assert.rejects(() => executor.execute(ctx), /412/);
+        executor = this.getExecutor(ctx, "test.webda.io", "DELETE", `/binary/users/${user1.getUuid()}/images/0/${user1.images[0].hash}`, {});
+        await executor.execute(ctx);
+        await user1.refresh();
+        assert.ok(user1.images.length === 0);
     }
     getOperationName() {
         let binary = this.getBinary();
@@ -273,288 +239,269 @@ class BinaryTest extends test_1.WebdaTest {
         // @ts-ignore
         assert.strictEqual(binary.getOperationName(), binary.getName());
     }
-    httpGetError() {
-        return __awaiter(this, void 0, void 0, function* () {
-            let { binary, user1, ctx } = yield this.setupDefault();
-            // @ts-ignore
-            let stub = sinon.stub(binary, "get").callsFake(() => {
-                return {
-                    pipe: stream => {
-                        stream.emit("error", new Error("I/O"));
-                    }
-                };
-            });
-            yield this.execute(ctx, "test.webda.io", "GET", `/binary/users/${user1.getUuid()}/images/0`, {});
-            yield assert.rejects(() => this.execute(ctx, "test.webda.io", "GET", ctx.getResponseHeaders().Location.substring("http://test.webda.io".length)), /500/, "GET binary with I/O");
-            stub.restore();
+    async httpGetError() {
+        let { binary, user1, ctx } = await this.setupDefault();
+        // @ts-ignore
+        let stub = sinon.stub(binary, "get").callsFake(() => {
+            return {
+                pipe: stream => {
+                    stream.emit("error", new Error("I/O"));
+                }
+            };
         });
+        await this.execute(ctx, "test.webda.io", "GET", `/binary/users/${user1.getUuid()}/images/0`, {});
+        await assert.rejects(() => this.execute(ctx, "test.webda.io", "GET", ctx.getResponseHeaders().Location.substring("http://test.webda.io".length)), /500/, "GET binary with I/O");
+        stub.restore();
     }
-    httpGet() {
-        return __awaiter(this, void 0, void 0, function* () {
-            let { binary, user1, ctx } = yield this.setupDefault(false);
-            let executor = this.getExecutor(ctx, "test.webda.io", "GET", `/binary/users/${user1.getUuid()}/images/0`, {});
-            // If not logged in
-            yield assert.rejects(() => executor.execute(ctx), /403/, "GET binary w/o permission");
-            executor = this.getExecutor(ctx, "test.webda.io", "GET", `/binary/users/unknown/images/0`, {});
-            yield assert.rejects(() => executor.execute(ctx), /404/, "GET binary on unknown object");
-            // Login
-            ctx.getSession().login(user1.getUuid(), "fake");
-            executor = this.getExecutor(ctx, "test.webda.io", "GET", `/binary/users/${user1.getUuid()}/images/0`, {});
-            yield executor.execute(ctx);
-            yield binary.delete(user1, "images", 0);
-            // Check 404 now it is gone
-            executor = this.getExecutor(ctx, "test.webda.io", "GET", `/binary/users/${user1.getUuid()}/images/0`, {});
-            yield assert.rejects(() => executor.execute(ctx), /404/);
-        });
+    async httpGet() {
+        let { binary, user1, ctx } = await this.setupDefault(false);
+        let executor = this.getExecutor(ctx, "test.webda.io", "GET", `/binary/users/${user1.getUuid()}/images/0`, {});
+        // If not logged in
+        await assert.rejects(() => executor.execute(ctx), /403/, "GET binary w/o permission");
+        executor = this.getExecutor(ctx, "test.webda.io", "GET", `/binary/users/unknown/images/0`, {});
+        await assert.rejects(() => executor.execute(ctx), /404/, "GET binary on unknown object");
+        // Login
+        ctx.getSession().login(user1.getUuid(), "fake");
+        executor = this.getExecutor(ctx, "test.webda.io", "GET", `/binary/users/${user1.getUuid()}/images/0`, {});
+        await executor.execute(ctx);
+        await binary.delete(user1, "images", 0);
+        // Check 404 now it is gone
+        executor = this.getExecutor(ctx, "test.webda.io", "GET", `/binary/users/${user1.getUuid()}/images/0`, {});
+        await assert.rejects(() => executor.execute(ctx), /404/);
     }
-    httpMetadata() {
+    async httpMetadata() {
         var _a;
-        var _b;
-        return __awaiter(this, void 0, void 0, function* () {
-            let { binary, user1, ctx } = yield this.setupDefault(false);
-            let executor = this.getExecutor(ctx, "test.webda.io", "PUT", `/binary/users/${user1.getUuid()}/images/0/${user1.images[0].hash}`, {}); // If not logged in
-            yield assert.rejects(() => executor.execute(ctx), /403/, "PUT metadata w/o permission");
-            executor = this.getExecutor(ctx, "test.webda.io", "PUT", `/binary/users/unknown/images/0/${user1.images[0].hash}`, {});
-            yield assert.rejects(() => executor.execute(ctx), /404/, "GET binary on unknown object");
-            // Login
-            ctx.getSession().login(user1.getUuid(), "fake");
-            executor = this.getExecutor(ctx, "test.webda.io", "PUT", `/binary/users/${user1.getUuid()}/images/0/${user1.images[0].hash}`, {
-                "my-metadata": "updated",
-                "my-other-metadata": true
-            });
-            yield executor.execute(ctx);
-            let user = yield ctx.getCurrentUser(true);
-            (_a = (_b = user.images[0]).metadata) !== null && _a !== void 0 ? _a : (_b.metadata = {});
-            assert.strictEqual(user.images[0].metadata["my-metadata"], "updated");
-            assert.strictEqual(user.images[0].metadata["my-other-metadata"], true);
-            executor = this.getExecutor(ctx, "test.webda.io", "PUT", `/binary/users/${user1.getUuid()}/images/0/${user1.images[0].hash}`, {
-                "my-metadata": "updated".repeat(4096),
-                "my-other-metadata": true
-            });
-            yield assert.rejects(() => executor.execute(ctx), /400/, "PUT metadata with big content should fail");
+        let { binary, user1, ctx } = await this.setupDefault(false);
+        let executor = this.getExecutor(ctx, "test.webda.io", "PUT", `/binary/users/${user1.getUuid()}/images/0/${user1.images[0].hash}`, {}); // If not logged in
+        await assert.rejects(() => executor.execute(ctx), /403/, "PUT metadata w/o permission");
+        executor = this.getExecutor(ctx, "test.webda.io", "PUT", `/binary/users/unknown/images/0/${user1.images[0].hash}`, {});
+        await assert.rejects(() => executor.execute(ctx), /404/, "GET binary on unknown object");
+        // Login
+        ctx.getSession().login(user1.getUuid(), "fake");
+        executor = this.getExecutor(ctx, "test.webda.io", "PUT", `/binary/users/${user1.getUuid()}/images/0/${user1.images[0].hash}`, {
+            "my-metadata": "updated",
+            "my-other-metadata": true
         });
-    }
-    getNotFound() {
-        return __awaiter(this, void 0, void 0, function* () {
-            yield assert.rejects(() => 
-            // @ts-ignore
-            this.getBinary().get({
-                hash: "none",
-                mimetype: "",
-                size: 10
-            }), binary_1.BinaryNotFoundError);
+        await executor.execute(ctx);
+        let user = await ctx.getCurrentUser(true);
+        (_a = user.images[0]).metadata ?? (_a.metadata = {});
+        assert.strictEqual(user.images[0].metadata["my-metadata"], "updated");
+        assert.strictEqual(user.images[0].metadata["my-other-metadata"], true);
+        executor = this.getExecutor(ctx, "test.webda.io", "PUT", `/binary/users/${user1.getUuid()}/images/0/${user1.images[0].hash}`, {
+            "my-metadata": "updated".repeat(4096),
+            "my-other-metadata": true
         });
+        await assert.rejects(() => executor.execute(ctx), /400/, "PUT metadata with big content should fail");
     }
-    setupDefault(withLogin = true) {
-        return __awaiter(this, void 0, void 0, function* () {
-            let userStore = this.getUserStore();
-            let binary = this.getBinary();
-            let user1 = yield userStore.save({
-                test: "plop"
-            });
-            yield binary.store(user1, "images", new binary_1.LocalBinaryFile(this.getTestFile()));
-            yield user1.refresh();
-            let ctx = yield this.newContext();
-            if (withLogin) {
-                ctx.getSession().login(user1.getUuid(), "fake");
-            }
-            user1.setContext(ctx);
-            return { userStore, binary, user1, ctx };
-        });
+    async getNotFound() {
+        await assert.rejects(() => 
+        // @ts-ignore
+        this.getBinary().get({
+            hash: "none",
+            mimetype: "",
+            size: 10
+        }), BinaryNotFoundError);
     }
-    testChallenge(remoteCheckHash = true) {
-        return __awaiter(this, void 0, void 0, function* () {
-            let { userStore, binary, user1, ctx } = yield this.setupDefault(false);
-            let { hash, challenge } = yield new binary_1.MemoryBinaryFile(Buffer.from("PLOP"), {}).getHashes();
-            let metadata = {
-                plop: "test"
-            };
-            let executor = this.getExecutor(ctx, "test.webda.io", "PUT", `/binary/upload/users/${user1.getUuid()}/images`, {
-                hash,
-                challenge,
-                metadata
-            });
-            yield assert.rejects(() => executor.execute(ctx), /403/);
+    async setupDefault(withLogin = true) {
+        let userStore = this.getUserStore();
+        let binary = this.getBinary();
+        let user1 = await userStore.save({
+            test: "plop"
+        });
+        await binary.store(user1, "images", new LocalBinaryFile(this.getTestFile()));
+        await user1.refresh();
+        let ctx = await this.newContext();
+        if (withLogin) {
             ctx.getSession().login(user1.getUuid(), "fake");
-            yield executor.execute(ctx);
-            let info = JSON.parse(ctx.getResponseBody());
-            assert.ok(!info.done, "should not be done yet");
-            // Execute twice as we should still get an upload
-            yield executor.execute(ctx);
-            let url = new URL(info.url);
-            if (url.host === "test.webda.io") {
-                executor = this.getExecutor(ctx, "test.webda.io", info.method || "PUT", url.pathname + url.search, "PLOP2", {
-                    "Content-MD5": info.md5,
-                    "Content-Type": "application/octet-stream"
-                });
-                yield assert.rejects(() => executor.execute(ctx));
-                executor = this.getExecutor(ctx, "test.webda.io", info.method || "PUT", url.pathname + url.search, "PLOP", {
-                    "Content-MD5": info.md5,
-                    "Content-Type": "application/octet-stream"
-                });
-                yield executor.execute(ctx);
-            }
-            else {
-                if (remoteCheckHash) {
-                    yield assert.rejects(() => this.sendChallengeData(info, "PLOP2"), "Should check content");
-                }
-                yield this.sendChallengeData(info, "PLOP");
-            }
-            yield user1.refresh();
-            assert.strictEqual(user1.images.length, 2);
-            assert.strictEqual((yield __1.Binary.streamToBuffer(yield binary.get(user1.images[1]))).toString(), "PLOP");
-            // If we try to re upload it should be already up
-            executor = this.getExecutor(ctx, "test.webda.io", "PUT", `/binary/upload/users/${user1.getUuid()}/images`, {
-                hash,
-                challenge,
-                metadata
+        }
+        user1.setContext(ctx);
+        return { userStore, binary, user1, ctx };
+    }
+    async testChallenge(remoteCheckHash = true) {
+        let { userStore, binary, user1, ctx } = await this.setupDefault(false);
+        let { hash, challenge } = await new MemoryBinaryFile(Buffer.from("PLOP"), {}).getHashes();
+        let metadata = {
+            plop: "test"
+        };
+        let executor = this.getExecutor(ctx, "test.webda.io", "PUT", `/binary/upload/users/${user1.getUuid()}/images`, {
+            hash,
+            challenge,
+            metadata
+        });
+        await assert.rejects(() => executor.execute(ctx), /403/);
+        ctx.getSession().login(user1.getUuid(), "fake");
+        await executor.execute(ctx);
+        let info = JSON.parse(ctx.getResponseBody());
+        assert.ok(!info.done, "should not be done yet");
+        // Execute twice as we should still get an upload
+        await executor.execute(ctx);
+        let url = new URL(info.url);
+        if (url.host === "test.webda.io") {
+            executor = this.getExecutor(ctx, "test.webda.io", info.method || "PUT", url.pathname + url.search, "PLOP2", {
+                "Content-MD5": info.md5,
+                "Content-Type": "application/octet-stream"
             });
-            yield executor.execute(ctx);
-            info = JSON.parse(ctx.getResponseBody());
-            assert.ok(info.done, "should be done");
-            assert.ok(info.url === undefined);
-            // Use another user to try adding same binary
-            let user2 = yield userStore.save({});
-            ctx.getSession().login(user2.getUuid(), "fake");
-            executor = this.getExecutor(ctx, "test.webda.io", "PUT", `/binary/upload/users/${user2.getUuid()}/images`, {
-                hash,
-                challenge,
-                metadata
+            await assert.rejects(() => executor.execute(ctx));
+            executor = this.getExecutor(ctx, "test.webda.io", info.method || "PUT", url.pathname + url.search, "PLOP", {
+                "Content-MD5": info.md5,
+                "Content-Type": "application/octet-stream"
             });
-            yield executor.execute(ctx);
-            info = JSON.parse(ctx.getResponseBody());
-            assert.ok(info.done, "should be done");
-            assert.ok(info.url === undefined);
+            await executor.execute(ctx);
+        }
+        else {
+            if (remoteCheckHash) {
+                await assert.rejects(() => this.sendChallengeData(info, "PLOP2"), "Should check content");
+            }
+            await this.sendChallengeData(info, "PLOP");
+        }
+        await user1.refresh();
+        assert.strictEqual(user1.images.length, 2);
+        assert.strictEqual((await Binary.streamToBuffer(await binary.get(user1.images[1]))).toString(), "PLOP");
+        // If we try to re upload it should be already up
+        executor = this.getExecutor(ctx, "test.webda.io", "PUT", `/binary/upload/users/${user1.getUuid()}/images`, {
+            hash,
+            challenge,
+            metadata
         });
+        await executor.execute(ctx);
+        info = JSON.parse(ctx.getResponseBody());
+        assert.ok(info.done, "should be done");
+        assert.ok(info.url === undefined);
+        // Use another user to try adding same binary
+        let user2 = await userStore.save({});
+        ctx.getSession().login(user2.getUuid(), "fake");
+        executor = this.getExecutor(ctx, "test.webda.io", "PUT", `/binary/upload/users/${user2.getUuid()}/images`, {
+            hash,
+            challenge,
+            metadata
+        });
+        await executor.execute(ctx);
+        info = JSON.parse(ctx.getResponseBody());
+        assert.ok(info.done, "should be done");
+        assert.ok(info.url === undefined);
     }
-    sendChallengeData(info, data) {
-        return __awaiter(this, void 0, void 0, function* () {
-            return axios_1.default.request({
-                method: info.method,
-                url: info.url,
-                data,
-                headers: {
-                    "Content-MD5": info.md5,
-                    "Content-Type": "application/octet-stream"
-                }
-            });
+    async sendChallengeData(info, data) {
+        return axios.request({
+            method: info.method,
+            url: info.url,
+            data,
+            headers: {
+                "Content-MD5": info.md5,
+                "Content-Type": "application/octet-stream"
+            }
         });
     }
 }
 __decorate([
-    mocha_1.test
+    test
 ], BinaryTest.prototype, "normal", null);
 __decorate([
-    mocha_1.test
+    test
 ], BinaryTest.prototype, "notMapped", null);
 __decorate([
-    mocha_1.test
+    test
 ], BinaryTest.prototype, "update", null);
 __decorate([
-    mocha_1.test
+    test
 ], BinaryTest.prototype, "checkMap", null);
 __decorate([
-    mocha_1.test
+    test
 ], BinaryTest.prototype, "httpCreate", null);
 __decorate([
-    mocha_1.test
+    test
 ], BinaryTest.prototype, "httpDelete", null);
 __decorate([
-    mocha_1.test
+    test
 ], BinaryTest.prototype, "getOperationName", null);
 __decorate([
-    mocha_1.test
+    test
 ], BinaryTest.prototype, "httpGetError", null);
 __decorate([
-    mocha_1.test
+    test
 ], BinaryTest.prototype, "httpGet", null);
 __decorate([
-    mocha_1.test
+    test
 ], BinaryTest.prototype, "httpMetadata", null);
 __decorate([
-    mocha_1.test
+    test
 ], BinaryTest.prototype, "getNotFound", null);
-exports.BinaryTest = BinaryTest;
-let BinaryAbstractTest = class BinaryAbstractTest extends test_1.WebdaTest {
-    cov() {
-        return __awaiter(this, void 0, void 0, function* () {
-            let service = new TestBinaryService(undefined, "plop", {});
-            let ctx = yield this.newContext();
-            yield assert.rejects(() => service.putRedirectUrl(undefined), /404/);
-            ctx.getHttpContext().setBody({
-                hash: "plop"
-            });
-            yield assert.rejects(() => service.httpChallenge(ctx), /400/);
-            ctx = yield this.newContext();
-            ctx.getHttpContext().setBody({
-                challenge: "plop"
-            });
-            yield assert.rejects(() => service.httpChallenge(ctx), /400/);
-            ctx = yield this.newContext();
-            let binary = this.getService("binary");
-            ctx.setPathParameters({
-                property: "images",
-                store: "users",
-                uid: "notfound",
-                index: 0
-            });
-            ctx.getHttpContext().setBody({
-                hash: "p",
-                challenge: "z"
-            });
-            yield assert.rejects(() => binary.httpChallenge(ctx), /404/);
-            let stubs = [];
-            try {
-                let model = {};
-                stubs.push(sinon.stub(binary, "_verifyMapAndStore").callsFake(() => {
-                    return {
-                        get: (uuid, ctx) => __awaiter(this, void 0, void 0, function* () {
-                            return model;
-                        })
-                    };
-                }));
-                yield assert.rejects(() => binary.httpRoute(ctx), /404/);
-                model = {
-                    images: [
-                        {
-                            size: 10
-                        }
-                    ],
-                    canAct: () => __awaiter(this, void 0, void 0, function* () { return true; })
-                };
-                stubs.push(
-                // @ts-ignore
-                sinon.stub(binary, "get").callsFake(() => __awaiter(this, void 0, void 0, function* () {
-                    return {
-                        pipe: stream => {
-                            stream.emit("error", new Error("I/O"));
-                        }
-                    };
-                })));
-                yield assert.rejects(() => binary.httpRoute(ctx), /500/);
-            }
-            finally {
-                stubs.forEach(stub => stub.restore());
-            }
+let BinaryAbstractTest = class BinaryAbstractTest extends WebdaTest {
+    async cov() {
+        let service = new TestBinaryService(undefined, "plop", {});
+        let ctx = await this.newContext();
+        await assert.rejects(() => service.putRedirectUrl(undefined), /404/);
+        ctx.getHttpContext().setBody({
+            hash: "plop"
         });
-    }
-    streamToBufferError() {
-        return __awaiter(this, void 0, void 0, function* () {
-            let stream = new events_1.EventEmitter();
-            // @ts-ignore
-            let p = assert.rejects(() => __1.Binary.streamToBuffer(stream), /Bad I\/O/);
-            stream.emit("error", new Error("Bad I/O"));
-            yield p;
+        await assert.rejects(() => service.httpChallenge(ctx), /400/);
+        ctx = await this.newContext();
+        ctx.getHttpContext().setBody({
+            challenge: "plop"
+        });
+        await assert.rejects(() => service.httpChallenge(ctx), /400/);
+        ctx = await this.newContext();
+        let binary = this.getService("binary");
+        ctx.setPathParameters({
+            property: "images",
+            store: "users",
+            uid: "notfound",
+            index: 0
         });
+        ctx.getHttpContext().setBody({
+            hash: "p",
+            challenge: "z"
+        });
+        await assert.rejects(() => binary.httpChallenge(ctx), /404/);
+        let stubs = [];
+        try {
+            let model = {};
+            stubs.push(sinon.stub(binary, "_verifyMapAndStore").callsFake(() => {
+                return {
+                    get: async (uuid, ctx) => {
+                        return model;
+                    }
+                };
+            }));
+            await assert.rejects(() => binary.httpRoute(ctx), /404/);
+            model = {
+                images: [
+                    {
+                        size: 10
+                    }
+                ],
+                canAct: async () => true
+            };
+            stubs.push(
+            // @ts-ignore
+            sinon.stub(binary, "get").callsFake(async () => {
+                return {
+                    pipe: stream => {
+                        stream.emit("error", new Error("I/O"));
+                    }
+                };
+            }));
+            await assert.rejects(() => binary.httpRoute(ctx), /500/);
+        }
+        finally {
+            stubs.forEach(stub => stub.restore());
+        }
+    }
+    async streamToBufferError() {
+        let stream = new EventEmitter();
+        // @ts-ignore
+        let p = assert.rejects(() => Binary.streamToBuffer(stream), /Bad I\/O/);
+        stream.emit("error", new Error("Bad I/O"));
+        await p;
     }
 };
 __decorate([
-    mocha_1.test
+    test
 ], BinaryAbstractTest.prototype, "cov", null);
 __decorate([
-    mocha_1.test
+    test
 ], BinaryAbstractTest.prototype, "streamToBufferError", null);
 BinaryAbstractTest = __decorate([
-    mocha_1.suite
+    suite
 ], BinaryAbstractTest);
+export { BinaryTest };
 //# sourceMappingURL=binary.spec.js.map