test: fix hmr-ssr

playground/hmr-ssr/__tests__/hmr-ssr.spec.ts

@@ -844,20 +844,23 @@ if (!isBuild) {
     await untilUpdated(() => hmr('.optional-chaining')?.toString(), '2')
   })
 
-  test.skip('hmr works for self-accepted module within circular imported files', async () => {
+  test('hmr works for self-accepted module within circular imported files', async () => {
     await setupModuleRunner('/self-accept-within-circular/index')
     const el = () => hmr('.self-accept-within-circular')
     expect(el()).toBe('c')
     editFile('self-accept-within-circular/c.js', (code) =>
       code.replace(`export const c = 'c'`, `export const c = 'cc'`),
     )
+    // it throws a same error as browser case,
+    // but it doesn't auto reload and it calls `hot.accept` called with `undefined`
+    await untilUpdated(() => el(), '')
+
+    // test reloading manually for now
+    // TODO: why clearCache not working?
+    // runner.clearCache();
+    // runner.import('/self-accept-within-circular/index')
+    await setupModuleRunner('/self-accept-within-circular/index')
     await untilUpdated(() => el(), 'cc')
-    await vi.waitFor(() => {
-      expect(serverLogs.length).greaterThanOrEqual(1)
-      // Should still keep hmr update, but it'll error on the browser-side and will refresh itself.
-      // Match on full log not possible because of color markers
-      expect(serverLogs.at(-1)!).toContain('hmr update')
-    })
   })
 
   test('hmr should not reload if no accepted within circular imported files', async (ctx) => {

playground/hmr-ssr/self-accept-within-circular/c.js

@@ -8,5 +8,5 @@ function render(content) {
 render(c)
 
 import.meta.hot?.accept((nextExports) => {
-  render(nextExports.c)
+  render(nextExports?.c)
 })