Remove skip button on disk selection dialog

yast · Nov 27, 2017 · 1058c0e · 1058c0e
1 parent 1fad392
commit 1058c0e
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 17 deletions.
diff --git a/src/lib/autoinstall/dialogs/disk_selector.rb b/src/lib/autoinstall/dialogs/disk_selector.rb
@@ -95,11 +95,6 @@ def abort_handler
         finish_dialog(:abort)
       end
 
-      # Handler for the `Skip` button
-      def skip_handler
-        finish_dialog(:skip)
-      end
-
     protected
 
       # Returns a list of options containing available disks
@@ -130,7 +125,6 @@ def disks
       def buttons
         [
           PushButton(Id(:ok), Opt(:okButton, :key_F10, :default), Yast::Label.ContinueButton),
-          PushButton(Id(:skip), Yast::Label.SkipButton),
           abort_button
         ]
       end

diff --git a/test/lib/dialogs/disk_selector_test.rb b/test/lib/dialogs/disk_selector_test.rb
@@ -49,9 +49,8 @@
       dialog.dialog_content
     end
 
-    it "displays ok, skip and abort buttons" do
+    it "displays ok and abort buttons" do
       expect(dialog).to receive(:PushButton).with(Id(:ok), anything, anything)
-      expect(dialog).to receive(:PushButton).with(Id(:skip), anything)
       expect(dialog).to receive(:PushButton).with(Id(:abort), anything, anything)
       dialog.dialog_content
     end
@@ -76,7 +75,6 @@
 
       it "only shows the abort button" do
         expect(dialog).to_not receive(:PushButton).with(Id(:ok), anything, anything)
-        expect(dialog).to_not receive(:PushButton).with(Id(:skip), anything)
         expect(dialog).to receive(:PushButton).with(Id(:abort), anything, anything)
         dialog.dialog_content
       end
@@ -120,13 +118,5 @@
         expect(dialog.run).to eq(:abort)
       end
     end
-
-    context "when the user presses 'Skip'" do
-      let(:button) { :skip }
-
-      it "returns :skip" do
-        expect(dialog.run).to eq(:skip)
-      end
-    end
   end
 end