#418: updated specs for BuildLists::DependentPackagesJob

spec/jobs/dependent_packages_job_spec.rb

@@ -5,12 +5,22 @@ describe BuildLists::DependentPackagesJob do
   let(:user)        { build_list.user }
   let(:project)     { build_list.project }
   let(:ability)     { double(:ability) }
+  let(:project_ids) { [build_list.project_id] }
+  let(:arch_ids)    { [build_list.arch_id] }
+  let(:options)     { {
+    auto_publish_status:            'none',
+    auto_create_container:          '0',
+    include_testing_subrepository:  '0',
+    use_cached_chroot:              '0',
+    use_extra_tests:                '0'
+  } }
 
   before do
     stub_symlink_methods
     allow(BuildList).to receive(:find).with(123).and_return(build_list)
-    BuildList::Package.stub_chain(:joins, :where, :reorder, :uniq, :pluck).and_return([project.id])
+    # BuildList::Package.stub_chain(:joins, :where, :reorder, :uniq, :pluck).and_return([project.id])
     Project.stub_chain(:where, :to_a).and_return([project])
+    Arch.stub_chain(:where, :to_a).and_return([build_list.arch])
 
     allow(Ability).to receive(:new).and_return(ability)
     allow(ability).to receive(:can?).with(:show, build_list).and_return(true)
@@ -22,34 +32,34 @@ describe BuildLists::DependentPackagesJob do
 
   it 'ensures that not raises error' do
     expect do
-      subject.perform build_list.id, user.id
+      subject.perform build_list.id, user.id, project_ids, arch_ids, options
     end.to_not raise_exception
   end
 
   it 'ensures that creates build_list' do
     expect do
-      subject.perform build_list.id, user.id
+      subject.perform build_list.id, user.id, project_ids, arch_ids, options
     end.to change(BuildList, :count).by(1)
   end
 
   it 'ensures that do nothing if user has no access for show of build_list' do
     allow(ability).to receive(:can?).with(:show, build_list).and_return(false)
     expect do
-      subject.perform build_list.id, user.id
+      subject.perform build_list.id, user.id, project_ids, arch_ids, options
     end.to change(BuildList, :count).by(0)
   end
 
   it 'ensures that do nothing if user has no access for write of project' do
     allow(ability).to receive(:can?).with(:write, project).and_return(false)
     expect do
-      subject.perform build_list.id, user.id
+      subject.perform build_list.id, user.id, project_ids, arch_ids, options
     end.to change(BuildList, :count).by(0)
   end
 
   it 'ensures that do nothing if user has no access for create of build_list' do
     allow(ability).to receive(:can?).with(:create, anything).and_return(false)
     expect do
-      subject.perform build_list.id, user.id
+      subject.perform build_list.id, user.id, project_ids, arch_ids, options
     end.to change(BuildList, :count).by(0)
   end