fix: override empty file

Fixes #844
Belphemur · Jan 23, 2025 · 024169f · 024169f
1 parent 25b1064
commit 024169f
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/src/adapter/data/JsonAdapter.ts b/src/adapter/data/JsonAdapter.ts
@@ -25,7 +25,7 @@ export class JsonAdapter implements IAdapter<any> {
 
     async readAsync(): Promise<any> {
         const data = await this.adapter.readAsync();
-        if (data == null) {
+        if (data == null || data === '') {
             await this.writeAsync({});
             return {};
         }

diff --git a/test/adapter/adapters.test.ts b/test/adapter/adapters.test.ts
@@ -1,5 +1,6 @@
 import {FileAdapter} from "../../src/adapter/file/FileAdapter";
 import * as fs from "fs";
+import * as fsPromise from 'node:fs/promises'
 import {JsonAdapter} from "../../src/adapter/data/JsonAdapter";
 import {IAdapter} from "../../src/adapter/IAdapter";
 import {ConfigWithAdapter} from "../../src/lib/JsonDBConfig";
@@ -106,6 +107,18 @@ describe('Adapter', () => {
 
             })
 
+            test('should override empty file when loading', async () => {
+                const filename = "data/emptyFile.json";
+                let fh = await fsPromise.open(filename, 'a');
+                await fh.close();
+                const adapter = new JsonAdapter(new FileAdapter(filename, false), false);
+                await adapter.readAsync();
+
+                const fileExists = await checkFileExists(filename);
+                expect(fileExists).toBeTruthy();
+                await fsPromise.rm(filename);
+            })
+
             test('should serialize and deserialize dates', async () => {
                 const adapter = new JsonAdapter(new MemoryAdapter(), false);
                 const data = {