Merge 718cf22 into 67b347d

package/yast2-ntp-client.changes

@@ -1,3 +1,9 @@
+-------------------------------------------------------------------
+Tue May  4 10:22:49 UTC 2021 - Knut Anderssen <kanderssen@suse.com>
+
+- Adapted proposal client returning the dhcp ntp servers as strings
+- 4.4.1 (bsc#1185545)
+
 -------------------------------------------------------------------
 Tue Apr 20 13:51:55 UTC 2021 - Ladislav Slezák <lslezak@suse.cz>
 

package/yast2-ntp-client.spec

@@ -17,7 +17,7 @@
 
 
 Name:           yast2-ntp-client
-Version:        4.4.0
+Version:        4.4.1
 Release:        0
 Summary:        YaST2 - NTP Client Configuration
 License:        GPL-2.0-or-later

src/clients/ntp-client_proposal.rb

@@ -60,7 +60,7 @@ def main
         NtpClient.ntp_selected = Ops.get_boolean(@param, "ntp_used", false)
         @ret = true
       when "dhcp_ntp_servers"
-        @ret = NtpClient.dhcp_ntp_servers
+        @ret = NtpClient.dhcp_ntp_servers.map(&:hostname)
       when "MakeProposal"
         @ret = MakeProposal()
       when "Write"

test/ntp_client_proposal_test.rb

@@ -13,8 +13,32 @@
     client
   end
 
+  let(:dhcp_ntp_servers) { [] }
+
+  before do
+    allow(Yast::Lan).to receive(:dhcp_ntp_servers)
+      .and_return(dhcp_ntp_servers)
+  end
+
+  describe "#main" do
+    let(:client) { described_class.new }
+    let(:func) { "dhcp_ntp_servers" }
+
+    before do
+      allow(Yast::WFM).to receive(:Args).with(no_args).and_return([func])
+      allow(Yast::WFM).to receive(:Args).with(0).and_return(func)
+    end
+
+    context "when call with 'dhcp_ntp_servers' argument" do
+      let(:dhcp_ntp_servers) { ["test.example.net", "test2.example.net"] }
+
+      it "returns servers found via DHCP" do
+        expect(client.main).to eql(dhcp_ntp_servers)
+      end
+    end
+  end
+
   describe "#MakeProposal" do
-    let(:dhcp_ntp_servers) { [] }
     let(:config_was_read?) { false }
     let(:ntp_was_selected?) { false }