Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #262 from mchf/SLE-12-GA
Ported last fixes into SLE-12 branch
- Loading branch information
Showing
10 changed files
with
260 additions
and
6 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,36 @@ | ||
<?xml version="1.0"?> | ||
<productDefines xmlns="http://www.suse.com/1.0/yast2ns" xmlns:config="http://www.suse.com/1.0/configns"> | ||
|
||
<!-- | ||
Work around for the text domain | ||
textdomain="control" | ||
--> | ||
|
||
<textdomain>control</textdomain> | ||
|
||
<globals> | ||
<additional_kernel_parameters/> | ||
<enable_autologin config:type="boolean">false</enable_autologin> | ||
<write_hostname_to_hosts config:type="boolean">true</write_hostname_to_hosts> | ||
<dhclient_set_hostname config:type="boolean">false</dhclient_set_hostname> | ||
</globals> | ||
|
||
<software> | ||
<delete_old_packages config:type="boolean">true</delete_old_packages> | ||
</software> | ||
|
||
<partitioning> | ||
<try_separate_home config:type="boolean">true</try_separate_home> | ||
</partitioning> | ||
|
||
<network> | ||
<force_static_ip config:type="boolean">false</force_static_ip> | ||
<network_manager>always</network_manager> | ||
<startmode>auto</startmode> | ||
</network> | ||
|
||
<texts> | ||
<dummy_desktop><label/></dummy_desktop> | ||
</texts> | ||
|
||
</productDefines> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,35 @@ | ||
<?xml version="1.0"?> | ||
<productDefines xmlns="http://www.suse.com/1.0/yast2ns" xmlns:config="http://www.suse.com/1.0/configns"> | ||
|
||
<!-- | ||
Work around for the text domain | ||
textdomain="control" | ||
--> | ||
|
||
<textdomain>control</textdomain> | ||
|
||
<globals> | ||
<additional_kernel_parameters/> | ||
<enable_autologin config:type="boolean">false</enable_autologin> | ||
<write_hostname_to_hosts config:type="boolean">true</write_hostname_to_hosts> | ||
</globals> | ||
|
||
<software> | ||
<delete_old_packages config:type="boolean">true</delete_old_packages> | ||
</software> | ||
|
||
<partitioning> | ||
<try_separate_home config:type="boolean">true</try_separate_home> | ||
</partitioning> | ||
|
||
<network> | ||
<force_static_ip config:type="boolean">false</force_static_ip> | ||
<network_manager>always</network_manager> | ||
<startmode>auto</startmode> | ||
</network> | ||
|
||
<texts> | ||
<dummy_desktop><label/></dummy_desktop> | ||
</texts> | ||
|
||
</productDefines> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,36 @@ | ||
<?xml version="1.0"?> | ||
<productDefines xmlns="http://www.suse.com/1.0/yast2ns" xmlns:config="http://www.suse.com/1.0/configns"> | ||
|
||
<!-- | ||
Work around for the text domain | ||
textdomain="control" | ||
--> | ||
|
||
<textdomain>control</textdomain> | ||
|
||
<globals> | ||
<additional_kernel_parameters/> | ||
<enable_autologin config:type="boolean">false</enable_autologin> | ||
<write_hostname_to_hosts config:type="boolean">true</write_hostname_to_hosts> | ||
<dhclient_set_hostname config:type="boolean">true</dhclient_set_hostname> | ||
</globals> | ||
|
||
<software> | ||
<delete_old_packages config:type="boolean">true</delete_old_packages> | ||
</software> | ||
|
||
<partitioning> | ||
<try_separate_home config:type="boolean">true</try_separate_home> | ||
</partitioning> | ||
|
||
<network> | ||
<force_static_ip config:type="boolean">false</force_static_ip> | ||
<network_manager>always</network_manager> | ||
<startmode>auto</startmode> | ||
</network> | ||
|
||
<texts> | ||
<dummy_desktop><label/></dummy_desktop> | ||
</texts> | ||
|
||
</productDefines> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,111 @@ | ||
#!/usr/bin/env rspec | ||
|
||
ENV["Y2DIR"] = File.expand_path("../../src", __FILE__) | ||
|
||
require "yast" | ||
require_relative "SCRStub" | ||
|
||
module Yast | ||
RSpec.configure do |c| | ||
c.include SCRStub | ||
end | ||
|
||
import "Arch" | ||
import "DNS" | ||
import "ProductControl" | ||
|
||
describe DNS do | ||
describe ".default_dhcp_hostname" do | ||
before do | ||
allow(Arch).to receive(:is_laptop).and_return laptop | ||
ProductControl.ReadControlFile(File.join(SCRStub::DATA_PATH, control_file)) | ||
end | ||
|
||
context "with dhcp_hostname=true in control file" do | ||
let(:control_file) { "dhcp_hostname_true.xml" } | ||
|
||
context "in a laptop" do | ||
let(:laptop) { true } | ||
|
||
it "returns the value from product features" do | ||
expect(DNS.default_dhcp_hostname).to eql(true) | ||
end | ||
end | ||
|
||
context "in a workstation" do | ||
let(:laptop) { false } | ||
|
||
it "returns the value from product features" do | ||
expect(DNS.default_dhcp_hostname).to eql(true) | ||
end | ||
end | ||
end | ||
|
||
context "with dhcp_hostname=false in control file" do | ||
let(:control_file) { "dhcp_hostname_false.xml" } | ||
|
||
context "in a laptop" do | ||
let(:laptop) { true } | ||
|
||
it "returns the value from product features" do | ||
expect(DNS.default_dhcp_hostname).to eql(false) | ||
end | ||
end | ||
|
||
context "in a workstation" do | ||
let(:laptop) { false } | ||
|
||
it "returns the value from product features" do | ||
expect(DNS.default_dhcp_hostname).to eql(false) | ||
end | ||
end | ||
end | ||
|
||
context "without dhcp_hostname in control file" do | ||
let(:control_file) { "dhcp_hostname_nil.xml" } | ||
|
||
context "in a laptop" do | ||
let(:laptop) { true } | ||
|
||
it "returns false" do | ||
expect(DNS.default_dhcp_hostname).to eql(false) | ||
end | ||
end | ||
|
||
context "in a workstation" do | ||
let(:laptop) { false } | ||
|
||
it "returns true" do | ||
expect(DNS.default_dhcp_hostname).to eql(true) | ||
end | ||
end | ||
end | ||
end | ||
|
||
describe ".Import" do | ||
context "with present dhcp_hostname and write_hostname" do | ||
let(:settings) { {"hostname" => "host", "dhcp_hostname" => true, "write_hostname" => true} } | ||
|
||
it "honors the provided values" do | ||
expect(DNS).to_not receive(:DefaultWriteHostname) | ||
expect(DNS).to_not receive(:default_dhcp_hostname) | ||
DNS.Import(settings) | ||
expect(DNS.write_hostname).to eql(true) | ||
expect(DNS.dhcp_hostname).to eql(true) | ||
end | ||
end | ||
|
||
context "with missing dhcp_hostname and write_hostname" do | ||
let(:settings) { {"hostname" => "host"} } | ||
|
||
it "relies on proper methods to get default values" do | ||
expect(DNS).to receive(:DefaultWriteHostname).and_return false | ||
expect(DNS).to receive(:default_dhcp_hostname).and_return false | ||
DNS.Import(settings) | ||
expect(DNS.write_hostname).to eql(false) | ||
expect(DNS.dhcp_hostname).to eql(false) | ||
end | ||
end | ||
end | ||
end | ||
end |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters