aeolus-image 0.0.1 → 0.4.0

data/spec/aeolus_image/model/factory/provider_image_spec.rb

@@ -0,0 +1,12 @@
+require 'spec_helper'
+
+module Aeolus
+  module Image
+    module Factory
+      describe ProviderImage do
+        p = ProviderImage.new
+        Aeolus::Image::Factory::Base.site = "https://localhost:8075/imagefactory"
+      end
+    end
+  end
+end

data/spec/models/factory/base_spec.rb

@@ -0,0 +1,94 @@
+#
+#   Copyright 2011 Red Hat, Inc.
+#
+#   Licensed under the Apache License, Version 2.0 (the "License");
+#   you may not use this file except in compliance with the License.
+#   You may obtain a copy of the License at
+#
+#       http://www.apache.org/licenses/LICENSE-2.0
+#
+#   Unless required by applicable law or agreed to in writing, software
+#   distributed under the License is distributed on an "AS IS" BASIS,
+#   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#   See the License for the specific language governing permissions and
+#   limitations under the License.
+#
+require 'spec_helper'
+require 'timecop'
+
+describe Aeolus::Image::Factory::Base do
+
+  before(:each) do
+    Timecop.travel(Time.local(2011, 10, 17, 13, 38, 20))
+  end
+
+  after(:each) do
+    Timecop.return
+  end
+
+  it "should not use_oauth? if configuration is missing" do
+    Aeolus::Image::Factory::Base.config = {
+      :site => 'http://127.0.0.1:8075/imagefactory'
+    }
+    Aeolus::Image::Factory::Base.use_oauth?.should be_false
+  end
+
+  it "should use_oauth? if configuration is present" do
+    Aeolus::Image::Factory::Base.config = {
+      :site => 'http://127.0.0.1:8075/imagefactory',
+      :consumer_key => 'something',
+      :consumer_secret => 'anything'
+    }
+    Aeolus::Image::Factory::Base.use_oauth?.should be_true
+  end
+
+  context do
+    use_vcr_cassette "oauth_success_valid"
+
+    it "should succeed with valid OAuth credentials" do
+      Aeolus::Image::Factory::Base.config = {
+        :site => 'http://127.0.0.1:8075/imagefactory',
+        :consumer_key => 'mock-key',
+        :consumer_secret => 'mock-secret'
+      }
+      template = "<template>\n  <name>f14jeos</name>\n  <os>\n    <name>Fedora</name>\n    <version>14</version>\n    <arch>x86_64</arch>\n    <install type='url'>\n      <url>http://download.fedoraproject.org/pub/fedora/linux/releases/14/Fedora/x86_64/os/</url>\n    </install>\n  </os>\n  <description>Fedora 14</description>\n</template>\n"
+      img = Aeolus::Image::Factory::Image.new(:targets => 'ec2', :template => template)
+      img.save!.should be_true
+    end
+  end
+
+  context do
+    use_vcr_cassette "oauth_fail_invalid"
+
+    it "should fail with invalid OAuth credentials" do
+      Aeolus::Image::Factory::Base.config = {
+        :site => 'http://127.0.0.1:8075/imagefactory',
+        :consumer_key => 'mock-key',
+        :consumer_secret => 'wrong-secret'
+      }
+      template = "<template>\n  <name>f14jeos</name>\n  <os>\n    <name>Fedora 2</name>\n    <version>14</version>\n    <arch>x86_64</arch>\n    <install type='url'>\n      <url>http://download.fedoraproject.org/pub/fedora/linux/releases/14/Fedora/x86_64/os/</url>\n    </install>\n  </os>\n  <description>Fedora 14</description>\n</template>\n"
+      img = Aeolus::Image::Factory::Image.new(:targets => 'ec2', :template => template)
+      lambda {
+        img.save!
+      }.should raise_error(ActiveResource::UnauthorizedAccess)
+    end
+  end
+
+  context do
+    use_vcr_cassette "oauth_fail_no"
+
+    it "should fail with no OAuth credentials" do
+      Aeolus::Image::Factory::Base.config = {
+        :site => 'http://127.0.0.1:8075/imagefactory'
+      }
+      template = "<template>\n  <name>f14jeos</name>\n  <os>\n    <name>Fedora 3</name>\n    <version>14</version>\n    <arch>x86_64</arch>\n    <install type='url'>\n      <url>http://download.fedoraproject.org/pub/fedora/linux/releases/14/Fedora/x86_64/os/</url>\n    </install>\n  </os>\n  <description>Fedora 14</description>\n</template>\n"
+      img = Aeolus::Image::Factory::Image.new(:targets => 'ec2', :template => template)
+      # ServerError is actually not what _should_ be returned, but a bug means that's what we get at the moment.
+      # When the bug is fixed, we need to update to test for that.
+      lambda {
+        img.save!.should be_false
+      }.should raise_error(ActiveResource::ServerError)
+    end
+  end
+
+end

