snapshot on function starts to look wonderful

jsenv · Jul 24, 2024 · daa0f24 · daa0f24
1 parent ac8f3fb
commit daa0f24
Show file tree

Hide file tree

Showing 11 changed files with 52 additions and 22 deletions.
diff --git a/package.json b/package.json
@@ -58,7 +58,7 @@
     "@jsenv/package-publish": "1.10.5",
     "@jsenv/performance-impact": "4.2.1",
     "@jsenv/server": "15.2.14",
-    "@jsenv/snapshot": "2.4.1",
+    "@jsenv/snapshot": "2.5.0",
     "@jsenv/test": "3.3.17",
     "eslint": "8.57.0",
     "eslint-plugin-html": "8.1.1",

diff --git a/tests/entry_point_check/auto_mapping_bare_specifier/auto_mapping_bare_specifier.test.mjs b/tests/entry_point_check/auto_mapping_bare_specifier/auto_mapping_bare_specifier.test.mjs
@@ -17,11 +17,11 @@ const test = async (scenario, { bareSpecifierAutomapping }) => {
         },
       }),
     import.meta.url,
-    `./output/${scenario}/`,
+    `./output/${scenario}.md`,
   );
 };
 
-await test("0_default", {
+await test("0_bare_specifier_warning", {
   bareSpecifierAutomapping: undefined,
 });
 await test("1_bare_specifier_automapping", {

diff --git a/...utput/0_default/0_default_side_effects.md → ...cifier/output/0_bare_specifier_warning.md b/...utput/0_default/0_default_side_effects.md → ...cifier/output/0_bare_specifier_warning.md
@@ -26,7 +26,7 @@ add mapping to "manualImportmap"
 
 ```
 
-3. write file "input/0_default.importmap"
+3. write file "input/0_bare_specifier_warning.importmap"
 ```importmap
 {
   "imports": {},
@@ -37,7 +37,7 @@ add mapping to "manualImportmap"
 4. resolve
 ```js
 {
-  "0_default.importmap": {
+  "0_bare_specifier_warning.importmap": {
     "imports": {},
     "scopes": {}
   }

diff --git a/...are_specifier_automapping_side_effects.md → ...er/output/1_bare_specifier_automapping.md b/...are_specifier_automapping_side_effects.md → ...er/output/1_bare_specifier_automapping.md
diff --git a/tests/entry_point_check/auto_mapping_css_to_js/auto_mapping_css_to_js.test.mjs b/tests/entry_point_check/auto_mapping_css_to_js/auto_mapping_css_to_js.test.mjs
@@ -17,7 +17,7 @@ const test = async () => {
         },
       }),
     import.meta.url,
-    `./output/`,
+    `./output/auto_mapping_css_to_js.md`,
   );
 };
 await test();
diff --git a/..._css_to_js/output/output_side_effects.txt → ...ss_to_js/output/auto_mapping_css_to_js.md b/..._css_to_js/output/output_side_effects.txt → ...ss_to_js/output/auto_mapping_css_to_js.md
@@ -1,17 +1,23 @@
-1. write file "input/test.importmap"
+1. return promise
+
+2. write file "input/test.importmap"
+```importmap
 {
   "imports": {
     "./file.css": "./file.css.js"
   },
   "scopes": {}
 }
+```
 
-2. resolve
+3. resolve
+```js
 {
   "test.importmap": {
     "imports": {
       "./file.css": "./file.css.js"
     },
     "scopes": {}
   }
-}
+}
+```
diff --git a/...o_mapping_extensionless_in_node_module/auto_mapping_extensionless_in_node_module.test.mjs b/...o_mapping_extensionless_in_node_module/auto_mapping_extensionless_in_node_module.test.mjs
@@ -17,11 +17,11 @@ const test = async (scenario, { magicExtensions } = {}) => {
         },
       }),
     import.meta.url,
-    `./output/${scenario}/`,
+    `./output/${scenario}.md`,
   );
 };
 
-await test("0_default", {
+await test("0_magic_extensions_warning", {
   magicExtensions: undefined,
 });
 await test("1_magic_extensions_js", {

diff --git a/...tput/0_default/0_default_side_effects.txt → ...dule/output/0_magic_extensions_warning.md b/...tput/0_default/0_default_side_effects.txt → ...dule/output/0_magic_extensions_warning.md
@@ -1,4 +1,7 @@
-1. console.warn
+1. return promise
+
+2. console.warn
+```console
 
 Import resolution failed for "./file"
 --- import trace ---
@@ -21,21 +24,26 @@ add mapping to "manualImportmap"
   }
 }
 
+```
 
-2. write file "input/test.importmap"
+3. write file "input/test.importmap"
+```importmap
 {
   "imports": {
     "leftpad": "./node_modules/leftpad/index.js"
   },
   "scopes": {}
 }
+```
 
-3. resolve
+4. resolve
+```js
 {
   "test.importmap": {
     "imports": {
       "leftpad": "./node_modules/leftpad/index.js"
     },
     "scopes": {}
   }
-}
+}
+```
diff --git a/...js/1_magic_extensions_js_side_effects.txt → ...de_module/output/1_magic_extensions_js.md b/...js/1_magic_extensions_js_side_effects.txt → ...de_module/output/1_magic_extensions_js.md
@@ -1,4 +1,7 @@
-1. console.warn
+1. return promise
+
+2. console.warn
+```console
 
 Import resolution failed for "./other-file"
 --- import trace ---
@@ -8,8 +11,10 @@ file:///cwd()/input/node_modules/leftpad/file.js:1:7
 --- reason ---
 file not found on filesystem at cwd()/input/node_modules/leftpad/other-file
 
+```
 
-2. write file "input/test.importmap"
+3. write file "input/test.importmap"
+```importmap
 {
   "imports": {
     "leftpad": "./node_modules/leftpad/index.js"
@@ -20,8 +25,10 @@ file not found on filesystem at cwd()/input/node_modules/leftpad/other-file
     }
   }
 }
+```
 
-3. resolve
+4. resolve
+```js
 {
   "test.importmap": {
     "imports": {
@@ -33,4 +40,5 @@ file not found on filesystem at cwd()/input/node_modules/leftpad/other-file
       }
     }
   }
-}
+}
+```
diff --git a/...magic_extensions_inherit_side_effects.txt → ...dule/output/2_magic_extensions_inherit.md b/...magic_extensions_inherit_side_effects.txt → ...dule/output/2_magic_extensions_inherit.md
@@ -1,4 +1,7 @@
-1. write file "input/test.importmap"
+1. return promise
+
+2. write file "input/test.importmap"
+```importmap
 {
   "imports": {
     "leftpad": "./node_modules/leftpad/index.js"
@@ -10,8 +13,10 @@
     }
   }
 }
+```
 
-2. resolve
+3. resolve
+```js
 {
   "test.importmap": {
     "imports": {
@@ -24,4 +29,5 @@
       }
     }
   }
-}
+}
+```
diff --git a/tests/snapshot_write_importmaps_side_effects.js b/tests/snapshot_write_importmaps_side_effects.js
@@ -4,10 +4,12 @@ export const snapshotWriteImportsMapsSideEffects = async (
   fn,
   fnFileUrl,
   sideEffectDirectoryRelativeUrl,
+  options,
 ) => {
   await snapshotFunctionSideEffects(
     fn,
     fnFileUrl,
     sideEffectDirectoryRelativeUrl,
+    options,
   );
 };