639394606

Build Type

github

Committed by GitHub

Commit Message

<a href="https://github.com/yast/yast-bootloader/commit/<a class=hub.com/yast/yast-bootloader/commit/<a class="double-link" href="https://git"><a class=hub.com/yast/yast-bootloader/commit/<a class="double-link" href="https://git"><a class=hub.com/yast/yast-bootloader/commit/da9f6c918ead68932c4e7744b6b3b76e9c629266">da9f6c918">&lt;a href=&quot;https://github.com/yast/yast-bootloader/commit/</a><a class="double-link" href="https://github.com/yast/yast-bootloader/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/yast/yast-bootloader/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/yast/yast-bootloader/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/yast/yast-bootloader/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/yast/yast-bootloader/commit/&lt;a class=&quot;double-link&quot; href=&quot;https://git">&lt;a class=</a>hub.com/yast/yast-bootloader/commit/da9f6c918ead68932c4e7744b6b3b76e9c629266">da9f6c918</a><a href="https://github.com/yast/yast-bootloader/commit/da9f6c918ead68932c4e7744b6b3b76e9c629266">&quot;&gt;&amp;lt;a href=&amp;quot;https://github.com/yast/yast-bootloader/commit/&lt;/a&gt;&lt;a class=&quot;double-link&quot; href=&quot;https://github.com/yast/yast-bootloader/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/yast/yast-bootloader/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/yast/yast-bootloader/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/yast/yast-bootloader/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/yast/yast-bootloader/commit/&amp;lt;a class=&amp;quot;double-link&amp;quot; href=&amp;quot;https://git&quot;&gt;&amp;lt;a class=&lt;/a&gt;hub.com/yast/yast-bootloader/commit/da9f6c918ead68932c4e7744b6b3b76e9c629266&quot;&gt;da9f6c918&lt;/a&gt;&lt;a href=&quot;https://github.com/yast/yast... (continued)

Pull Request Pull Request #637: Device map dialog validation

Run Details

6 of 6 new or added lines in 1 file covered. (100.0%)

2840 of 3215 relevant lines covered (88.34%)

13.38 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

88.43

/src/lib/bootloader/device_map_dialog.rb

# frozen_string_literal: true

require "yast"
require "y2storage"

Yast.import "BootStorage"
Yast.import "Label"
Yast.import "Popup"
Yast.import "Mode"

require "bootloader/device_map"