data/spec/models/factory/builder_spec.rb

@@ -0,0 +1,31 @@
+#
+#   Copyright 2011 Red Hat, Inc.
+#
+#   Licensed under the Apache License, Version 2.0 (the "License");
+#   you may not use this file except in compliance with the License.
+#   You may obtain a copy of the License at
+#
+#       http://www.apache.org/licenses/LICENSE-2.0
+#
+#   Unless required by applicable law or agreed to in writing, software
+#   distributed under the License is distributed on an "AS IS" BASIS,
+#   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#   See the License for the specific language governing permissions and
+#   limitations under the License.
+#
+require 'spec_helper'
+require 'timecop'
+
+describe Aeolus::Image::Factory::Builder do
+  VCR::Cassette.new 'builder', :record => :new_episodes, :match_requests_on => [:method, :uri, :body]
+  it "should get builder object" do
+    Aeolus::Image::Factory::Base.config = {
+      :site => 'http://127.0.0.1:8075/imagefactory'
+    }
+    builder = Aeolus::Image::Factory::Builder.first
+    builder.builders.should_not be_empty
+    build = builder.builders.first
+    builder.find_active_build(build.build_id, build.target).should_not be_nil
+    builder.find_active_build(build.build_id, 'invalid_target').should be_nil
+  end
+end

data/spec/models/factory/provider_image_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+module Aeolus
+  module Image
+    module Factory
+      describe ProviderImage do
+        it "should return nil when a builder is found but operation is build" do
+          @builder = mock(Builder, :operation => "build", :status => "PUSHING")
+          Builder.stub!(:find).and_return(@builder)
+          ProviderImage.status("1234").should == nil
+        end
+
+        it "should return a builder when a builder is found and operation is push" do
+          @builder = mock(Builder, :operation => "push", :status => "PUSHING")
+          Builder.stub!(:find).and_return(@builder)
+          ProviderImage.status("1234").should == "PUSHING"
+        end
+      end
+    end
+  end
+end

data/spec/models/factory/target_image_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+module Aeolus
+  module Image
+    module Factory
+      describe TargetImage do
+        it "should return nil when a builder is found but operation is push" do
+          @builder = mock(Builder, :operation => "push")
+          Builder.stub!(:find).and_return(@builder)
+          TargetImage.status("1234").should == nil
+        end
+
+        it "should return a builder when a builder is found and operation is build" do
+          @builder = mock(Builder, :operation => "build", :status => "BUILDING")
+          Builder.stub!(:find).and_return(@builder)
+          TargetImage.status("1234").should == "BUILDING"
+        end
+      end
+    end
+  end
+end

data/spec/models/warehouse/image_build_spec.rb

