modsvaskr 0.1.11 → 0.2.0

data/lib/modsvaskr/tests_suites/npc_head.rb

@@ -4,6 +4,7 @@ module Modsvaskr
 
   module TestsSuites
 
+    # Test NPCs by taking head screenshots
     class NpcHead < TestsSuites::Npc
 
       # Return the in-game tests suite to which we forward the tests to be run
@@ -21,7 +22,7 @@ module Modsvaskr
       # * Hash< String, Hash<Symbol,Object> >: Ordered hash of test information, per test name
       def discover_tests
         tests = super
-        tests.values.each do |test_info|
+        tests.each_value do |test_info|
           test_info[:name].gsub!('Take screenshot', 'Take head screenshot')
         end
         tests

data/lib/modsvaskr/ui.rb

@@ -1,204 +1,205 @@
-require 'curses_menu'
-require 'modsvaskr/logger'
-require 'modsvaskr/tests_runner'
-require 'modsvaskr/run_cmd'
-require 'modsvaskr/version'
-
-module Modsvaskr
-
-  class Ui
-
-    include Logger, RunCmd
-
-    # Constructor
-    #
-    # Parameters::
-    # * *config* (Config): Configuration object
-    def initialize(config:, skyrim: nil)
-      log "Launch Modsvaskr UI v#{Modsvaskr::VERSION} - Logs in #{Logger.log_file}"
-      @config = config
-    end
-
-    # Run the UI
-    def run
-      begin
-        last_modsvaskr_version = nil
-        gem_list_stdout = `gem list modsvaskr --remote`
-        gem_list_stdout.split("\n").each do |line|
-          if line =~ /^modsvaskr \((.+?)\)/
-            last_modsvaskr_version = $1
-            break
-          end
-        end
-        log "!!! Could not get latest Modsvaskr version. Output of gem list modsvaskr --remote:\n#{gem_list_stdout}" if last_modsvaskr_version.nil?
-        key_presses = @config.auto_keys.map do |key_str|
-          case key_str
-          when 'KEY_ENTER', 'KEY_ESCAPE'
-            CursesMenu.const_get(key_str.to_sym)
-          when /^KEY_\w+$/
-            Curses.const_get(key_str.to_sym)
-          else
-            key_str
-          end
-        end
-        CursesMenu.new(
-          "Modsvaskr v#{Modsvaskr::VERSION} - Stronghold of Mods#{!last_modsvaskr_version.nil? && last_modsvaskr_version != Modsvaskr::VERSION ? " - !!! New version available: #{last_modsvaskr_version}" : ''}",
-          key_presses: key_presses
-        ) do |main_menu|
-          @config.games.each do |game|
-            main_menu.item "#{game.name} (#{game.path})" do
-              CursesMenu.new(
-                "Modsvaskr v#{Modsvaskr::VERSION} - Stronghold of Mods > #{game.name}",
-                key_presses: key_presses
-              ) do |game_menu|
-                game_menu.item 'Testing' do
-                  # Read tests info
-                  tests_runner = TestsRunner.new(@config, game)
-                  # Selected test names, per test type
-                  # Hash< Symbol, Hash< String, nil > >
-                  selected_tests_suites = {}
-                  CursesMenu.new(
-                    "Modsvaskr v#{Modsvaskr::VERSION} - Stronghold of Mods > #{game.name} > Testing",
-                    key_presses: key_presses
-                  ) do |test_menu|
-                    tests_runner.tests_suites.each do |tests_suite|
-                      statuses_for_suite = tests_runner.statuses_for(tests_suite)
-                      all_tests_selected = selected_tests_suites.key?(tests_suite) &&
-                        selected_tests_suites[tests_suite].keys.sort == statuses_for_suite.map { |(test_name, _test_status)| test_name }.sort
-                      test_menu.item(
-                        "[#{
-                          if all_tests_selected
-                            '*'
-                          elsif selected_tests_suites.key?(tests_suite)
-                            '+'
-                          else
-                            ' '
-                          end
-                        }] #{tests_suite} - #{statuses_for_suite.select { |(_name, status)| status == 'ok' }.size} / #{statuses_for_suite.size}",
-                        actions: {
-                          'd' => {
-                            name: 'Details',
-                            execute: proc do
-                              CursesMenu.new(
-                                "Modsvaskr v#{Modsvaskr::VERSION} - Stronghold of Mods > #{game.name} > Testing > Tests #{tests_suite}",
-                                key_presses: key_presses
-                              ) do |tests_suite_menu|
-                                statuses_for_suite.each do |(test_name, test_status)|
-                                  test_selected = selected_tests_suites.key?(tests_suite) && selected_tests_suites[tests_suite].key?(test_name)
-                                  tests_suite_menu.item "[#{test_selected ? '*' : ' '}] #{test_name} - #{test_status} - #{tests_runner.test_info(tests_suite, test_name)[:name]}" do
-                                    if test_selected
-                                      selected_tests_suites[tests_suite].delete(test_name)
-                                      selected_tests_suites.delete(tests_suite) if selected_tests_suites[tests_suite].empty?
-                                    else
-                                      selected_tests_suites[tests_suite] = {} unless selected_tests_suites.key?(tests_suite)
-                                      selected_tests_suites[tests_suite][test_name] = nil
-                                    end
-                                    :menu_refresh
-                                  end
-                                end
-                                tests_suite_menu.item 'Back' do
-                                  :menu_exit
-                                end
-                              end
-                              :menu_refresh
-                            end
-                          }
-                        }
-                      ) do
-                        if all_tests_selected
-                          selected_tests_suites.delete(tests_suite)
-                        else
-                          selected_tests_suites[tests_suite] = Hash[statuses_for_suite.map { |(test_name, _test_status)| [test_name, nil] }]
-                        end
-                        :menu_refresh
-                      end
-                    end
-                    test_menu.item 'Select tests that are not ok' do
-                      selected_tests_suites = {}
-                      tests_runner.tests_suites.map do |tests_suite|
-                        tests_not_ok = {}
-                        tests_runner.statuses_for(tests_suite).each do |(test_name, test_status)|
-                          tests_not_ok[test_name] = nil unless test_status == 'ok'
-                        end
-                        selected_tests_suites[tests_suite] = tests_not_ok unless tests_not_ok.empty?
-                      end
-                      :menu_refresh
-                    end
-                    test_menu.item 'Register tests from selected test suites' do
-                      selected_tests_suites.keys.each do |tests_suite|
-                        tests_runner.set_statuses_for(
-                          tests_suite,
-                          (
-                            tests_runner.discover_tests_for(tests_suite) -
-                              tests_runner.statuses_for(tests_suite).map { |(test_name, _test_status)| test_name }
-                          ).map { |test_name| [test_name, ''] }
-                        )
-                      end
-                      :menu_refresh
-                    end
-                    test_menu.item 'Unregister tests from selected test suites' do
-                      selected_tests_suites.keys.each do |tests_suite|
-                        tests_runner.clear_tests_for(tests_suite)
-                      end
-                      :menu_refresh
-                    end
-                    test_menu.item 'Clear selected test statuses' do
-                      selected_tests_suites.each do |tests_suite, test_names_set|
-                        tests_runner.set_statuses_for(tests_suite, test_names_set.keys.map { |test_name| [test_name, ''] })
-                      end
-                      :menu_refresh
-                    end
-                    test_menu.item 'Run remaining selected tests' do
-                      tests_runner.run(
-                        selected_tests_suites.map do |selected_tests_suite, selected_test_names_set|
-                          [
-                            selected_tests_suite,
-                            # Make sure tests to be run are ordered from the registered list
-                            tests_runner.
-                              statuses_for(selected_tests_suite).map { |(test_name, _test_status)| test_name }.
-                              select { |test_name| selected_test_names_set.key?(test_name) }
-                          ]
-                        end
-                      )
-                      :menu_refresh
-                    end
-                    test_menu.item 'Back' do
-                      :menu_exit
-                    end
-                  end
-                end
-                game.complete_game_menu(game_menu) if game.respond_to?(:complete_game_menu)
-                game_menu.item 'Back' do
-                  :menu_exit
-                end
-              end
-            end
-          end
-          main_menu.item 'See logs' do
-            CursesMenu.new(
-              'Modsvaskr - Stronghold of Mods > Logs',
-              key_presses: key_presses
-            ) do |logs_menu|
-              File.read(Logger.log_file).split("\n").each do |line|
-                logs_menu.item line
-              end
-              logs_menu.item 'Back' do
-                :menu_exit
-              end
-            end
-          end
-          main_menu.item 'Quit' do
-            :menu_exit
-          end
-        end
-      rescue
-        log "Unhandled exception: #{$!}\n#{$!.backtrace.join("\n")}"
-        raise
-      ensure
-        log 'Close Modsvaskr UI'
-      end
-    end
-
-  end
-
-end
+require 'English'
+require 'curses_menu'
+require 'modsvaskr/logger'
+require 'modsvaskr/tests_runner'
+require 'modsvaskr/run_cmd'
+require 'modsvaskr/version'
+
+module Modsvaskr
+
+  # Main UI, using ncurses
+  class Ui
+
+    include RunCmd
+    include Logger
+
+    # Constructor
+    #
+    # Parameters::
+    # * *config* (Config): Configuration object
+    def initialize(config:)
+      log "Launch Modsvaskr UI v#{Modsvaskr::VERSION} - Logs in #{Logger.log_file}"
+      @config = config
+    end
+
+    # Run the UI
+    def run
+      last_modsvaskr_version = nil
+      gem_list_stdout = `gem list modsvaskr --remote`
+      gem_list_stdout.split("\n").each do |line|
+        if line =~ /^modsvaskr \((.+?)\)/
+          last_modsvaskr_version = Regexp.last_match(1)
+          break
+        end
+      end
+      log "!!! Could not get latest Modsvaskr version. Output of gem list modsvaskr --remote:\n#{gem_list_stdout}" if last_modsvaskr_version.nil?
+      key_presses = @config.auto_keys.map do |key_str|
+        case key_str
+        when 'KEY_ENTER', 'KEY_ESCAPE'
+          CursesMenu.const_get(key_str.to_sym)
+        when /^KEY_\w+$/
+          Curses.const_get(key_str.to_sym)
+        else
+          key_str
+        end
+      end
+      CursesMenu.new(
+        "Modsvaskr v#{Modsvaskr::VERSION} - Stronghold of Mods#{!last_modsvaskr_version.nil? && last_modsvaskr_version != Modsvaskr::VERSION ? " - !!! New version available: #{last_modsvaskr_version}" : ''}",
+        key_presses:
+      ) do |main_menu|
+        @config.games.each do |game|
+          main_menu.item "#{game.name} (#{game.path})" do
+            CursesMenu.new(
+              "Modsvaskr v#{Modsvaskr::VERSION} - Stronghold of Mods > #{game.name}",
+              key_presses:
+            ) do |game_menu|
+              game_menu.item 'Testing' do
+                # Read tests info
+                tests_runner = TestsRunner.new(@config, game)
+                # Selected test names, per test type
+                # Hash< Symbol, Hash< String, nil > >
+                selected_tests_suites = {}
+                CursesMenu.new(
+                  "Modsvaskr v#{Modsvaskr::VERSION} - Stronghold of Mods > #{game.name} > Testing",
+                  key_presses:
+                ) do |test_menu|
+                  tests_runner.tests_suites.each do |tests_suite|
+                    statuses_for_suite = tests_runner.statuses_for(tests_suite)
+                    all_tests_selected = selected_tests_suites.key?(tests_suite) &&
+                      selected_tests_suites[tests_suite].keys.sort == statuses_for_suite.map { |(test_name, _test_status)| test_name }.sort
+                    test_menu.item(
+                      "[#{
+                          if all_tests_selected
+                            '*'
+                          elsif selected_tests_suites.key?(tests_suite)
+                            '+'
+                          else
+                            ' '
+                          end
+                        }] #{tests_suite} - #{statuses_for_suite.select { |(_name, status)| status == 'ok' }.size} / #{statuses_for_suite.size}",
+                      actions: {
+                        'd' => {
+                          name: 'Details',
+                          execute: proc do
+                            CursesMenu.new(
+                              "Modsvaskr v#{Modsvaskr::VERSION} - Stronghold of Mods > #{game.name} > Testing > Tests #{tests_suite}",
+                              key_presses:
+                            ) do |tests_suite_menu|
+                              statuses_for_suite.each do |(test_name, test_status)|
+                                test_selected = selected_tests_suites.key?(tests_suite) && selected_tests_suites[tests_suite].key?(test_name)
+                                tests_suite_menu.item "[#{test_selected ? '*' : ' '}] #{test_name} - #{test_status} - #{tests_runner.test_info(tests_suite, test_name)[:name]}" do
+                                  if test_selected
+                                    selected_tests_suites[tests_suite].delete(test_name)
+                                    selected_tests_suites.delete(tests_suite) if selected_tests_suites[tests_suite].empty?
+                                  else
+                                    selected_tests_suites[tests_suite] = {} unless selected_tests_suites.key?(tests_suite)
+                                    selected_tests_suites[tests_suite][test_name] = nil
+                                  end
+                                  :menu_refresh
+                                end
+                              end
+                              tests_suite_menu.item 'Back' do
+                                :menu_exit
+                              end
+                            end
+                            :menu_refresh
+                          end
+                        }
+                      }
+                    ) do
+                      if all_tests_selected
+                        selected_tests_suites.delete(tests_suite)
+                      else
+                        selected_tests_suites[tests_suite] = statuses_for_suite.to_h { |(test_name, _test_status)| [test_name, nil] }
+                      end
+                      :menu_refresh
+                    end
+                  end
+                  test_menu.item 'Select tests that are not ok' do
+                    selected_tests_suites = {}
+                    tests_runner.tests_suites.map do |tests_suite|
+                      tests_not_ok = {}
+                      tests_runner.statuses_for(tests_suite).each do |(test_name, test_status)|
+                        tests_not_ok[test_name] = nil unless test_status == 'ok'
+                      end
+                      selected_tests_suites[tests_suite] = tests_not_ok unless tests_not_ok.empty?
+                    end
+                    :menu_refresh
+                  end
+                  test_menu.item 'Register tests from selected test suites' do
+                    selected_tests_suites.each_key do |tests_suite|
+                      tests_runner.set_statuses_for(
+                        tests_suite,
+                        (
+                          tests_runner.discover_tests_for(tests_suite) -
+                            tests_runner.statuses_for(tests_suite).map { |(test_name, _test_status)| test_name }
+                        ).map { |test_name| [test_name, ''] }
+                      )
+                    end
+                    :menu_refresh
+                  end
+                  test_menu.item 'Unregister tests from selected test suites' do
+                    selected_tests_suites.each_key do |tests_suite|
+                      tests_runner.clear_tests_for(tests_suite)
+                    end
+                    :menu_refresh
+                  end
+                  test_menu.item 'Clear selected test statuses' do
+                    selected_tests_suites.each do |tests_suite, test_names_set|
+                      tests_runner.set_statuses_for(tests_suite, test_names_set.keys.map { |test_name| [test_name, ''] })
+                    end
+                    :menu_refresh
+                  end
+                  test_menu.item 'Run remaining selected tests' do
+                    tests_runner.run(
+                      selected_tests_suites.map do |selected_tests_suite, selected_test_names_set|
+                        [
+                          selected_tests_suite,
+                          # Make sure tests to be run are ordered from the registered list
+                          tests_runner.
+                            statuses_for(selected_tests_suite).map { |(test_name, _test_status)| test_name }.
+                            select { |test_name| selected_test_names_set.key?(test_name) }
+                        ]
+                      end
+                    )
+                    :menu_refresh
+                  end
+                  test_menu.item 'Back' do
+                    :menu_exit
+                  end
+                end
+              end
+              game.complete_game_menu(game_menu) if game.respond_to?(:complete_game_menu)
+              game_menu.item 'Back' do
+                :menu_exit
+              end
+            end
+          end
+        end
+        main_menu.item 'See logs' do
+          CursesMenu.new(
+            'Modsvaskr - Stronghold of Mods > Logs',
+            key_presses:
+          ) do |logs_menu|
+            File.read(Logger.log_file).split("\n").each do |line|
+              logs_menu.item line
+            end
+            logs_menu.item 'Back' do
+              :menu_exit
+            end
+          end
+        end
+        main_menu.item 'Quit' do
+          :menu_exit
+        end
+      end
+    rescue
+      log "Unhandled exception: #{$ERROR_INFO}\n#{$ERROR_INFO.backtrace.join("\n")}"
+      raise
+    ensure
+      log 'Close Modsvaskr UI'
+    end
+
+  end
+
+end