module Bootloader
  # Represents dialog for modification of device map
  class DeviceMapDialog
    include Yast::UIShortcuts
    include Yast::I18n

    def self.run(device_map)
      new(device_map).run
    end

    def initialize(device_map)
      @device_map = device_map
    end

    def run
      textdomain "bootloader"

      return unless create_dialog

      begin
        return controller_loop
      ensure
        close_dialog
      end
    end

  private

    def create_dialog
      res = Yast::UI.OpenDialog dialog_content
      return false unless res

      refresh_buttons

      true
    end

    def close_dialog
      Yast::UI.CloseDialog
    end

    def controller_loop
      loop do
        input = Yast::UI.UserInput
        pos = selected_disk_index
        case input
        when :ok
          if disks.empty?
            Yast::Popup.Error(_("Device map must contain at least one device"))
            next
          end
          max_dev = Bootloader::DeviceMap::BIOS_LIMIT
          if disks.size > max_dev
            Yast::Popup.Error(_("Device map can have at maximum %i devices") % max_dev)
            next
          end
          store_order
          return :back # we just go back to original dialog
        when :cancel
          return :back
        when :up, :down, :delete, :add
          pos = handle_buttons(input, pos)
        when :disks
          refresh_buttons
          next
        else
          raise "Unknown action #{input}"
        end
        refresh_disks
        Yast::UI.ChangeWidget(Id(:disks), :CurrentItem, disks[pos])
        refresh_buttons
      end
    end

    def handle_buttons(action, pos)
      case action
      when :up
        disks.insert(pos - 1, disks.delete_at(pos))
        pos -= 1
      when :down
        disks.insert(pos + 1, disks.delete_at(pos))
        pos += 1
      when :delete
        disks.delete_at(pos)
        pos = (pos == disks.size) ? pos - 1 : pos
      when :add
        disks_to_add = if Yast::Mode.config || Yast::Mode.auto
          add_device_popup_ay_mode
        else
          add_devices_popup
        end
        disks.concat disks_to_add
        pos = disks.size - 1
      end

      pos
    end

    def dialog_content
      VBox(
        headings,
        VSpacing(1),
        contents,
        VSpacing(1),
        ending_buttons
      )
    end

    def headings
      Heading(_("Disk order settings"))
    end

    def contents
      HBox(
        VBox(
          SelectionBox(Id(:disks), Opt(:notify), _("D&isks"), disks),
          # make dialog reasonable big, but increace mainly selection box which contain device names
          HSpacing(60)
        ),
        action_buttons
      )
    end

    def action_buttons
      VBox(
        PushButton(Id(:add), Opt(:key_F3), Yast::Label.AddButton),
        PushButton(Id(:delete), Opt(:key_F5), Yast::Label.DeleteButton),
        VSpacing(1),
        PushButton(Id(:up), Opt(:hstretch), Yast::Label.UpButton),
        PushButton(Id(:down), Opt(:hstretch), Yast::Label.DownButton),
        VStretch()
      )
    end

    def ending_buttons
      ButtonBox(
        PushButton(Id(:ok), Yast::Label.OKButton),
        PushButton(Id(:cancel), Yast::Label.CancelButton)
      )
    end

    def disks
      @disks ||= @device_map.disks_order
    end

    def refresh_disks
      Yast::UI.ChangeWidget(Id(:disks), :Items, disks)
    end

    def available_devices
      staging = Y2Storage::StorageManager.instance.staging
      staging.disk_devices.map(&:name)
    end

    def store_order
      @device_map.clear_mapping
      disks.each_with_index do |disk, index|
        @device_map.add_mapping("hd#{index}", disk)
      end
    end

    def add_device_popup_ay_mode
      popup = VBox(
        VSpacing(1),
        # textentry header
        InputField(Id(:devname), Opt(:hstretch), _("&Device")),
        VSpacing(1),
        ButtonBox(
          PushButton(Id(:ok), Opt(:key_F10, :default), Yast::Label.OKButton),
          PushButton(Id(:cancel), Opt(:key_F8), Yast::Label.CancelButton)
        ),
        VSpacing(1)
      )
      Yast::UI.OpenDialog(popup)
      Yast::UI.SetFocus(:devname)
      pushed = Yast::UI.UserInput
      new_dev = Yast::UI.QueryWidget(Id(:devname), :Value)
      Yast::UI.CloseDialog

      (pushed == :ok) ? [new_dev] : []
    end

    def add_devices_popup
      devices = available_devices - disks
      popup = VBox(
        MultiSelectionBox(
          Id(:devnames),
          _("&Devices:"),
          devices
        ),
        ending_buttons,
        VSpacing(1)
      )
      Yast::UI.OpenDialog(popup)
      Yast::UI.SetFocus(:devnames)
      pushed = Yast::UI.UserInput
      new_devs = Yast::UI.QueryWidget(Id(:devnames), :SelectedItems)
      Yast::UI.CloseDialog

      (pushed == :ok) ? new_devs : []
    end

    def selected_disk_index
      disks.index(Yast::UI.QueryWidget(Id(:disks), :CurrentItem))
    end

    def refresh_buttons
      # by default enable delete and later disable if there are no disks
      Yast::UI.ChangeWidget(Id(:delete), :Enabled, true)
      pos = selected_disk_index
      if !pos # nothing selected
        disk_to_select = disks.first
        # there is no disks
        if !disk_to_select
          up_down_enablement(false, false)
          Yast::UI.ChangeWidget(Id(:delete), :Enabled, false)
          return
        end
        Yast::UI.ChangeWidget(Id(:disks), :CurrentItem, disk_to_select)
        refresh_buttons
        return
      end

      up_down_enablement(pos > 0, pos < (disks.size - 1))
    end

    def up_down_enablement(up_enabled, down_enabled)
      Yast::UI.ChangeWidget(Id(:up), :Enabled, up_enabled)
      Yast::UI.ChangeWidget(Id(:down), :Enabled, down_enabled)
    end
  end
