On 08/20/2012 11:53 AM, Jason Guiditta wrote:
On 16/08/12 16:06 -0700, Richard Su wrote:
https://www.aeolusproject.org/redmine/issues/3728

Pool.catalog_images_collection makes call out to warehouse.
Stubbed out that method to return expected values.
---
src/features/step_definitions/pool_steps.rb |    6 ++++++
1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/src/features/step_definitions/pool_steps.rb b/src/features/step_definitions/pool_steps.rb
index f575dc2..9012fe3 100644
--- a/src/features/step_definitions/pool_steps.rb
+++ b/src/features/step_definitions/pool_steps.rb
@@ -141,6 +141,12 @@ end
Given /^"([^"]*)" has catalog_entry "([^"]*)"$/ do |catalog_name, catalog_entry_name|
  catalog = Catalog.find_by_name(catalog_name)
deployable = FactoryGirl.create :deployable, :name => catalog_entry_name, :catalogs => [catalog]
+  Pool.any_instance.stub(:catalog_images_collection){
+    entries = []
+ entries << {:catalog => catalog.name ,:deployable => deployable.name,
+                   :image => "name uuid", :provider_images =>[] }
+    entries
+  }

Perhaps I am missing something obvious, but for the ablove method
body, why would it not just be collapsed into:

Pool.any_instance.stub(:catalog_images_collection) do
  [{:catalog => catalog.name ,:deployable => deployable.name,
    :image => "name uuid", :provider_images =>[]]
end

-j

That is much nicer. I'll repost.

end

Given /^"([^"]*)" has catalog_entry with parameters "([^"]*)"$/ do |catalog_name, catalog_entry_name|
--
1.7.7.6


Reply via email to