extern/nushell
1
0
Fork 1
mirror of https://github.com/nushell/nushell.git synced 2024-11-25 01:43:47 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix std log (#12470)

Browse Source 
related to
- https://github.com/nushell/nushell/pull/12196

# Description
while i'm 100% okey with the original intent behind
https://github.com/nushell/nushell/pull/12196, i think the PR did
introduce two unintended things:
- extra parentheses that make the `log.nu` module look like Lisp lol
- a renaming of the `NU_LOG_LEVEL` environment variable to
`NU_log-level`. this breaks previous usage of `std log` and, as it's not
mentionned at all in the PR, i thought it was not intentional 😋

# User-Facing Changes
users can now control `std log` with `$env.NU_LOG_LEVEL`

# Tests + Formatting
the "log" tests have been fixed as well.

# After Submitting
This commit is contained in:
Antoine Stevan 2024-04-10 23:30:58 +02:00 committed by GitHub
parent 83674909f1
commit 39156930f5
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
5 changed files with 76 additions and 76 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

98
crates/nu-std/std/log.nu
View File

@ -44,34 +44,34 @@ def log-types [] {
( (
{ {
"CRITICAL": { "CRITICAL": {
"ansi": ((log-ansi).CRITICAL), "ansi": (log-ansi).CRITICAL,
"level": ((log-level).CRITICAL), "level": (log-level).CRITICAL,
"prefix": ((log-prefix).CRITICAL), "prefix": (log-prefix).CRITICAL,
"short_prefix": ((log-short-prefix).CRITICAL) "short_prefix": (log-short-prefix).CRITICAL
}, },
"ERROR": { "ERROR": {
"ansi": ((log-ansi).ERROR), "ansi": (log-ansi).ERROR,
"level": ((log-level).ERROR), "level": (log-level).ERROR,
"prefix": ((log-prefix).ERROR), "prefix": (log-prefix).ERROR,
"short_prefix": ((log-short-prefix).ERROR) "short_prefix": (log-short-prefix).ERROR
}, },
"WARNING": { "WARNING": {
"ansi": ((log-ansi).WARNING), "ansi": (log-ansi).WARNING,
"level": ((log-level).WARNING), "level": (log-level).WARNING,
"prefix": ((log-prefix).WARNING), "prefix": (log-prefix).WARNING,
"short_prefix": ((log-short-prefix).WARNING) "short_prefix": (log-short-prefix).WARNING
}, },
"INFO": { "INFO": {
"ansi": ((log-ansi).INFO), "ansi": (log-ansi).INFO,
"level": ((log-level).INFO), "level": (log-level).INFO,
"prefix": ((log-prefix).INFO), "prefix": (log-prefix).INFO,
"short_prefix": ((log-short-prefix).INFO) "short_prefix": (log-short-prefix).INFO
}, },
"DEBUG": { "DEBUG": {
"ansi": ((log-ansi).DEBUG), "ansi": (log-ansi).DEBUG,
"level": ((log-level).DEBUG), "level": (log-level).DEBUG,
"prefix": ((log-prefix).DEBUG), "prefix": (log-prefix).DEBUG,
"short_prefix": ((log-short-prefix).DEBUG) "short_prefix": (log-short-prefix).DEBUG
} }
} }
) )
@ -83,16 +83,16 @@ def parse-string-level [
] { ] {
let level = ($level | str upcase) let level = ($level | str upcase)
if $level in [((log-prefix).CRITICAL) ((log-short-prefix).CRITICAL) "CRIT" "CRITICAL"] { if $level in [(log-prefix).CRITICAL (log-short-prefix).CRITICAL "CRIT" "CRITICAL"] {
((log-level).CRITICAL) (log-level).CRITICAL
} else if $level in [((log-prefix).ERROR) ((log-short-prefix).ERROR) "ERROR"] { } else if $level in [(log-prefix).ERROR (log-short-prefix).ERROR "ERROR"] {
((log-level).ERROR) (log-level).ERROR
} else if $level in [((log-prefix).WARNING) ((log-short-prefix).WARNING) "WARN" "WARNING"] { } else if $level in [(log-prefix).WARNING (log-short-prefix).WARNING "WARN" "WARNING"] {
((log-level).WARNING) (log-level).WARNING
} else if $level in [((log-prefix).DEBUG) ((log-short-prefix).DEBUG) "DEBUG"] { } else if $level in [(log-prefix).DEBUG (log-short-prefix).DEBUG "DEBUG"] {
((log-level).DEBUG) (log-level).DEBUG
} else { } else {
((log-level).INFO) (log-level).INFO
} }
} }
@ -101,41 +101,41 @@ def parse-int-level [
level: int, level: int,
--short (-s) --short (-s)
] { ] {
if $level >= ((log-level).CRITICAL) { if $level >= (log-level).CRITICAL {
if $short { if $short {
((log-short-prefix).CRITICAL) (log-short-prefix).CRITICAL
} else { } else {
((log-prefix).CRITICAL) (log-prefix).CRITICAL
} }
} else if $level >= ((log-level).ERROR) { } else if $level >= (log-level).ERROR {
if $short { if $short {
((log-short-prefix).ERROR) (log-short-prefix).ERROR
} else { } else {
((log-prefix).ERROR) (log-prefix).ERROR
} }
} else if $level >= ((log-level).WARNING) { } else if $level >= (log-level).WARNING {
if $short { if $short {
((log-short-prefix).WARNING) (log-short-prefix).WARNING
} else { } else {
((log-prefix).WARNING) (log-prefix).WARNING
} }
} else if $level >= ((log-level).INFO) { } else if $level >= (log-level).INFO {
if $short { if $short {
((log-short-prefix).INFO) (log-short-prefix).INFO
} else { } else {
((log-prefix).INFO) (log-prefix).INFO
} }
} else { } else {
if $short { if $short {
((log-short-prefix).DEBUG) (log-short-prefix).DEBUG
} else { } else {
((log-prefix).DEBUG) (log-prefix).DEBUG
} }
} }
} }
def current-log-level [] { def current-log-level [] {
let env_level = ($env.NU_log-level? | default (((log-level).INFO))) let env_level = ($env.NU_LOG_LEVEL? | default (log-level).INFO)
try { try {
$env_level | into int $env_level | into int
@ -264,11 +264,11 @@ export def custom [
} }
let valid_levels_for_defaulting = [ let valid_levels_for_defaulting = [
((log-level).CRITICAL) (log-level).CRITICAL
((log-level).ERROR) (log-level).ERROR
((log-level).WARNING) (log-level).WARNING
((log-level).INFO) (log-level).INFO
((log-level).DEBUG) (log-level).DEBUG
] ]
let prefix = if ($level_prefix | is-empty) { let prefix = if ($level_prefix | is-empty) {

4
crates/nu-std/tests/logger_tests/test_basic_commands.nu
View File

@ -6,9 +6,9 @@ def run [
--short --short
] { ] {
if $short { if $short {
^$nu.current-exe --no-config-file --commands $'use std; NU_log-level=($system_level) std log ($message_level) --short "test message"' ^$nu.current-exe --no-config-file --commands $'use std; NU_LOG_LEVEL=($system_level) std log ($message_level) --short "test message"'
} else { } else {
^$nu.current-exe --no-config-file --commands $'use std; NU_log-level=($system_level) std log ($message_level) "test message"' ^$nu.current-exe --no-config-file --commands $'use std; NU_LOG_LEVEL=($system_level) std log ($message_level) "test message"'
} }
| complete | get --ignore-errors stderr | complete | get --ignore-errors stderr
} }

6
crates/nu-std/tests/logger_tests/test_log_custom.nu
View File

@ -12,12 +12,12 @@ def run-command [
] { ] {
if ($level_prefix | is-empty) { if ($level_prefix | is-empty) {
if ($ansi | is-empty) { if ($ansi | is-empty) {
^$nu.current-exe --no-config-file --commands $'use std; NU_log-level=($system_level) std log custom "($message)" "($format)" ($log_level)' ^$nu.current-exe --no-config-file --commands $'use std; NU_LOG_LEVEL=($system_level) std log custom "($message)" "($format)" ($log_level)'
} else { } else {
^$nu.current-exe --no-config-file --commands $'use std; NU_log-level=($system_level) std log custom "($message)" "($format)" ($log_level) --ansi "($ansi)"' ^$nu.current-exe --no-config-file --commands $'use std; NU_LOG_LEVEL=($system_level) std log custom "($message)" "($format)" ($log_level) --ansi "($ansi)"'
} }
} else { } else {
^$nu.current-exe --no-config-file --commands $'use std; NU_log-level=($system_level) std log custom "($message)" "($format)" ($log_level) --level-prefix "($level_prefix)" --ansi "($ansi)"' ^$nu.current-exe --no-config-file --commands $'use std; NU_LOG_LEVEL=($system_level) std log custom "($message)" "($format)" ($log_level) --level-prefix "($level_prefix)" --ansi "($ansi)"'
} }
| complete | get --ignore-errors stderr | complete | get --ignore-errors stderr
} }

4
crates/nu-std/tests/logger_tests/test_log_format_flag.nu
View File

@ -10,9 +10,9 @@ def run-command [
--short --short
] { ] {
if $short { if $short {
^$nu.current-exe --no-config-file --commands $'use std; NU_log-level=($system_level) std log ($message_level) --format "($format)" --short "($message)"' ^$nu.current-exe --no-config-file --commands $'use std; NU_LOG_LEVEL=($system_level) std log ($message_level) --format "($format)" --short "($message)"'
} else { } else {
^$nu.current-exe --no-config-file --commands $'use std; NU_log-level=($system_level) std log ($message_level) --format "($format)" "($message)"' ^$nu.current-exe --no-config-file --commands $'use std; NU_LOG_LEVEL=($system_level) std log ($message_level) --format "($format)" "($message)"'
} }
| complete | get --ignore-errors stderr | complete | get --ignore-errors stderr
} }

40
crates/nu-std/tests/logger_tests/test_logger_env.nu
View File

@ -3,38 +3,38 @@ use std log *
#[test] #[test]
def env_log-ansi [] { def env_log-ansi [] {
assert equal ((log-ansi).CRITICAL) (ansi red_bold) assert equal (log-ansi).CRITICAL (ansi red_bold)
assert equal ((log-ansi).ERROR) (ansi red) assert equal (log-ansi).ERROR (ansi red)
assert equal ((log-ansi).WARNING) (ansi yellow) assert equal (log-ansi).WARNING (ansi yellow)
assert equal ((log-ansi).INFO) (ansi default) assert equal (log-ansi).INFO (ansi default)
assert equal ((log-ansi).DEBUG) (ansi default_dimmed) assert equal (log-ansi).DEBUG (ansi default_dimmed)
} }
#[test] #[test]
def env_log-level [] { def env_log-level [] {
assert equal ((log-level).CRITICAL) 50 assert equal (log-level).CRITICAL 50
assert equal ((log-level).ERROR) 40 assert equal (log-level).ERROR 40
assert equal ((log-level).WARNING) 30 assert equal (log-level).WARNING 30
assert equal ((log-level).INFO) 20 assert equal (log-level).INFO 20
assert equal ((log-level).DEBUG) 10 assert equal (log-level).DEBUG 10
} }
#[test] #[test]
def env_log-prefix [] { def env_log-prefix [] {
assert equal ((log-prefix).CRITICAL) "CRT" assert equal (log-prefix).CRITICAL "CRT"
assert equal ((log-prefix).ERROR) "ERR" assert equal (log-prefix).ERROR "ERR"
assert equal ((log-prefix).WARNING) "WRN" assert equal (log-prefix).WARNING "WRN"
assert equal ((log-prefix).INFO) "INF" assert equal (log-prefix).INFO "INF"
assert equal ((log-prefix).DEBUG) "DBG" assert equal (log-prefix).DEBUG "DBG"
} }
#[test] #[test]
def env_log-short-prefix [] { def env_log-short-prefix [] {
assert equal ((log-short-prefix).CRITICAL) "C" assert equal (log-short-prefix).CRITICAL "C"
assert equal ((log-short-prefix).ERROR) "E" assert equal (log-short-prefix).ERROR "E"
assert equal ((log-short-prefix).WARNING) "W" assert equal (log-short-prefix).WARNING "W"
assert equal ((log-short-prefix).INFO) "I" assert equal (log-short-prefix).INFO "I"
assert equal ((log-short-prefix).DEBUG) "D" assert equal (log-short-prefix).DEBUG "D"
} }
#[test] #[test]