fix(setItems): call driver native setItems only to avoid duplicate write (#392)

Author: juliusmarminge

src/storage.ts

@@ -231,7 +231,7 @@ export function createStorage<T extends StorageValue>(
     async setItems(items, commonOptions) {
       await runBatch(items, commonOptions, async (batch) => {
         if (batch.driver.setItems) {
-          await asyncCall(
+          return asyncCall(
             batch.driver.setItems,
             batch.items.map((item) => ({
               key: item.relativeKey,

test/storage.test.ts

@@ -160,3 +160,33 @@ describe("utils", () => {
     expect(async () => await storage.setItem("foo", [])).not.toThrow();
   });
 });
+
+describe("Regression", () => {
+  it("setItems doeesn't upload twice", async () => {
+    /**
+     * https://github.com/unjs/unstorage/pull/392
+     */
+
+    const setItem = vi.fn();
+    const setItems = vi.fn();
+
+    const driver = memory();
+    const storage = createStorage({
+      driver: {
+        ...driver,
+        setItem: (...args) => {
+          setItem(...args);
+          return driver.setItem?.(...args);
+        },
+        setItems: (...args) => {
+          setItems(...args);
+          return driver.setItems?.(...args);
+        },
+      },
+    });
+
+    await storage.setItems([{ key: "foo.txt", value: "bar" }]);
+    expect(setItem).toHaveBeenCalledTimes(0);
+    expect(setItems).toHaveBeenCalledTimes(1);
+  });
+});