fix : fix bug delete function not concistence , add test unit

Author: nuzulul

.gitignore

@@ -138,4 +138,5 @@ data/
 docs/
 kvstorage/
 .dev.vars
-.wrangler/
+.wrangler/
+package-backup.json

config/rollup.config.dev.ts

@@ -3,7 +3,6 @@ import typescript from '@rollup/plugin-typescript';
 export default [
 {
   input: 'src/kv-storage.ts',
-  exclude:["test","src","**/*.d.ts"],
   output: [
     {
       file: 'public/umd/kv-storage.umd.js',

config/rollup.config.umd.ts

@@ -3,7 +3,7 @@ import typescript from '@rollup/plugin-typescript';
 export default [
 {
   input: 'src/kv-storage.ts',
-  exclude:["test","src","**/*.d.ts"],
+  //exclude:["test","src","**/*.d.ts"],
   output: [
     {
       file: 'dist/umd/kv-storage.js',

package.json

@@ -13,25 +13,31 @@
   "scripts": {
 		"start": "npm run dev-node",
 
-		"build-dev-cjs": "npm run removedir public && tsc -p tsconfig-devcjs.json",
-		"build-dev-esm": "npm run removedir public && tsc -p tsconfig-devesm.json",
-		"dev-node": "nodemon -e js,ts --watch src --watch test --exec \"npm run build-dev-cjs && node public/test/server.js\"",
-		"dev-ts-node": "nodemon -e js,ts --watch src --watch test --exec \"npm run build-dev-cjs && ts-node public/test/server.js\"",
-		"dev-bun": "nodemon -e js,ts --watch src --watch test --exec \"bun run test/server.ts\"",
+		"build-dev-cjs": "tsc -p tsconfig-devcjs.json",
+		"build-dev-esm": "tsc -p tsconfig-devesm.json",
+		"dev-node": "nodemon -e js,ts --watch src --watch test --exec \"npm run build-dev-cjs && node public/cjs/test/server-node.js\"",
+		"dev-ts-node": "nodemon -e js,ts --watch src --watch test --exec \"npm run build-dev-cjs && ts-node public/cjs/test/server-node.js\"",
+		"dev-bun": "nodemon -e js,ts --watch src --watch test --exec \"bun run test/server-node.ts\"",
 		"dev-deno": "nodemon -e js,ts --watch src --watch test --exec \"deno run --allow-read --allow-write test/test-deno/server-deno.ts\"",
-		"dev-cf": "nodemon -e js,ts,html --watch src --watch test --exec \"npm run build-dev-esm && wrangler dev\"",
+		"dev-cf": "nodemon -e js,ts,html --watch src --watch test --exec \"wrangler dev\"",
 		"dev-browser": "nodemon -e js,ts,html --watch src --watch test --exec \"npm run build-dev-esm && tsc -p tsconfig-umd.json && rollup -c public/config/rollup.config.dev.js && ts-node test/server-browser.ts\"",
 
 		"removedir": "node -e \"var fs = require('fs'); try{process.argv.slice(1).map((fpath) => fs.rmdirSync(fpath, { recursive: true }))}catch(err){console.log(`Dist not found`)}; process.exit(0);\"",
 		"build-all":"tsc -p tsconfig-mjs.json && tsc -p tsconfig-cjs.json && tsc -p tsconfig-umd.json && rollup -c public/config/rollup.config.umd.js && echo {\"type\": \"commonjs\"}>dist\\cjs\\package.json && echo {\"type\": \"module\"}>dist\\mjs\\package.json",
 		"build":"npm run removedir dist && npm run build-all",
+
+		"backup": "node -e \"var fs = require('fs'); try{fs.copyFileSync('package.json', 'package-backup.json')}catch(err){console.log(`Copy failed`)}; process.exit(0);\"",
+		"restore": "node -e \"var fs = require('fs'); try{fs.copyFileSync('package-backup.json', 'package.json')}catch(err){console.log(`Copy failed`)}; process.exit(0);\"",
+		"prepublish":"npm run backup && npm pkg delete devDependencies",
+		"postpublish":"npm run restore",
 
 		"gh-deploy-init": "git push origin && git subtree push --prefix docs origin gh-pages",
 		"gh-deploy": "git push origin :gh-pages && git subtree push --prefix docs origin gh-pages",
 		"typedoc": "npm run removedir docs && typedoc src/kv-storage.ts src/kv-storage-module.ts",
 		"gh-deploydoc": "npm run typedoc && git add docs -f && git commit -m \"docs\" && npm run gh-deploy && git reset --soft HEAD~ && git restore --staged .",
 
-		"test": "npm run dev-node"
+		"test-all": "nodemon -e js,ts --watch src --watch test --exec \"npm run build-dev-cjs && node public/cjs/test/test-all.js\"",
+		"test": "npm run test-all"
   },
   "files": [
     "dist/",

src/kv-storage-module.ts

@@ -87,7 +87,11 @@ export class KVStorageModule{
 			const keyFilePath = this._storageDir+'/'+key
 
 				  fs.unlink(keyFilePath,(err)=>{
-					if (err) {resolve(false)}else{resolve(true)}
+            if (err) {
+              fs.stat(keyFilePath, (err)=>{
+                if (err) {resolve(true)}else{resolve(false)}
+              });
+            }else{resolve(true)}
 				  });
 
 		})

src/kv-storage.ts

@@ -27,11 +27,11 @@ export async function KVStorage({
 		    if(typeof caches === "undefined" && typeof global !== "undefined" && typeof window === "undefined"){
                 let runnode;
                 try {
-                    const url = './kv-storage-module';
+                    const url = './kv-storage-module'
                     runnode = await import(url);
                 }
                 catch {
-                    const url = './kv-storage-module.js';
+                    const url = './kv-storage-module.js'
                     runnode = await import(url);
                 }
 				const dbnode = await runnode.KVStorageModule.init({

test/esm.html

@@ -1,8 +1,9 @@
 <html>
-<body>
-<p>open console</p>
+<body style="margin:0;padding:0">
+<div id="out">open console</div>
 <script type="module">
-import {KVStorage} from './../public/src/kv-storage.js'
+import {KVStorage} from './../public/esm/src/kv-storage.js'
+import {test} from './../public/esm/test/test-unit.js'
 
 void async function main() {
 
@@ -11,13 +12,9 @@
 		storageName:'storage'
 	})
 
-	console.log(await db.put('yes','no'))
-	console.log(await db.get('yes'))
-	console.log(await db.delete('yes'))
-	console.log(await db.get('yes'))
-	console.log(await db.put('yes1','no1'))
-	console.log(await db.list())
-	console.log(await db.has('key'))
+    const data = await test(db)
+    console.log(data)
+    document.getElementById("out").innerHTML = "<textarea style='width:100vw;height:100vh'>"+data+"</textarea>"
 }()
 </script>	
 </body>

test/server-node.ts

@@ -0,0 +1,13 @@
+import {KVStorage} from './../src/kv-storage'
+import {test} from './test-unit'
+
+void async function main() {
+
+	const db = await KVStorage({
+		runtime:'node',
+		storageName:'storage'
+	}) 
+	
+	console.log(await test(db))
+	
+}()

test/server.ts

@@ -0,0 +1,25 @@
+import { spawnSync } from 'child_process'
+
+const node = spawnSync("node", ["public/cjs/test/server-node.js"])
+
+console.log("node = "+node.stdout.toString())
+
+const bun = spawnSync("bun", ["run","test/server-node.ts"])
+
+console.log("bun = "+bun.stdout.toString())
+
+const deno = spawnSync("deno", ["run","--allow-read","--allow-write","test/test-deno/server-deno.ts"])
+
+console.log("deno = "+deno.stdout.toString())
+
+
+
+for(const [runtime,test] of [["node",node.stdout.toString()],["bun",bun.stdout.toString()],["deno",deno.stdout.toString()]]){
+  
+  let passed = "passed all"
+  for(const input of JSON.parse(test)){
+    if(input.passed = false)passed = "failed"
+  }
+  console.log(runtime+" = "+passed)
+  
+}