Add first specs; do some refactoring

.rspec

@@ -0,0 +1 @@
+--require spec_helper --color spec

.rubocop_todo.yml

@@ -1,6 +1,6 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2018-02-20 13:29:43 +0100 using RuboCop version 0.52.1.
+# on 2018-02-21 17:27:56 +0100 using RuboCop version 0.52.1.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
@@ -19,6 +19,21 @@ Metrics/MethodLength:
 Metrics/PerceivedComplexity:
   Max: 14
 
+# Offense count: 6
+# Configuration parameters: .
+# SupportedStyles: have_received, receive
+RSpec/MessageSpies:
+  EnforcedStyle: receive
+
+# Offense count: 1
+RSpec/MultipleExpectations:
+  Max: 6
+
+# Offense count: 2
+RSpec/SubjectStub:
+  Exclude:
+    - 'spec/rmt/wizard_spec.rb'
+
 # Offense count: 2
 Style/MultipleComparison:
   Exclude:

spec/rmt/wizard_spec.rb

@@ -0,0 +1,26 @@
+require 'rmt/wizard'
+
+Yast.import 'Wizard'
+Yast.import 'Sequencer'
+
+describe RMT::Wizard do
+  subject(:wizard) { described_class.new }
+
+  let(:config) { { foo: 'bar' } }
+
+  before do
+  end
+
+  it 'runs and goes through the sequence' do
+    expect(RMT::Base).to receive(:read_config_file).and_return({})
+
+    expect(Yast::Wizard).to receive(:CreateDialog)
+    expect(Yast::Wizard).to receive(:SetTitleIcon)
+
+    expect(wizard).to receive(:step1).and_return(:next)
+    expect(wizard).to receive(:step2).and_return(:next)
+
+    expect(Yast::UI).to receive(:CloseDialog)
+    wizard.run
+  end
+end

spec/spec_helper.rb

@@ -1,3 +1,5 @@
+$LOAD_PATH.unshift(File.expand_path('../../src/lib/', __FILE__))
+
 require 'yast'
 require 'yast/rspec'
 

src/lib/rmt/base.rb

@@ -64,7 +64,7 @@ def run_command(command, *params)
     params = params.map { |p| String.Quote(p) }
 
     SCR.Execute(
-        path('.target.bash'),
+      path('.target.bash'),
         Builtins.sformat(command, *params)
     )
   end

src/lib/rmt/wizard.rb

@@ -29,7 +29,7 @@ class RMT::Wizard < RMT::Base
   include Yast::UIShortcuts
   include Yast::Logger
 
-  def run
+  def initialize
     Yast.import 'UI'
     Yast.import 'Wizard'
     Yast.import 'Sequencer'
@@ -40,8 +40,6 @@ def run
     textdomain 'rmt'
 
     @config = RMT::Base.read_config_file
-
-    run_wizard
   end
 
   def step1
@@ -54,7 +52,7 @@ def step2
     page.run
   end
 
-  def run_wizard
+  def run
     aliases = {
       'step1' => -> { step1 },
       'step2' => -> { step2 }
@@ -71,10 +69,6 @@ def run_wizard
 
     Sequencer.Run(aliases, sequence)
 
-    Wizard.RestoreNextButton
-    Wizard.RestoreAbortButton
-    Wizard.RestoreBackButton
-
     UI.CloseDialog()
   end
 end