data/lib/modsvaskr/version.rb

@@ -1,5 +1,5 @@
-module Modsvaskr
-
-  VERSION = '0.1.11'
-
-end
+module Modsvaskr
+
+  VERSION = '0.2.0'
+
+end

data/lib/modsvaskr/xedit.rb

@@ -1,65 +1,65 @@
-require 'csv'
-require 'modsvaskr/encoding'
-require 'modsvaskr/run_cmd'
-
-module Modsvaskr
-
-  # Helper to use an instance of xEdit
-  class Xedit
-
-    include RunCmd
-
-    # String: Installation path
-    attr_reader :install_path
-
-    # Constructor
-    #
-    # Parameters::
-    # * *install_path* (String): Installation path of xEdit
-    # * *game_path* (String): Installation path of the game to use xEdit on
-    def initialize(install_path, game_path)
-      @install_path = install_path
-      @game_path = game_path
-      # Set of scripts that have been run
-      @runs = {}
-    end
-
-    # Run an xEdit script
-    #
-    # Parameters::
-    # * *script* (String): Script name, as defined in xedit_scripts (without the Modsvaskr_ prefix and .pas suffix)
-    # * *only_once* (Boolean): If true, then make sure this script is run only once by instance [default: false]
-    def run_script(script, only_once: false)
-      if !only_once || !@runs.key?(script)
-        FileUtils.cp "#{__dir__}/../../xedit_scripts/Modsvaskr_#{script}.pas", "#{@install_path}/Edit Scripts/Modsvaskr_#{script}.pas"
-        run_cmd(
-          {
-            dir: @install_path,
-            exe: 'SSEEdit.exe'
-          },
-          args: %W[
-            -IKnowWhatImDoing
-            -AllowMasterFilesEdit
-            -SSE
-            -autoload
-            -script:"Modsvaskr_#{script}.pas"
-          ]
-        )
-        @runs[script] = nil
-      end
-    end
-
-    # Parse a CSV that has been dumped by a previous run of xEdit
-    #
-    # Parameters::
-    # * *csv* (String): Name of the CSV file (from Edit Scripts), without .csv
-    # * *row_block* (Proc): Code called for each CSV row
-    #   Parameters::
-    #   * *row* (Array<String>): CSV row
-    def parse_csv(csv, &row_block)
-      CSV.parse(Encoding.to_utf8(File.read("#{install_path}/Edit Scripts/#{csv}.csv", mode: 'rb'))).each(&row_block)
-    end
-
-  end
-
-end
+require 'csv'
+require 'modsvaskr/encoding'
+require 'modsvaskr/run_cmd'
+
+module Modsvaskr
+
+  # Helper to use an instance of xEdit
+  class Xedit
+
+    include RunCmd
+
+    # String: Installation path
+    attr_reader :install_path
+
+    # Constructor
+    #
+    # Parameters::
+    # * *install_path* (String): Installation path of xEdit
+    # * *game_path* (String): Installation path of the game to use xEdit on
+    def initialize(install_path, game_path)
+      @install_path = install_path
+      @game_path = game_path
+      # Set of scripts that have been run
+      @runs = {}
+    end
+
+    # Run an xEdit script
+    #
+    # Parameters::
+    # * *script* (String): Script name, as defined in xedit_scripts (without the Modsvaskr_ prefix and .pas suffix)
+    # * *only_once* (Boolean): If true, then make sure this script is run only once by instance [default: false]
+    def run_script(script, only_once: false)
+      return if only_once && @runs.key?(script)
+
+      FileUtils.cp "#{__dir__}/../../xedit_scripts/Modsvaskr_#{script}.pas", "#{@install_path}/Edit Scripts/Modsvaskr_#{script}.pas"
+      run_cmd(
+        {
+          dir: @install_path,
+          exe: 'SSEEdit.exe'
+        },
+        args: %W[
+          -IKnowWhatImDoing
+          -AllowMasterFilesEdit
+          -SSE
+          -autoload
+          -script:"Modsvaskr_#{script}.pas"
+        ]
+      )
+      @runs[script] = nil
+    end
+
+    # Parse a CSV that has been dumped by a previous run of xEdit
+    #
+    # Parameters::
+    # * *csv* (String): Name of the CSV file (from Edit Scripts), without .csv
+    # * Proc: Code called for each CSV row
+    #   Parameters::
+    #   * *row* (Array<String>): CSV row
+    def parse_csv(csv, &)
+      CSV.parse(Encoding.to_utf_8(File.read("#{install_path}/Edit Scripts/#{csv}.csv", mode: 'rb'))).each(&)
+    end
+
+  end
+
+end