@@ -0,0 +1,189 @@
+#   Copyright 2011 Red Hat, Inc.
+#
+#   Licensed under the Apache License, Version 2.0 (the "License");
+#   you may not use this file except in compliance with the License.
+#   You may obtain a copy of the License at
+#
+#       http://www.apache.org/licenses/LICENSE-2.0
+#
+#   Unless required by applicable law or agreed to in writing, software
+#   distributed under the License is distributed on an "AS IS" BASIS,
+#   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#   See the License for the specific language governing permissions and
+#   limitations under the License.
+
+require 'spec_helper'
+
+module Aeolus
+  module Image
+    module Warehouse
+      describe ImageBuild do
+        before(:each) do
+          @image_build_attributes = {
+            :image => 'image',
+            :uuid => 'uuid',
+            :template => 'template',
+            :other_attribute => 'other_attribute',
+            :another_attribute => 'another_attribute'
+          }
+          @object = mock(Object, :attr_list => @image_build_attributes.keys, :attrs => @image_build_attributes)
+          @image_build_attributes.each do |key, value|
+            @object.stub(key.to_sym, value)
+          end
+          @image_build = ImageBuild.new(@object)
+
+          @target_image_mock_with_no_build = mock(TargetImage, :build => nil, :provider_images => [mock(ProviderImage)])
+          @target_image_mock_with_correct_build = mock(TargetImage, :build => @image_build, :provider_images => [mock(ProviderImage)])
+          @other_target_image_mock_with_correct_build = mock(TargetImage, :build => @image_build, :provider_images => [mock(ProviderImage)])
+
+          @other_image_build_attributes = @image_build_attributes.merge(:uuid => 'other_uuid')
+          @other_object = mock(Object, :attr_list => @other_image_build_attributes.keys, :attrs => @other_image_build_attributes)
+          @other_image_build_attributes.each do |key, value|
+            @other_object.stub(key.to_sym, value)
+          end
+          @other_image_build = ImageBuild.new(@other_object)
+
+          @target_image_mock_with_other_image_build = mock(TargetImage, :build => @other_image_build, :provider_images => [mock(ProviderImage)] * 2)
+
+          @all_target_images = [ @target_image_mock_with_no_build, @target_image_mock_with_correct_build, @other_target_image_mock_with_correct_build, @target_image_mock_with_other_image_build ]
+
+          @correct_target_images = [ @target_image_mock_with_correct_build, @other_target_image_mock_with_correct_build ]
+
+          @first_target_image_provider_images = []
+          3.times do
+            @first_target_image_provider_images << mock(ProviderImage, :target_image => @target_image_mock_with_correct_build)
+          end
+
+          @second_target_image_provider_images = []
+          3.times do
+            @second_target_image_provider_images << mock(ProviderImage, :target_image => @other_target_image_mock_with_correct_build)
+          end
+
+          @incorrect_target_image_provider_images = []
+          3.times do
+            @incorrect_target_image_provider_images << mock(ProviderImage, :target_image => @target_image_mock_with_other_image_build)
+          end
+
+          @correct_provider_images = @first_target_image_provider_images + @second_target_image_provider_images
+          @incorrect_provider_images = @incorrect_target_image_provider_images
+          @all_provider_images = @correct_provider_images + @incorrect_provider_images
+
+        end
+
+        context "@bucket_name" do
+
+          it "should be set correctly" do
+            # accessor set in WarehouseModel
+            @image_build.class.bucket_name.should be_eql('builds')
+          end
+
+        end
+
+        context "#initialize" do
+
+          before(:each) do
+            @attr_writers = [ :image ]
+            @attr_accessors = @image_build_attributes.keys - @attr_writers
+          end
+
+          it "should correctly set attribute writers" do
+            @attr_writers.each do |writer|
+              @image_build.respond_to?(:"#{writer.to_s}=").should be_true
+            end
+          end
+
+          it "should correctly set attribute accessors" do
+            @attr_accessors.each do |accessor|
+              @image_build.respond_to?(:"#{accessor.to_s}").should be_true
+              @image_build.respond_to?(:"#{accessor.to_s}=").should be_true
+            end
+          end
+
+          it "should set attributes to correct values" do
+            @attr_accessors.each do |key|
+              @image_build.send(:"#{key.to_s}").should be_equal(@image_build_attributes[key])
+            end
+          end
+
+        end
+
+        #TODO: implement this test
+        context ".find_all_by_image_uuid" do
+
+        end
+
+        context "#image" do
+          context "with @image present" do
+            before(:each) do
+              @image_mock = mock(Image)
+              Image.stub(:find).and_return(@image_mock)
+            end
+            it "should call Image.find with correct parameter" do
+              Image.should_receive(:find).with(@image_build.instance_variable_get( :@image ))
+              @image_build.image
+            end
+
+            it "should return found Image" do
+              @image_build.image.should be_eql(@image_mock)
+            end
+          end
+
+          context "with @image absent" do
+            before(:each) do
+              @image_build.instance_variable_set(:@image, nil)
+            end
+            it "should not call Image.find at all" do
+              Image.should_not_receive(:find)
+              @image_build.image
+            end
+          end
+        end
+
+        context "#target_images" do
+          before(:each) do
+            TargetImage.stub(:where).and_return(@all_target_images)
+          end
+
+          context "should return collection" do
+            it "with correct target image" do
+              @image_build.target_images.should include(@target_image_mock_with_correct_build)
+            end
+            it "with other correct target image" do
+              @image_build.target_images.should include(@other_target_image_mock_with_correct_build)
+            end
+          end
+        end
+
+        context "#provider_images" do
+          before(:each) do
+            TargetImage.stub(:where).and_return(@all_target_images)
+          end
+
+          context "should return collection" do
+            it "with correct amount of provider images" do
+              @image_build.provider_images.size.should == 5
+            end
+          end
+        end
+
+        context "#delete!" do
+
+          before(:each) do
+            @correct_target_images.each{|ti| ti.stub(:delete!)}
+            @image_build.stub(:target_images).and_return(@correct_target_images)
+            ImageBuild.stub(:delete)
+          end
+          it "should delete all associated provider images" do
+            @correct_target_images.each{|ti| ti.should_receive(:delete!)}
+            @image_build.delete!
+          end
+          it "should call TargetImage.delete with @uuid" do
+            ImageBuild.should_receive(:delete).with(@image_build.instance_variable_get(:@uuid))
+            @image_build.delete!
+          end
+
+        end
+      end
+    end
+  end
+end

