Adapted to the new API of Y2Issues

yast · Jun 25, 2021 · 6f9d1d2 · 6f9d1d2
1 parent 1a95261
commit 6f9d1d2
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 16 deletions.
diff --git a/package/yast2-network.spec b/package/yast2-network.spec
@@ -33,8 +33,8 @@ BuildRequires:  yast2-devtools >= 3.1.15
 #for install task
 BuildRequires:  rubygem(%rb_default_ruby_abi:yast-rake)
 BuildRequires:  yast2-storage-ng
-# Yast::Equatable
-BuildRequires:  yast2 >= 4.4.7
+# Updated API for Y2Issues
+BuildRequires:  yast2 >= 4.4.14
 
 BuildRequires:  yast2-packager >= 4.0.18
 # Product control need xml agent
@@ -49,8 +49,8 @@ PreReq:         /bin/rm
 Requires:       sysconfig >= 0.80.0
 Requires:       yast2-proxy
 Requires:       yast2-storage-ng
-# Yast::Equatable
-Requires:       yast2 >= 4.4.7
+# Updated API for Y2Issues
+Requires:       yast2 >= 4.4.14
 # Packages::vnc_packages
 Requires:       yast2-packager >= 4.0.18
 Requires:       rubygem(%rb_default_ruby_abi:cfa) >= 0.6.4

diff --git a/src/modules/Lan.rb b/src/modules/Lan.rb
@@ -764,7 +764,7 @@ def yast_config
     def read_config(report: true)
       result = Y2Network::Config.from(:wicked)
       if result.issues? && report
-        return false unless Y2Issues.report(result.issues) == :yes
+        return false unless Y2Issues.report(result.issues)
       end
 
       system_config = result.config

diff --git a/test/lan_test.rb b/test/lan_test.rb
@@ -556,23 +556,15 @@
       end
 
       context "when the user asks to continue" do
-        let(:user_answer) { :yes }
+        let(:user_answer) { true }
 
         it "returns true" do
           expect(subject.read_config).to eq(true)
         end
       end
 
-      context "when the user asks to not continue" do
-        let(:user_answer) { :no }
-
-        it "returns false" do
-          expect(subject.read_config).to eq(false)
-        end
-      end
-
-      context "when the user asks to abort" do
-        let(:user_answer) { :abort }
+      context "when the user does not want to continue" do
+        let(:user_answer) { false }
 
         it "returns false" do
           expect(subject.read_config).to eq(false)