add test for multiple deletes at once

Author: Jake Champion

integration-tests/js-compute/fixtures/app/src/kv-store.js

@@ -617,10 +617,7 @@ import { routes } from "./routes.js";
         });
         routes.set("/kv-store/delete/key-does-not-exist-returns-undefined", async () => {
             let store = createValidStore()
-            let result = store.delete(Math.random())
-            let error = assert(result instanceof Promise, true, `store.delete(Math.random()) instanceof Promise`)
-            if (error) { return error }
-            error = assert(await result, undefined, `await store.delete(Math.random())`)
+            let error = await assertRejects(() => store.delete(Math.random()), "KVStore.prototype.delete: can not delete key which does not exist")
             if (error) { return error }
             return pass()
         });
@@ -636,6 +633,45 @@ import { routes } from "./routes.js";
             if (error) { return error }
             return pass()
         });
+        routes.set("/kv-store/delete/delete-key-twice", async () => {
+            let store = createValidStore()
+            let key = `key-exists-${Math.random()}`;
+            await store.put(key, 'hello')
+            let result = store.delete(key)
+            let error = assert(result instanceof Promise, true, `store.delete(key) instanceof Promise`)
+            if (error) { return error }
+            result = await result
+            error = assert(result, undefined, `(await store.delete(key) === undefined)`)
+            if (error) { return error }
+            error = await assertRejects(() => store.delete(key), "KVStore.prototype.delete: can not delete key which does not exist")
+            if (error) { return error }
+            return pass()
+        });
+        routes.set("/kv-store/delete/multiple-deletes-at-once", async () => {
+            let store = createValidStore()
+            let key1 = `key-exists-${Math.random()}`;
+            await store.put(key1, '1hello1')
+            let key2 = `key-exists-${Math.random()}`;
+            await store.put(key2, '2hello2')
+            let key3 = `key-exists-${Math.random()}`;
+            await store.put(key3, '3hello3')
+            let key4 = `key-exists-${Math.random()}`;
+            await store.put(key4, '4hello4')
+            let key5 = `key-exists-${Math.random()}`;
+            await store.put(key5, '5hello5')
+            let error = await assertResolves(() => {
+                return Promise.all([
+                    store.delete(key1),
+                    store.delete(key2),
+                    store.delete(key3),
+                    store.delete(key4),
+                    store.delete(key5),
+                ]);
+            });
+
+            if (error) { return error }
+            return pass()
+        });
     }
 
     // KVStore get method

integration-tests/js-compute/fixtures/app/tests.json

@@ -3678,11 +3678,21 @@
       "status": 200
     }
   },
-  "GET /kv-store/delete/multiple-lookups-at-once": {
+  "GET /kv-store/delete/delete-key-twice": {
     "environments": ["compute", "viceroy"],
     "downstream_request": {
       "method": "GET",
-      "pathname": "/kv-store/delete/multiple-lookups-at-once"
+      "pathname": "/kv-store/delete/delete-key-twice"
+    },
+    "downstream_response": {
+      "status": 200
+    }
+  },
+  "GET /kv-store/delete/multiple-deletes-at-once": {
+    "environments": ["compute", "viceroy"],
+    "downstream_request": {
+      "method": "GET",
+      "pathname": "/kv-store/delete/multiple-deletes-at-once"
     },
     "downstream_response": {
       "status": 200