From: Petr Blaho<pblaho(a)redhat.com>
TargetImages are scoped by build_id so this has to be searched on ImageBuild instead of
ProviderImage
This was discovered during working on
https://bugzilla.redhat.com/show_bug.cgi?id=754956
---
.../controllers/api/target_images_controller.rb | 2 +-
.../api/target_images_controller_spec.rb | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/app/controllers/api/target_images_controller.rb
b/src/app/controllers/api/target_images_controller.rb
index 0c676e4..e597a8d 100644
--- a/src/app/controllers/api/target_images_controller.rb
+++ b/src/app/controllers/api/target_images_controller.rb
@@ -28,7 +28,7 @@ module Api
def index
if params[:build_id]
- @images =
Aeolus::Image::Warehouse::ProviderImage.find(params[:build_id]).target_images
+ @images =
Aeolus::Image::Warehouse::ImageBuild.find(params[:build_id]).target_images
else
@images = Aeolus::Image::Warehouse::TargetImage.all
end
diff --git a/src/spec/controllers/api/target_images_controller_spec.rb
b/src/spec/controllers/api/target_images_controller_spec.rb
index 6175710..424ff15 100644
--- a/src/spec/controllers/api/target_images_controller_spec.rb
+++ b/src/spec/controllers/api/target_images_controller_spec.rb
@@ -26,7 +26,7 @@ describe Api::TargetImagesController do
before(:each) do
send_and_accept_xml
- @pimage = mock(Aeolus::Image::Warehouse::ProviderImage,
+ @pimage = mock(Aeolus::Image::Warehouse::ImageBuild,
:id => '42')
@build = mock(Aeolus::Image::Warehouse::ImageBuild,
:id => '543')
wfm. ack