Skip to content

Commit

Permalink
test: fix snapshots
Browse files Browse the repository at this point in the history
  • Loading branch information
jdx committed Oct 4, 2024
1 parent 8dfc6be commit e1bc269
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 1 deletion.
2 changes: 2 additions & 0 deletions src/cli/settings/ls.rs
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ mod tests {
libgit2 = true
not_found_auto_install = true
paranoid = false
pin = false
pipx_uvx = false
plugin_autoupdate_last_check_duration = "20m"
python_default_packages_file = "~/.default-python-packages"
Expand Down Expand Up @@ -143,6 +144,7 @@ mod tests {
node
not_found_auto_install
paranoid
pin
pipx_uvx
plugin_autoupdate_last_check_duration
python_default_packages_file
Expand Down
2 changes: 2 additions & 0 deletions src/cli/settings/set.rs
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@ impl SettingsSet {
"node.mirror_url" => self.value.into(),
"not_found_auto_install" => parse_bool(&self.value)?,
"paranoid" => parse_bool(&self.value)?,
"pin" => parse_bool(&self.value)?,
"pipx_uvx" => parse_bool(&self.value)?,
"plugin_autoupdate_last_check_duration" => self.value.into(),
"python_compile" => parse_bool(&self.value)?,
Expand Down Expand Up @@ -172,6 +173,7 @@ pub mod tests {
libgit2 = true
not_found_auto_install = true
paranoid = false
pin = false
pipx_uvx = false
plugin_autoupdate_last_check_duration = "1"
python_default_packages_file = "~/.default-python-packages"
Expand Down
1 change: 1 addition & 0 deletions src/cli/settings/unset.rs
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,7 @@ mod tests {
libgit2 = true
not_found_auto_install = true
paranoid = false
pin = false
pipx_uvx = false
plugin_autoupdate_last_check_duration = "20m"
python_default_packages_file = "~/.default-python-packages"
Expand Down
1 change: 0 additions & 1 deletion src/cli/upgrade.rs
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,6 @@ impl Upgrade {

let to_remove = outdated
.iter()
.filter(|o| o.bump.is_some())

This comment has been minimized.

Copy link
@liskin

liskin Oct 8, 2024

Contributor

This partially reverts 8dfc6be (which, as we both noted in #2704, doesn't actually fix the issue) — did you mean to do it? Seems a bit out of place in this commit, more like a forgotten hunk when you were trying to figure out #2704.

.filter_map(|o| o.current.as_ref().map(|current| (o, current)))
.collect::<Vec<_>>();

Expand Down

0 comments on commit e1bc269

Please sign in to comment.