Run pre-scripts just after the profile has been processed

[teclator]

Problem

AutoYaST pre-scripts are expected to be run before any other configuration, but since the introduction of the unified installation medium them are run after a check of the selected product.

If the profile is modified by user pre-scripts providing the product name to be used, a error eport will be shown as there is no product defined in the profile.

• https://trello.com/c/cDcxLbbN/1919-5-sle15-1110413-autoyast-pre-scripts-are-not-executed-before-software-section-products-entries-are-verified

Solution

Move the execution of the user pre-scripts as early as possible, that is, just after the profile has been processed.

This solution was already merged into SLE-15-GA and SLE-15-SP1 (#638 & #641) , so, this PR is a merge PR o that changes.

Test

• Tested manually using a shell script to fetch the completely profile:

Minimal fetch profile

<?xml version="1.0"?>
<!DOCTYPE profile>
<profile xmlns="http://www.suse.com/1.0/yast2ns" xmlns:config="http://www.suse.com/1.0/configns">
  <scripts>
    <pre-scripts config:type="list">
      <script>
        <debug config:type="boolean">false</debug>
        <feedback config:type="boolean">false</feedback>
        <filename>pre-fetch.sh</filename>
        <interpreter>shell</interpreter>
	<location><![CDATA[http://192.168.122.1/dud/scripts/pre-fetch.sh]]></location>
        <notification>Please wait while pre-fetch.sh is running...</notification>
      </script>
    </pre-scripts>
  </scripts>
</profile>

fetch script

#!/bin/sh

wget "http://192.168.122.1/dud/sle15_sp2_minimal.xml" -O /tmp/profile/modified.xml

exit 0

[coveralls]

Coverage increased (+0.2%) to 36.201% when pulling aa3cd68 on execute_prescripts_earlier into cceb533 on SLE-15-SP2.

[teclator]

During tests we have found that askDialog is already broken in SLE-15-SP2 since:

https://github.com/yast/yast-autoinstallation/pull/543/files#diff-0ca1c0540430356a2ed134aeeda8c987R80

as the client does not include Yast::UIShortcuts. Before the move to a class it inherited from Yast::Client, so already included it.

https://github.com/yast/yast-autoinstallation/pull/543/files#diff-c48716dab37675fe6b5e8c896f961ba0L13
https://github.com/yast/yast-ruby-bindings/blob/master/src/ruby/yast/client.rb#L11