Skip to content

Commit

Permalink
fixes from manual testing
Browse files Browse the repository at this point in the history
  • Loading branch information
jreidinger committed Dec 9, 2022
1 parent 8df81fb commit fc66f99
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion src/include/kdump/dialogs.rb
Original file line number Diff line number Diff line change
Expand Up @@ -415,7 +415,7 @@ def tabs
"widget_names" => [
"DisBackButton",
"EnableDisalbeKdump",
(Kdump.system.supports_fadump? ? "FADump" : ""),
(Kdump.fadump_supported? ? "FADump" : ""),
"KdumpMemory"
]
},
Expand Down
2 changes: 1 addition & 1 deletion src/include/kdump/uifunctions.rb
Original file line number Diff line number Diff line change
Expand Up @@ -1435,7 +1435,7 @@ def StoreKdumpMemory(_key, _event)

# Initializes FADump settings in UI
def InitFADump(_key)
if Kdump.supports_fadump? && UI.WidgetExists(Id("FADump"))
if Kdump.fadump_supported? && UI.WidgetExists(Id("FADump"))
UI.ReplaceWidget(
Id("FADump"),
VBox(
Expand Down
4 changes: 2 additions & 2 deletions src/lib/kdump/clients/kdump.rb
Original file line number Diff line number Diff line change
Expand Up @@ -351,7 +351,7 @@ def main
}
}

if Kdump.system.supports_fadump?
if Kdump.fadump_supported?
@cmdline_description["actions"]["fadump"] = {
"handler" => fun_ref(method(:cmd_handle_fadump), "boolean (map)"),
# TRANSLATORS: CommandLine help
Expand Down Expand Up @@ -718,7 +718,7 @@ def cmdKdumpShow(_options)

CommandLine.Print("")

if Kdump.system.supports_fadump?
if Kdump.fadump_supported?
show_fadump_status
CommandLine.Print("")
end
Expand Down
2 changes: 1 addition & 1 deletion test/kdump_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -429,7 +429,7 @@
describe ".WriteKdumpBootParameter" do
before do
# FIXME: current tests do not cover fadump (ppc64 specific)
allow(Yast::Kdump.system).to receive(:supports_fadump?).and_return false
allow(Yast::Kdump).to receive(:fadump_supported?).and_return false
end

context "during autoinstallation" do
Expand Down

0 comments on commit fc66f99

Please sign in to comment.