mirror of
https://github.com/nushell/nushell.git
synced 2024-11-28 11:24:09 +01:00
add $.extra_usage
to modules (#11649)
- should fix https://github.com/nushell/nushell/issues/11648 # Description this PR - adds a test that should pass but fails - adds `$.extra_usage` to the output of `scope modules`, fixing both the new test and the linked issue # User-Facing Changes `$.extra_usage` is now a column in the output of `scope modules` # Tests + Formatting a new test case has been added to `correct_scope_modules_fields` # After Submitting
This commit is contained in:
parent
39b020037d
commit
859f7b3dc7
@ -504,10 +504,9 @@ impl<'e, 's> ScopeData<'e, 's> {
|
|||||||
|block_id| Value::block(block_id, span),
|
|block_id| Value::block(block_id, span),
|
||||||
);
|
);
|
||||||
|
|
||||||
let module_usage = self
|
let (module_usage, module_extra_usage) = self
|
||||||
.engine_state
|
.engine_state
|
||||||
.build_module_usage(*module_id)
|
.build_module_usage(*module_id)
|
||||||
.map(|(usage, _)| usage)
|
|
||||||
.unwrap_or_default();
|
.unwrap_or_default();
|
||||||
|
|
||||||
Value::record(
|
Value::record(
|
||||||
@ -520,6 +519,7 @@ impl<'e, 's> ScopeData<'e, 's> {
|
|||||||
"constants" => Value::list(export_consts, span),
|
"constants" => Value::list(export_consts, span),
|
||||||
"env_block" => export_env_block,
|
"env_block" => export_env_block,
|
||||||
"usage" => Value::string(module_usage, span),
|
"usage" => Value::string(module_usage, span),
|
||||||
|
"extra_usage" => Value::string(module_extra_usage, span),
|
||||||
"module_id" => Value::int(*module_id as i64, span),
|
"module_id" => Value::int(*module_id as i64, span),
|
||||||
},
|
},
|
||||||
span,
|
span,
|
||||||
|
@ -91,6 +91,8 @@ fn correctly_report_of_shadowed_alias() {
|
|||||||
fn correct_scope_modules_fields() {
|
fn correct_scope_modules_fields() {
|
||||||
let module_setup = r#"
|
let module_setup = r#"
|
||||||
# nice spam
|
# nice spam
|
||||||
|
#
|
||||||
|
# and some extra usage for spam
|
||||||
|
|
||||||
export module eggs {
|
export module eggs {
|
||||||
export module bacon {
|
export module bacon {
|
||||||
@ -123,6 +125,13 @@ fn correct_scope_modules_fields() {
|
|||||||
let actual = nu!(cwd: dirs.test(), &inp.join("; "));
|
let actual = nu!(cwd: dirs.test(), &inp.join("; "));
|
||||||
assert_eq!(actual.out, "nice spam");
|
assert_eq!(actual.out, "nice spam");
|
||||||
|
|
||||||
|
let inp = &[
|
||||||
|
"use spam.nu",
|
||||||
|
"scope modules | where name == spam | get 0.extra_usage",
|
||||||
|
];
|
||||||
|
let actual = nu!(cwd: dirs.test(), &inp.join("; "));
|
||||||
|
assert_eq!(actual.out, "and some extra usage for spam");
|
||||||
|
|
||||||
let inp = &[
|
let inp = &[
|
||||||
"use spam.nu",
|
"use spam.nu",
|
||||||
"scope modules | where name == spam | get 0.env_block | is-empty",
|
"scope modules | where name == spam | get 0.env_block | is-empty",
|
||||||
|
Loading…
Reference in New Issue
Block a user