Teardown test bases that use tmpDir (#48)

* teardown bases that use tmpDir

* teardown order matters

* reenable windows ci

* teardown corestore as well

* fix suspend tests

.github/workflows/ci.yaml

@@ -18,7 +18,7 @@ jobs:
     strategy:
       matrix:
         node-version: [lts/*]
-        os: [ubuntu-latest, macos-latest]
+        os: [ubuntu-latest, macos-latest, windows-latest]
     timeout-minutes: 30
     steps:
       - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1 https://github.com/actions/checkout/releases/tag/v4.1.1

package.json

@@ -52,7 +52,7 @@
     "random-access-memory": "^6.2.0",
     "same-data": "^1.0.0",
     "standard": "^17.0.0",
-    "test-tmp": "^1.0.0"
+    "test-tmp": "^1.2.0"
   },
   "standard": {
     "ignore": [

test/fast-forward.js

@@ -47,6 +47,13 @@ test('fast-forward - simple', async t => {
 
   await b.ready()
 
+  t.teardown(async () => {
+    await a.close()
+    await b.close()
+    await store.close()
+    await store2.close()
+  })
+
   for (let i = 0; i < 1000; i++) {
     await a.append('a' + i)
   }
@@ -72,6 +79,12 @@ test('fast-forward - migrate', async t => {
     storage: () => tmpDir(t)
   })
 
+  t.teardown(async () => {
+    await a.close()
+    await b.close()
+    await c.close()
+  })
+
   for (let i = 0; i < 2000; i++) {
     await a.append('a' + i)
   }
@@ -103,6 +116,12 @@ test('fast-forward - fast forward after migrate', async t => {
     storage: () => tmpDir(t)
   })
 
+  t.teardown(async () => {
+    await a.close()
+    await b.close()
+    await c.close()
+  })
+
   for (let i = 0; i < 2000; i++) {
     await a.append('a' + i)
   }
@@ -156,6 +175,13 @@ test('fast-forward - multiple writers added', async t => {
     storage: () => tmpDir(t)
   })
 
+  t.teardown(async () => {
+    await a.close()
+    await b.close()
+    await c.close()
+    await d.close()
+  })
+
   await addWriterAndSync(a, b)
   await addWriterAndSync(a, c)
 
@@ -208,6 +234,13 @@ test('fast-forward - multiple queues', async t => {
     storage: () => tmpDir(t)
   })
 
+  t.teardown(async () => {
+    await a.close()
+    await b.close()
+    await c.close()
+    await d.close()
+  }, { order: 1 })
+
   // this value should be long enough that 2 fast-forwards are
   // queued (ie. we don't just replicate the last state), but short
   // enough that the second is queued before the first has completed
@@ -348,6 +381,13 @@ test('fast-forward - open with no remote io', async t => {
     ackThreshold: 0
   })
 
+  t.teardown(async () => {
+    await a.close()
+    await b2.close()
+    await store.close()
+    await store2.close()
+  })
+
   await b2.ready()
   await t.execution(b2.ready())
 
@@ -381,6 +421,12 @@ test('fast-forward - force reset then ff', async t => {
     storage: () => tmpDir(t)
   })
 
+  t.teardown(async () => {
+    await a.close()
+    await b.close()
+    await c.close()
+  })
+
   await addWriterAndSync(a, b)
   await addWriterAndSync(a, c)
   await confirm([a, b, c])