data/spec/models/warehouse/image_spec.rb

@@ -0,0 +1,241 @@
+#   Copyright 2011 Red Hat, Inc.
+#
+#   Licensed under the Apache License, Version 2.0 (the "License");
+#   you may not use this file except in compliance with the License.
+#   You may obtain a copy of the License at
+#
+#       http://www.apache.org/licenses/LICENSE-2.0
+#
+#   Unless required by applicable law or agreed to in writing, software
+#   distributed under the License is distributed on an "AS IS" BASIS,
+#   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#   See the License for the specific language governing permissions and
+#   limitations under the License.
+
+require 'spec_helper'
+
+module Aeolus
+  module Image
+    module Warehouse
+      describe Image do
+        let(:body) { 'body' }
+        before(:each) do
+          @image_attributes = {
+            :latest_build => 'latest_build',
+            :uuid => 'uuid',
+            :body => body,
+            :other_attribute => 'other_attribute',
+            :another_attribute => 'another_attribute'
+          }
+          @object = mock(Object, :attr_list => @image_attributes.keys, :attrs => @image_attributes)
+          @image_attributes.each do |key, value|
+            @object.stub(key.to_sym, value)
+          end
+          @image = Image.new(@object)
+
+        end
+
+        context "@bucket_name" do
+
+          it "should be set correctly" do
+            # accessor set in WarehouseModel
+            @image.class.bucket_name.should be_eql('images')
+          end
+
+        end
+
+        context "#initialize" do
+
+          before(:each) do
+            @attr_writers = [ :latest_build ]
+            @attr_accessors = @image_attributes.keys - @attr_writers
+          end
+
+          it "should correctly set attribute writers" do
+            @attr_writers.each do |writer|
+              @image.respond_to?(:"#{writer.to_s}=").should be_true
+            end
+          end
+
+          it "should correctly set attribute accessors" do
+            @attr_accessors.each do |accessor|
+              @image.respond_to?(:"#{accessor.to_s}").should be_true
+              @image.respond_to?(:"#{accessor.to_s}=").should be_true
+            end
+          end
+
+          #it "should set attributes to correct values" do
+          #  @attr_accessors.each do |key|
+          #    @image.send(:"#{key.to_s}").should be_equal(@image_attributes[key])
+          #  end
+          #end
+
+        end
+
+        context "#template_xml" do
+
+          before(:each) do
+            @body = "<template><item>X</item></template>"
+            @template_xml = Nokogiri::XML(@body)
+
+            @empty_body = "<template></template>"
+            @empty_template_xml = nil
+          end
+
+          context "with correct associated objects" do
+
+            before(:each) do
+              @target_template = mock(Template, :body => @template_xml.to_s)
+              @target_image = [ mock(TargetImage, :target_template => @target_template) ]
+              @image_builds = [ mock(ImageBuild, :target_images => @target_image, :provider_images => [mock(ProviderImage)] * 2) ] * 2
+              @image.stub(:image_builds).and_return(@image_builds)
+            end
+
+        context "#provider_images" do
+          before(:each) do
+            ImageBuild.stub(:where).and_return(@image_builds)
+          end
+
+          context "should return collection" do
+            it "with correct amount of provider images" do
+              @image.provider_images.size.should == 4
+            end
+          end
+        end
+
+            # TODO: There shoud be a way to test equality of Nokogiri::XML documents better than string comparison
+            it "should return correct template" do
+              @image.template_xml.to_s.should be_eql(@template_xml.to_s)
+            end
+
+          end
+
+          context "with incorect associated object" do
+
+            before(:each) do
+              @image.stub(:image_builds).and_return(nil)
+            end
+
+            # TODO: There shoud be a way to test equality of Nokogiri::XML documents better than string comparison
+            it "should return empty template" do
+              @image.template_xml.to_s.should be_eql(@empty_template_xml.to_s)
+            end
+          end
+        end
+
+        context "#name" do
+          subject { @image.name }
+          context "with /image/name text value defined in body" do
+            let(:body) { "<image><name>image-name-string</name></image>" }
+            before(:each) do
+              @image.instance_variable_set(:@xml_body, Nokogiri::XML(body))
+            end
+
+            it "should return /image/name text value" do
+              puts subject
+              should be_eql('image-name-string')
+            end
+          end
+
+          context "without /image/name text value defined in body" do
+            before(:each) do
+              @image.stub(:template_xml).and_return(Nokogiri::XML(template_xml))
+            end
+            context "with /template/name text value from template_xml" do
+              let(:template_xml) { "<template><name>template-name-string</name></template>" }
+              it "should return /template/name text value" do
+                should be_eql('template-name-string')
+              end
+            end
+
+            context "without /template/name text value from template_xml" do
+              let(:template_xml) { "<template></template>" }
+              it "should return empty string" do
+                should be_eql("")
+              end
+            end
+          end
+        end
+
+        context "#os" do
+          subject { @image.os }
+          let(:template_xml) { "<template><os><name>template-os-name</name><version>template-os-version</version><arch>template-os-arch</arch></os></template>" }
+          before(:each) do
+            @image.stub(:template_xml).and_return(Nokogiri::XML(template_xml))
+          end
+
+          it "should return correct OS struct" do
+            subject.name.should be_eql("template-os-name")
+            subject.version.should be_eql("template-os-version")
+            subject.arch.should be_eql("template-os-arch")
+          end
+        end
+
+        context "#description" do
+
+          subject { @image.description }
+          let(:template_xml) { "<template><description>template-description-string</description></template>" }
+
+          before(:each) do
+            @image.stub(:template_xml).and_return(Nokogiri::XML(template_xml))
+          end
+
+          it "should return /template/description text value from template_xml" do
+            should be_eql("template-description-string")
+          end
+
+        end
+
+        context "#latest_pushed_build" do
+
+          context "with @latest_build present" do
+            before(:each) do
+              @latest_build_mock = mock(ImageBuild)
+              ImageBuild.stub(:find).and_return(@latest_build_mock)
+            end
+            it "should call ImageBuild.find with correct parameter" do
+              ImageBuild.should_receive(:find).with(@image.instance_variable_get( :@latest_build ))
+              @image.latest_pushed_build
+            end
+
+            it "should return found ImageBuild" do
+              @image.latest_pushed_build.should be_eql(@latest_build_mock)
+            end
+          end
+
+          context "with @latest_build absent" do
+            before(:each) do
+              @image.instance_variable_set(:@latest_build, nil)
+            end
+            it "should not call ImageBuild.find at all" do
+              ImageBuild.should_not_receive(:find)
+              @image.latest_pushed_build
+            end
+          end
+        end
+
+        context "#delete!" do
+
+          before(:each) do
+            @image_builds = [ mock(ImageBuild) ]
+            @image_builds.each{|ib| ib.stub(:delete!)}
+            @image.stub(:image_builds).and_return(@image_builds)
+            Image.stub(:delete)
+          end
+
+          it "should delete all associated image builds" do
+            @image_builds.each{|im| im.should_receive(:delete!)}
+            @image.delete!
+          end
+
+          it "should call Image.delete with @uuid" do
+            Image.should_receive(:delete).with(@image.instance_variable_get(:@uuid))
+            @image.delete!
+          end
+
+        end
+
+      end
+    end
+  end
+end