Hello community,

here is the log from the commit of package yast2-installation for 
openSUSE:Factory checked in at 2015-03-30 19:09:51
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-installation (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-installation.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-installation"

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2-installation/yast2-installation.changes    
2015-02-22 17:25:23.000000000 +0100
+++ 
/work/SRC/openSUSE:Factory/.yast2-installation.new/yast2-installation.changes   
    2015-03-30 19:09:52.000000000 +0200
@@ -1,0 +2,6 @@
+Fri Mar 27 13:14:22 UTC 2015 - [email protected]
+
+- allow proposal runner dialog to use different proposal store
+- 3.1.135
+
+-------------------------------------------------------------------

Old:
----
  yast2-installation-3.1.134.tar.bz2

New:
----
  yast2-installation-3.1.135.tar.bz2

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ yast2-installation.spec ++++++
--- /var/tmp/diff_new_pack.JU7nQ8/_old  2015-03-30 19:09:53.000000000 +0200
+++ /var/tmp/diff_new_pack.JU7nQ8/_new  2015-03-30 19:09:53.000000000 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-installation
-Version:        3.1.134
+Version:        3.1.135
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build

++++++ yast2-installation-3.1.134.tar.bz2 -> yast2-installation-3.1.135.tar.bz2 
++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-3.1.134/package/yast2-installation.changes 
new/yast2-installation-3.1.135/package/yast2-installation.changes
--- old/yast2-installation-3.1.134/package/yast2-installation.changes   
2015-02-19 12:14:12.000000000 +0100
+++ new/yast2-installation-3.1.135/package/yast2-installation.changes   
2015-03-27 15:14:16.000000000 +0100
@@ -1,4 +1,10 @@
 -------------------------------------------------------------------
+Fri Mar 27 13:14:22 UTC 2015 - [email protected]
+
+- allow proposal runner dialog to use different proposal store
+- 3.1.135
+
+-------------------------------------------------------------------
 Tue Feb 17 14:24:19 CET 2015 - [email protected]
 
 - get list of mounts from /proc/mounts in umount_finish (for
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-3.1.134/package/yast2-installation.spec 
new/yast2-installation-3.1.135/package/yast2-installation.spec
--- old/yast2-installation-3.1.134/package/yast2-installation.spec      
2015-02-19 12:14:12.000000000 +0100
+++ new/yast2-installation-3.1.135/package/yast2-installation.spec      
2015-03-27 15:14:16.000000000 +0100
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-installation
-Version:        3.1.134
+Version:        3.1.135
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-3.1.134/src/lib/installation/proposal_runner.rb 
new/yast2-installation-3.1.135/src/lib/installation/proposal_runner.rb
--- old/yast2-installation-3.1.134/src/lib/installation/proposal_runner.rb      
2015-02-19 12:14:12.000000000 +0100
+++ new/yast2-installation-3.1.135/src/lib/installation/proposal_runner.rb      
2015-03-27 15:14:16.000000000 +0100
@@ -38,7 +38,7 @@
       new.run
     end
 
-    def initialize
+    def initialize(store = ::Installation::ProposalStore)
       Yast.import "Pkg"
       Yast.import "UI"
       textdomain "installation"
@@ -63,6 +63,7 @@
 
       # BNC #463567
       @submods_already_called = []
+      @store_class = store
     end
 
     def run
@@ -78,7 +79,7 @@
         return :auto
       end
 
-      @store = ProposalStore.new(@proposal_mode)
+      @store = @store_class.new(@proposal_mode)
 
       build_dialog
 
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/yast2-installation-3.1.134/test/Makefile.am 
new/yast2-installation-3.1.135/test/Makefile.am
--- old/yast2-installation-3.1.134/test/Makefile.am     2015-02-19 
12:14:12.000000000 +0100
+++ new/yast2-installation-3.1.135/test/Makefile.am     2015-03-27 
15:14:16.000000000 +0100
@@ -1,5 +1,7 @@
 TESTS = \
   inst_functions_test.rb \
+  installation_misc_test.rb \
+  image_installation_test.rb \
   cio_ignore_test.rb \
   copy_logs_finish_test.rb \
   prep_shrink_test.rb \
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-3.1.134/test/image_installation_test.rb 
new/yast2-installation-3.1.135/test/image_installation_test.rb
--- old/yast2-installation-3.1.134/test/image_installation_test.rb      
2015-02-19 12:14:12.000000000 +0100
+++ new/yast2-installation-3.1.135/test/image_installation_test.rb      
2015-03-27 15:14:16.000000000 +0100
@@ -26,12 +26,12 @@
 describe Yast::ImageInstallation do
   describe "#FindImageSet" do
     before(:each) do
-      Yast::Pkg.stub(:SourceProvideDigestedFile).and_return(IMAGES_DESCR_FILE)
+      allow(Yast::Pkg).to 
receive(:SourceProvideDigestedFile).and_return(IMAGES_DESCR_FILE)
     end
 
     it "finds images matching architecture and selected patterns and returns 
if processing was successful" do
       ARCHS.each do |arch|
-        Yast::Arch.stub(:arch_short).and_return(arch)
+        allow(Yast::Arch).to receive(:arch_short).and_return(arch)
 
         [KDE4_PATTERNS, GNOME_PATTERNS, X11_PATTERNS, BASE_PATTERNS].each do 
|patterns|
           Yast::ImageInstallation.FreeInternalVariables()
@@ -44,7 +44,7 @@
 
     it "does not find any image using unsupported architecture and returns if 
processing was successful" do
       [KDE4_PATTERNS, GNOME_PATTERNS, X11_PATTERNS, BASE_PATTERNS].each do 
|patterns|
-        Yast::Arch.stub(:arch_short).and_return(NON_MATCHING_ARCH)
+        allow(Yast::Arch).to receive(:arch_short).and_return(NON_MATCHING_ARCH)
         Yast::ImageInstallation.FreeInternalVariables()
 
         expect(Yast::ImageInstallation.FindImageSet(patterns)).to eq(true)
@@ -55,7 +55,7 @@
 
     it "does not find any image using unsupported patterns and returns if 
processing was successful" do
       ARCHS.each do |arch|
-        Yast::Arch.stub(:arch_short).and_return(arch)
+        allow(Yast::Arch).to receive(:arch_short).and_return(arch)
 
         [NON_MATCHING_PATTERNS_2, NON_MATCHING_PATTERNS_2].each do |patterns|
           Yast::ImageInstallation.FreeInternalVariables()
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-3.1.134/test/inst_functions_test.rb 
new/yast2-installation-3.1.135/test/inst_functions_test.rb
--- old/yast2-installation-3.1.134/test/inst_functions_test.rb  2015-02-19 
12:14:12.000000000 +0100
+++ new/yast2-installation-3.1.135/test/inst_functions_test.rb  2015-03-27 
15:14:16.000000000 +0100
@@ -8,10 +8,10 @@
 Yast.import "Linuxrc"
 
 def stub_install_inf(install_inf)
-  Yast::Linuxrc.stub(:keys).and_return(install_inf.keys)
+  allow(Yast::Linuxrc).to receive(:keys).and_return(install_inf.keys)
 
   install_inf.keys.each do |key|
-    Yast::Linuxrc.stub(:InstallInf).with(key).and_return(install_inf[key])
+    allow(Yast::Linuxrc).to 
receive(:InstallInf).with(key).and_return(install_inf[key])
   end
 end
 
@@ -23,14 +23,14 @@
   it "returns empty list if no features were ignored on commandline" do
     linuxrc_commandline = "othercommand=xyz no_ignore_features=1 
something-else=555"
 
-    Yast::Linuxrc.stub(:InstallInf).and_return(linuxrc_commandline)
+    allow(Yast::Linuxrc).to 
receive(:InstallInf).and_return(linuxrc_commandline)
     expect(Yast::InstFunctions.ignored_features.sort).to be_empty
   end
 
   it "returns empty list if features ignored on commandline were empty" do
     linuxrc_commandline = "ignore_features="
 
-    Yast::Linuxrc.stub(:InstallInf).and_return(linuxrc_commandline)
+    allow(Yast::Linuxrc).to 
receive(:InstallInf).and_return(linuxrc_commandline)
     expect(Yast::InstFunctions.ignored_features.sort).to be_empty
   end
 
@@ -38,7 +38,7 @@
     linuxrc_commandline = "ignore_features=aa,b_b,c-c ignoredfeatures=a-a,dd 
othercommand=xyz"
     ignored_features    = ["aa", "bb", "cc", "dd"].sort
 
-    Yast::Linuxrc.stub(:InstallInf).and_return(linuxrc_commandline)
+    allow(Yast::Linuxrc).to 
receive(:InstallInf).and_return(linuxrc_commandline)
     expect(Yast::InstFunctions.ignored_features.sort).to eq ignored_features
   end
 
@@ -46,7 +46,7 @@
     linuxrc_commandline = "ignore_features=x-x,yy"
     ignored_features    = ["xx", "yy"].sort
 
-    Yast::Linuxrc.stub(:InstallInf).and_return(linuxrc_commandline)
+    allow(Yast::Linuxrc).to 
receive(:InstallInf).and_return(linuxrc_commandline)
     expect(Yast::InstFunctions.ignored_features.sort).to eq ignored_features
   end
 
@@ -54,7 +54,7 @@
     linuxrc_commandline = "trash=install ignore_feature=f.e.a.ture1 
ig.n.o.red_features=feature2 ignore_features=feature3"
     ignored_features    = ["feature1", "feature2", "feature3"].sort
 
-    Yast::Linuxrc.stub(:InstallInf).and_return(linuxrc_commandline)
+    allow(Yast::Linuxrc).to 
receive(:InstallInf).and_return(linuxrc_commandline)
     expect(Yast::InstFunctions.ignored_features.sort).to eq ignored_features
   end
 
@@ -62,7 +62,7 @@
     linuxrc_commandline = "i-g-n-o-r-e_feature=fff"
     ignored_features    = ["fff"]
 
-    Yast::Linuxrc.stub(:InstallInf).and_return(linuxrc_commandline)
+    allow(Yast::Linuxrc).to 
receive(:InstallInf).and_return(linuxrc_commandline)
     expect(Yast::InstFunctions.ignored_features.sort).to eq ignored_features
   end
 
@@ -70,7 +70,7 @@
     linuxrc_commandline = "Ignore_FeaTUres=ffF"
     ignored_features    = ["fff"]
 
-    Yast::Linuxrc.stub(:InstallInf).and_return(linuxrc_commandline)
+    allow(Yast::Linuxrc).to 
receive(:InstallInf).and_return(linuxrc_commandline)
     expect(Yast::InstFunctions.ignored_features.sort).to eq ignored_features
   end
 
@@ -111,42 +111,42 @@
   it "should be true if feature is exactly set on commandline" do
     linuxrc_commandline = "trash=install ignore_features=feature1 
ignored_features=feature2 ignore_features=feature3"
 
-    Yast::Linuxrc.stub(:InstallInf).and_return(linuxrc_commandline)
+    allow(Yast::Linuxrc).to 
receive(:InstallInf).and_return(linuxrc_commandline)
     expect(Yast::InstFunctions.feature_ignored?("feature2")).to eq(true)
   end
 
   it "should be true if feature is exactly on commandline using up/down case" 
do
     linuxrc_commandline = "trash=install ignore_features=fEAture1 
igno-RED_features=f-eatuRE_2 ignore_features=feature3"
 
-    Yast::Linuxrc.stub(:InstallInf).and_return(linuxrc_commandline)
+    allow(Yast::Linuxrc).to 
receive(:InstallInf).and_return(linuxrc_commandline)
     expect(Yast::InstFunctions.feature_ignored?("f-e-a-t-u-r-e-2")).to eq(true)
   end
 
   it "should be true if feature is set on commandline with dashes and 
underscores" do
     linuxrc_commandline = "trash=install ignore_features=feature1 
ignored_features=feature2 ignore_features=feature3"
 
-    Yast::Linuxrc.stub(:InstallInf).and_return(linuxrc_commandline)
+    allow(Yast::Linuxrc).to 
receive(:InstallInf).and_return(linuxrc_commandline)
     expect(Yast::InstFunctions.feature_ignored?("f-e-a-t-u-r-e_2")).to eq(true)
   end
 
   it "should be false if feature is not set on commandline" do
     linuxrc_commandline = "trash=install ignore_features=feature1 
ignored_features=feature2 ignore_features=feature3"
 
-    Yast::Linuxrc.stub(:InstallInf).and_return(linuxrc_commandline)
+    allow(Yast::Linuxrc).to 
receive(:InstallInf).and_return(linuxrc_commandline)
     expect(Yast::InstFunctions.feature_ignored?("no-such-feature")).to 
eq(false)
   end
 
   it "should be false if feature to check is empty" do
     linuxrc_commandline = "trash=install ignore_features=feature1 
ignored_features=feature2 ignore_features=feature3"
 
-    Yast::Linuxrc.stub(:InstallInf).and_return(linuxrc_commandline)
+    allow(Yast::Linuxrc).to 
receive(:InstallInf).and_return(linuxrc_commandline)
     expect(Yast::InstFunctions.feature_ignored?("")).to eq(false)
   end
 
   it "should be false if feature to check is undefined" do
     linuxrc_commandline = "trash=install ignore_features=feature1 
ignored_features=feature2 ignore_features=feature3"
 
-    Yast::Linuxrc.stub(:InstallInf).and_return(linuxrc_commandline)
+    allow(Yast::Linuxrc).to 
receive(:InstallInf).and_return(linuxrc_commandline)
     expect(Yast::InstFunctions.feature_ignored?(nil)).to eq(false)
   end
 
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-3.1.134/test/installation_misc_test.rb 
new/yast2-installation-3.1.135/test/installation_misc_test.rb
--- old/yast2-installation-3.1.134/test/installation_misc_test.rb       
2015-02-19 12:14:12.000000000 +0100
+++ new/yast2-installation-3.1.135/test/installation_misc_test.rb       
2015-03-27 15:14:16.000000000 +0100
@@ -8,10 +8,15 @@
   include Yast::InstallationMiscInclude
 end
 
+# fake AutoinstConfig to avoid dependency on autoyast
+module Yast
+  class AutoinstConfig
+  end
+end
+
 # we need to mock these modules
 Yast.import "Stage"
 Yast.import "Mode"
-Yast.import "AutoinstConfig"
 Yast.import "ProductControl"
 
 def stub_initialstage(bool)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-installation-3.1.134/test/proposal_runner_test.rb 
new/yast2-installation-3.1.135/test/proposal_runner_test.rb
--- old/yast2-installation-3.1.134/test/proposal_runner_test.rb 2015-02-19 
12:14:12.000000000 +0100
+++ new/yast2-installation-3.1.135/test/proposal_runner_test.rb 2015-03-27 
15:14:16.000000000 +0100
@@ -11,6 +11,13 @@
     stub_const("Yast::AutoinstConfig", autoinst_config)
   end
 
+  describe ".new" do
+    it "Allows passing different store implementation" do
+      class C < ::Installation::ProposalStore; end
+      expect { ::Installation::ProposalRunner.new(C) }.to_not raise_error
+    end
+  end
+
   describe "#run" do
     it "do nothing if run non-interactive" do
       Yast.import "Mode"
@@ -28,7 +35,7 @@
       expect(subject.run).to eq :auto
     end
 
-    it "passes a moske test" do
+    it "passes a smoke test" do
       expect { subject.run }.to_not raise_error
     end
   end


Reply via email to