diff options
| author | Bartek IwaĆczuk <biwanczuk@gmail.com> | 2023-03-08 07:44:54 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-03-08 12:44:54 +0100 |
| commit | 72fe9bb47005e720444e65a66e91559287137780 (patch) | |
| tree | d529ee7f3b2c33063ba4496f8fd3dfdbac0d1355 /cli/tools | |
| parent | d24c6ea27f7dea57e3bc0cda342d6cbe59782f7d (diff) | |
refactor: rename InternalModuleLoader to ExtModuleLoader, use ext: scheme for snapshotted modules (#18041)
This commit renames "deno_core::InternalModuleLoader" to
"ExtModuleLoader" and changes the specifiers used by the
modules loaded from this loader to "ext:".
"internal:" scheme was really ambiguous and it's more characters than
"ext:", which should result in slightly smaller snapshot size.
Closes https://github.com/denoland/deno/issues/18020
Diffstat (limited to 'cli/tools')
| -rw-r--r-- | cli/tools/coverage/mod.rs | 2 | ||||
| -rw-r--r-- | cli/tools/test.rs | 6 |
2 files changed, 3 insertions, 5 deletions
diff --git a/cli/tools/coverage/mod.rs b/cli/tools/coverage/mod.rs index 1bd2779bf..c7a4d58da 100644 --- a/cli/tools/coverage/mod.rs +++ b/cli/tools/coverage/mod.rs @@ -594,7 +594,7 @@ fn filter_coverages( coverages .into_iter() .filter(|e| { - let is_internal = e.url.starts_with("internal:") + let is_internal = e.url.starts_with("ext:") || e.url.ends_with("__anonymous__") || e.url.ends_with("$deno$test.js") || e.url.ends_with(".snap"); diff --git a/cli/tools/test.rs b/cli/tools/test.rs index 6468da249..9ca806fb0 100644 --- a/cli/tools/test.rs +++ b/cli/tools/test.rs @@ -655,8 +655,7 @@ fn abbreviate_test_error(js_error: &JsError) -> JsError { // check if there are any stack frames coming from user code let should_filter = frames.iter().any(|f| { if let Some(file_name) = &f.file_name { - !(file_name.starts_with("[internal:") - || file_name.starts_with("internal:")) + !(file_name.starts_with("[ext:") || file_name.starts_with("ext:")) } else { true } @@ -668,8 +667,7 @@ fn abbreviate_test_error(js_error: &JsError) -> JsError { .rev() .skip_while(|f| { if let Some(file_name) = &f.file_name { - file_name.starts_with("[internal:") - || file_name.starts_with("internal:") + file_name.starts_with("[ext:") || file_name.starts_with("ext:") } else { false } |