end

1	# frozen_string_literal: true
2
3	require "yast"	1✔
4	require "y2storage"	1✔
5
6	Yast.import "BootStorage"	1✔
7	Yast.import "Label"	1✔
8	Yast.import "Popup"	1✔
9	Yast.import "Mode"	1✔
10
11	require "bootloader/device_map"	1✔
12
13	module Bootloader	1✔
14	# Represents dialog for modification of device map
15	class DeviceMapDialog	1✔
16	include Yast::UIShortcuts	1✔
17	include Yast::I18n	1✔
18
19	def self.run(device_map)	1✔
20	new(device_map).run	×
21	end
22
23	def initialize(device_map)	1✔
24	@device_map = device_map	8✔
25	end
26
27	def run	1✔
28	textdomain "bootloader"	9✔
29
30	return unless create_dialog	9✔
31
32	begin	9✔
33	return controller_loop	9✔
34	ensure
35	close_dialog	9✔
36	end
37	end
38
39	private	1✔
40
41	def create_dialog	1✔
42	res = Yast::UI.OpenDialog dialog_content	9✔
43	return false unless res	9✔
44
45	refresh_buttons	9✔
46
47	true	9✔
48	end
49
50	def close_dialog	1✔
51	Yast::UI.CloseDialog	9✔
52	end
53
54	def controller_loop	1✔
55	loop do	9✔
56	input = Yast::UI.UserInput	16✔
57	pos = selected_disk_index	16✔
58	case input	16✔
59	when :ok
60	if disks.empty?	8✔
61	Yast::Popup.Error(_("Device map must contain at least one device"))	×
62	next	×
63	end
64	max_dev = Bootloader::DeviceMap::BIOS_LIMIT	8✔
65	if disks.size > max_dev	8✔
66	Yast::Popup.Error(_("Device map can have at maximum %i devices") % max_dev)	×
67	next	×
68	end
69	store_order	8✔
70	return :back # we just go back to original dialog	8✔
71	when :cancel
72	return :back	1✔
73	when :up, :down, :delete, :add
74	pos = handle_buttons(input, pos)	6✔
75	when :disks
76	refresh_buttons	1✔
77	next	1✔
78	else
79	raise "Unknown action #{input}"	×
80	end
81	refresh_disks	6✔
82	Yast::UI.ChangeWidget(Id(:disks), :CurrentItem, disks[pos])	6✔
83	refresh_buttons	6✔
84	end
85	end
86
87	def handle_buttons(action, pos)	1✔
88	case action	6✔
89	when :up
90	disks.insert(pos - 1, disks.delete_at(pos))	1✔
91	pos -= 1	1✔
92	when :down
93	disks.insert(pos + 1, disks.delete_at(pos))	1✔
94	pos += 1	1✔
95	when :delete
96	disks.delete_at(pos)	1✔
97	pos = (pos == disks.size) ? pos - 1 : pos	1✔
98	when :add
99	disks_to_add = if Yast::Mode.config \|\| Yast::Mode.auto	3✔
100	add_device_popup_ay_mode	2✔
101	else
102	add_devices_popup	1✔
103	end
104	disks.concat disks_to_add	3✔
105	pos = disks.size - 1	3✔
106	end
107
108	pos	6✔
109	end
110
111	def dialog_content	1✔
112	VBox(	9✔
113	headings,
114	VSpacing(1),
115	contents,
116	VSpacing(1),
117	ending_buttons
118	)
119	end
120
121	def headings	1✔
122	Heading(_("Disk order settings"))	9✔
123	end
124
125	def contents	1✔
126	HBox(	9✔
127	VBox(
128	SelectionBox(Id(:disks), Opt(:notify), _("D&isks"), disks),
129	# make dialog reasonable big, but increace mainly selection box which contain device names
130	HSpacing(60)
131	),
132	action_buttons
133	)
134	end
135
136	def action_buttons	1✔
137	VBox(	9✔
138	PushButton(Id(:add), Opt(:key_F3), Yast::Label.AddButton),
139	PushButton(Id(:delete), Opt(:key_F5), Yast::Label.DeleteButton),
140	VSpacing(1),
141	PushButton(Id(:up), Opt(:hstretch), Yast::Label.UpButton),
142	PushButton(Id(:down), Opt(:hstretch), Yast::Label.DownButton),
143	VStretch()
144	)
145	end
146
147	def ending_buttons	1✔
148	ButtonBox(	10✔
149	PushButton(Id(:ok), Yast::Label.OKButton),
150	PushButton(Id(:cancel), Yast::Label.CancelButton)
151	)
152	end
153
154	def disks	1✔
155	@disks \|\|= @device_map.disks_order	106✔
156	end
157
158	def refresh_disks	1✔
159	Yast::UI.ChangeWidget(Id(:disks), :Items, disks)	6✔
160	end
161
162	def available_devices	1✔
163	staging = Y2Storage::StorageManager.instance.staging	1✔
164	staging.disk_devices.map(&:name)	1✔
165	end
166
167	def store_order	1✔
168	@device_map.clear_mapping	8✔
169	disks.each_with_index do \|disk, index\|	8✔
170	@device_map.add_mapping("hd#{index}", disk)	20✔
171	end
172	end
173
174	def add_device_popup_ay_mode	1✔
175	popup = VBox(	2✔
176	VSpacing(1),
177	# textentry header
178	InputField(Id(:devname), Opt(:hstretch), _("&Device")),
179	VSpacing(1),
180	ButtonBox(
181	PushButton(Id(:ok), Opt(:key_F10, :default), Yast::Label.OKButton),
182	PushButton(Id(:cancel), Opt(:key_F8), Yast::Label.CancelButton)
183	),
184	VSpacing(1)
185	)
186	Yast::UI.OpenDialog(popup)	2✔
187	Yast::UI.SetFocus(:devname)	2✔
188	pushed = Yast::UI.UserInput	2✔
189	new_dev = Yast::UI.QueryWidget(Id(:devname), :Value)	2✔
190	Yast::UI.CloseDialog	2✔
191
192	(pushed == :ok) ? [new_dev] : []	2✔
193	end
194
195	def add_devices_popup	1✔
196	devices = available_devices - disks	1✔
197	popup = VBox(	1✔
198	MultiSelectionBox(
199	Id(:devnames),
200	_("&Devices:"),
201	devices
202	),
203	ending_buttons,
204	VSpacing(1)
205	)
206	Yast::UI.OpenDialog(popup)	1✔
207	Yast::UI.SetFocus(:devnames)	1✔
208	pushed = Yast::UI.UserInput	1✔
209	new_devs = Yast::UI.QueryWidget(Id(:devnames), :SelectedItems)	1✔
210	Yast::UI.CloseDialog	1✔
211
212	(pushed == :ok) ? new_devs : []	1✔
213	end
214
215	def selected_disk_index	1✔
216	disks.index(Yast::UI.QueryWidget(Id(:disks), :CurrentItem))	32✔
217	end
218
219	def refresh_buttons	1✔
220	# by default enable delete and later disable if there are no disks
221	Yast::UI.ChangeWidget(Id(:delete), :Enabled, true)	16✔
222	pos = selected_disk_index	16✔
223	if !pos # nothing selected	16✔
224	disk_to_select = disks.first	×
225	# there is no disks
226	if !disk_to_select	×
227	up_down_enablement(false, false)	×
228	Yast::UI.ChangeWidget(Id(:delete), :Enabled, false)	×
229	return	×
230	end
231	Yast::UI.ChangeWidget(Id(:disks), :CurrentItem, disk_to_select)	×
232	refresh_buttons	×
233	return	×
234	end
235
236	up_down_enablement(pos > 0, pos < (disks.size - 1))	16✔
237	end
238
239	def up_down_enablement(up_enabled, down_enabled)	1✔
240	Yast::UI.ChangeWidget(Id(:up), :Enabled, up_enabled)	16✔
241	Yast::UI.ChangeWidget(Id(:down), :Enabled, down_enabled)	16✔
242	end
243	end
244	end

yast / yast-bootloader / 639394606

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous