Fixed old testsuite not depending on named service and bind package i…

…n test mode.
yast · Jul 1, 2019 · d0e0f94 · d0e0f94
1 parent 1633830
commit d0e0f94
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/package/yast2-dhcp-server.changes b/package/yast2-dhcp-server.changes
@@ -1,3 +1,10 @@
+-------------------------------------------------------------------
+Mon Jul  1 16:18:15 UTC 2019 - Knut Anderssen <kanderssen@suse.com>
+
+- Fixed failing old testsuite: do not depend on the environment,
+  skip bind absence in Mode.test() (bsc#1138668)
+- 4.2.1
+
 -------------------------------------------------------------------
 Fri May 31 12:28:42 UTC 2019 - Stasiek Michalski <hellcp@mailbox.org>
 

diff --git a/package/yast2-dhcp-server.spec b/package/yast2-dhcp-server.spec
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-dhcp-server
-Version:        4.2.0
+Version:        4.2.1
 Release:        0
 Summary:        YaST2 - DHCP Server Configuration
 Group:          System/YaST

diff --git a/src/modules/DhcpServer.pm b/src/modules/DhcpServer.pm
@@ -1611,7 +1611,7 @@ configured yet. Create a new configuration?");
 
     Progress->NextStage ();
 
-    $dns_server_available = DnsServerAPI->IsServiceConfigurableExternally();
+    $dns_server_available = (!Mode->test () && DnsServerAPI->IsServiceConfigurableExternally());
     if ($dns_server_available) {
 	DnsServerAPI->Read();
     }
@@ -1656,7 +1656,7 @@ sub Write {
 
     my $ok = 1;
 
-    $modified = $modified || FirewalldWrapper->is_modified ();
+    $modified = $modified || (!Mode->test () && FirewalldWrapper->is_modified ());
 
     if (! $modified)
     {