fix(runtime-utils): devServer served html content didn't change after…

… repacking. (#5187)

.changeset/honest-cherries-compare.md

@@ -0,0 +1,6 @@
+---
+'@modern-js/runtime-utils': patch
+---
+
+fix(runtime-utils): FileReader#reset didn't clear storage as expected
+fix(runtime-utils): FileReader#reset 没有按预期清除缓存的问题。

packages/toolkit/runtime-utils/src/node/storer/storage.ts

@@ -63,7 +63,7 @@ export class Storage<V = unknown> {
     const keys = await this.keys?.();
     await Promise.all(
       keys?.map(async key => {
-        return this.delete(key);
+        return this.container.delete(key);
       }) || [],
     );
   }

packages/toolkit/runtime-utils/tests/fixtures/fileReader/.gitignore

@@ -0,0 +1 @@
+*.tmp

packages/toolkit/runtime-utils/tests/node/fileReader.test.ts

@@ -1,4 +1,5 @@
 import path from 'path';
+import * as FSModule from 'node:fs/promises';
 import { fileReader } from '../../src/node/fileReader';
 
 test('should fileReader work correctly', async () => {
@@ -16,3 +17,32 @@ test('should fileReader work correctly', async () => {
   expect(res1).not.toBeNull();
   expect(res).toMatch('modern');
 });
+
+describe('FileReader#cache', () => {
+  const tmpFile = path.resolve(__dirname, '../fixtures/fileReader/x.tmp');
+  const writeTmpFile = (content: string) =>
+    FSModule.writeFile(tmpFile, content);
+
+  const writeReadTmpFile = (content: string) =>
+    writeTmpFile(content).then(() => fileReader.readFile(tmpFile));
+
+  beforeEach(async () => {
+    await fileReader.reset();
+    // touch
+    await writeTmpFile('');
+  });
+  afterEach(async () => {
+    await FSModule.rm(tmpFile);
+  });
+
+  it('should read cached content first', async () => {
+    await expect(writeReadTmpFile('foo')).resolves.toBe('foo');
+    await expect(writeReadTmpFile('bar')).resolves.toBe('foo');
+  });
+
+  it('should read file again after FileReader reset', async () => {
+    await expect(writeReadTmpFile('foo2')).resolves.toBe('foo2');
+    await fileReader.reset();
+    await expect(writeReadTmpFile('bar2')).resolves.toBe('bar2');
+  });
+});