tim 0.0.1

data/spec/controllers/templates_controller_spec.rb

@@ -0,0 +1,41 @@
+require "spec_helper"
+
+module Tim
+  describe TemplatesController do
+    render_views
+
+    describe "Templates API" do
+
+      before(:each) do
+        send_and_accept_xml
+        TemplatesController.any_instance.stub(:template_exists?).and_return false
+      end
+
+      describe "Create Template" do
+
+        context "Success" do
+           it "should return a new template as xml" do
+             template = FactoryGirl.build(:template)
+             request.env['RAW_POST_DATA'] = template.xml
+             post :create
+
+             response.code.should == "201"
+             template_xml = ::Nokogiri::XML::Document.parse(response.body)
+             template_xml.xpath("//template/*").to_xml
+               .include?(template.xml_elements).should == true
+             template_xml.xpath("//template/@*").map { |node| node.name}
+               .should =~ ["id", "href"]
+          end
+        end
+
+        context "failure" do
+          it "should return a unprocessable entity error when the client sends invalid content" do
+            request.env['RAW_POST_DATA'] = "<invalid_template></invalid_template>"
+            post :create
+            response.code.should == "422"
+          end
+        end
+      end
+    end
+  end
+end

data/spec/decorators/controllers/templates_controller_decorator_spec.rb

@@ -0,0 +1,13 @@
+require "spec_helper"
+
+module Tim
+  describe TemplatesController, :type => :controller do
+    describe "Controller Decorator Pattern" do
+      it "should allow overriding of instance variables in before filter" do
+        Tim::Template.stub(:limit).and_return("test")
+        get :index
+        controller.instance_variable_get(:@templates).should == "test"
+      end
+    end
+  end
+end

data/spec/factories/tim/base_image.rb

@@ -0,0 +1,10 @@
+FactoryGirl.define do
+  factory :base_image, :class => Tim::BaseImage do
+    name "test base image"
+    description "description of test image"
+  end
+
+  factory :base_image_with_template, :parent => :base_image do
+    association :template, :factory => :template
+  end
+end

data/spec/factories/tim/image_factory/provider_image.rb

@@ -0,0 +1,10 @@
+FactoryGirl.define do
+  factory :image_factory_provider_image, :class => Tim::ImageFactory::ProviderImage do
+    status "New"
+    id "4cc3b024-5fe7-4b0b-934b-c5d463b990b0"
+    percent_complete "10"
+    provider "MockSphere"
+    identifier_on_provider "mock-123456"
+    provider_account_identifier "mock-account-123"
+  end
+end

data/spec/factories/tim/image_factory/target_image.rb

@@ -0,0 +1,7 @@
+FactoryGirl.define do
+  factory :image_factory_target_image, :class => Tim::ImageFactory::TargetImage do
+    status "NEW"
+    id '4cc3b024-5fe7-4b0b-934b-c5d463b990b0'
+    percent_complete '0'
+  end
+end

data/spec/factories/tim/image_version.rb

@@ -0,0 +1,20 @@
+module Tim
+  FactoryGirl.define do
+    factory :image_version, :class => Tim::ImageVersion do
+    end
+
+    factory :image_version_with_base_image, :parent => :image_version do
+      association :base_image, :factory => :base_image
+    end
+
+    factory :image_version_with_full_tree, :parent => :image_version do
+      association :base_image, :factory => :base_image_with_template
+    end
+
+    factory :image_version_with_target_images, :parent => :image_version_with_full_tree do
+      after(:create) do |image_version, evaluator|
+        FactoryGirl.create_list(:target_image, 2, :image_version => image_version)
+      end
+    end
+  end
+end

data/spec/factories/tim/provider_image.rb

@@ -0,0 +1,9 @@
+FactoryGirl.define do
+  factory :provider_image, :class => Tim::ProviderImage do
+    association :target_image, :factory => :target_image
+  end
+
+  factory :provider_image_with_full_tree, :parent => :provider_image do
+    association :target_image, :factory => :target_image_with_full_tree
+  end
+end

data/spec/factories/tim/target_image.rb

@@ -0,0 +1,10 @@
+FactoryGirl.define do
+  factory :target_image, :class => Tim::TargetImage do
+    association :image_version, :factory => :image_version
+    target 'Mock'
+  end
+
+  factory :target_image_with_full_tree, :parent => :target_image do
+    association :image_version, :factory => :image_version_with_full_tree
+  end
+end

data/spec/factories/tim/template.rb

@@ -0,0 +1,23 @@
+FactoryGirl.define do
+  factory :template, :class => Tim::Template do
+    xml "<template>
+           <name>Fedora 15</name>
+             <description>desc</description>
+             <os>
+             <rootpw>password</rootpw>
+              <name>Fedora</name>
+              <arch>x86_64</arch>
+              <version>15</version>
+              <install type='url'>
+                <url>http://download.fedoraproject.org/pub/fedora/linux/releases/15/Fedora/x86_64/os/</url>
+              </install>
+            </os>
+            <repositories>
+              <repository name='custom'>
+                <url>http://repos.fedorapeople.org/repos/aeolus/demo/webapp/</url>
+                <signed>false</signed>
+              </repository>
+            </repositories>
+          </template>"
+  end
+end

data/spec/filters/user_keys_filter_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+module Tim
+  describe UserKeysFilter do
+
+    before(:all) do |spec|
+      @user_keys = {:k1 => :a1, :k2 => :a2}
+    end
+
+    describe "replace user keys" do
+      it 'should replace user keys with respective specified alternate' do
+        hash = UserKeysFilter.replace_user_keys({:k1 => :v1, :k2 => :v2}, @user_keys)
+        hash[:a1].should == :v1
+        hash[:a2].should == :v2
+        hash.has_key?(:k1).should == false
+        hash.has_key?(:k2).should == false
+      end
+
+      it 'should replace nested user keys' do
+        hash = UserKeysFilter.replace_user_keys({:k1 => {:k2 => :v2}}, @user_keys)
+        hash[:a1].should == {:a2 => :v2}
+      end
+    end
+
+    describe "replace user keys" do
+      it 'should get the correct hash from params via controller name' do
+         request = mock("Request")
+         request.stub(:format).and_return "application/xml"
+         controller = double("ApplicationController")
+         controller.stub(:request).and_return request
+         controller.stub(:controller_name).and_return "resources"
+         controller.stub(:params).and_return({:resource => {:k1 => {:k2 => :v2}}})
+
+         UserKeysFilter.should_receive(:replace_user_keys).with({:k1 => {:k2 => :v2}}, nil)
+         UserKeysFilter.stub(:replace_user_keys).and_return({:a1 => {:a2 => :v2}}, nil)
+
+         UserKeysFilter.before(controller)
+      end
+    end
+  end
+end

data/spec/fixtures/template.xml

@@ -0,0 +1,13 @@
+<template>
+  <name>mock</name>
+  <description>Mock Template</description>
+  <os>
+    <name>RHELMock</name>
+    <version>1</version>
+    <arch>x86_64</arch>
+    <install type=\"iso\">
+      <iso>http://mockhost/RHELMock1-x86_64-DVD.iso</iso>
+    </install>
+    <rootpw>password</rootpw>
+  </os>
+</template>

data/spec/models/base_image_spec.rb

@@ -0,0 +1,39 @@
+require 'spec_helper'
+
+module Tim
+  describe BaseImage do
+    describe "Model relationships" do
+      it 'should have many image versions' do
+        base_image = BaseImage.new
+        2.times do
+          base_image.image_versions << ImageVersion.new
+        end
+        base_image.save!
+        BaseImage.find(base_image).image_versions.size.should == 2
+      end
+
+      it "should have one template" do
+        base_image = BaseImage.new
+        base_image.template = FactoryGirl.build(:template)
+        base_image.save!
+        BaseImage.find(base_image).template.should == base_image.template
+      end
+    end
+
+    describe "Dummy model relationships" do
+      it "should have one pool family" do
+        base_image = BaseImage.new
+        base_image.pool_family = PoolFamily.new
+        base_image.save!
+        BaseImage.find(base_image).pool_family.should == base_image.pool_family
+      end
+
+      it "should have one user" do
+        base_image = BaseImage.new
+        base_image.user = User.new
+        base_image.save!
+        BaseImage.find(base_image).user.should == base_image.user
+      end
+    end
+  end
+end

data/spec/models/dummy/pool_family_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+
+describe PoolFamily do
+  describe "Dummy Model relationships" do
+    it 'should have many base images' do
+      pool_family = PoolFamily.new
+      2.times do
+        pool_family.base_images << Tim::BaseImage.new
+      end
+      pool_family.save!
+      PoolFamily.find(pool_family).base_images.size.should == 2
+    end
+  end
+end

data/spec/models/dummy/provider_account_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+describe ProviderAccount do
+  describe "Dummy Model relationships" do
+    before(:each) do
+      Tim::ProviderImage.any_instance.stub(:create_factory_provider_image).and_return(true)
+    end
+
+    it 'should have many provider images' do
+      provider_account = ProviderAccount.new
+      2.times do
+        provider_account.provider_images << Tim::ProviderImage.new
+      end
+      provider_account.save!
+      ProviderAccount.find(provider_account).provider_images.size.should == 2
+    end
+  end
+end

data/spec/models/dummy/provider_type_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+describe ProviderType do
+  describe "Dummy Model relationships" do
+    before (:each) do
+      Tim::TargetImage.any_instance.stub(:create_factory_target_image).and_return(true)
+    end
+
+    it 'should have many target images' do
+      provider_type = ProviderType.new
+      2.times do
+        provider_type.target_images << Tim::TargetImage.new
+      end
+      provider_type.save!
+      ProviderType.find(provider_type).target_images.size.should == 2
+    end
+  end
+end

data/spec/models/dummy/user_spec.rb

@@ -0,0 +1,14 @@
+require 'spec_helper'
+
+describe User do
+  describe "Dummy Model relationships" do
+    it 'should have many base images' do
+      user = User.new
+      2.times do
+        user.base_images << Tim::BaseImage.new
+      end
+      user.save!
+      User.find(user).base_images.size.should == 2
+    end
+  end
+end

data/spec/models/image_version_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+module Tim
+  describe ImageVersion do
+    describe "Model relationships" do
+      before (:each) do
+        TargetImage.any_instance.stub(:create_factory_target_image).and_return(true)
+      end
+
+      it 'should have one base image' do
+        image_version = ImageVersion.new
+        image_version.base_image = BaseImage.new
+        image_version.save!
+        ImageVersion.find(image_version).base_image.should == image_version.base_image
+      end
+
+      it 'should have many target images' do
+        image_version = ImageVersion.new
+        2.times do
+          image_version.target_images << TargetImage.new
+        end
+        image_version.save!
+        ImageVersion.find(image_version).target_images.size.should == 2
+      end
+    end
+  end
+end

data/spec/models/provider_image_spec.rb

@@ -0,0 +1,69 @@
+require 'spec_helper'
+
+module Tim
+  describe ProviderImage do
+    before (:each) do
+      TargetImage.any_instance.stub(:create_factory_target_image).and_return(true)
+    end
+
+    describe "Model relationships" do
+      it 'should have one target image' do
+        provider_image = ProviderImage.new
+        provider_image.target_image = TargetImage.new
+        provider_image.stub(:create_factory_provider_image).and_return(true)
+        provider_image.save!
+        ProviderImage.find(provider_image).target_image.should == provider_image.target_image
+      end
+    end
+
+    describe "Dummy model relationships" do
+      it "should have one provider account" do
+        provider_image = ProviderImage.new
+        provider_image.provider_account = ProviderAccount.new
+        provider_image.stub(:create_factory_provider_image).and_return(true)
+        provider_image.save!
+        ProviderImage.find(provider_image).provider_account.should == provider_image.provider_account
+      end
+    end
+
+    describe "ImageFactory interactions" do
+      describe "Successful Requests" do
+        before(:each) do
+          ImageFactory::ProviderImage.any_instance.stub(:save!)
+          TargetImage.any_instance.stub(:factory_id).and_return("1234")
+        end
+
+        it "should create factory provider image and populate fields" do
+          # Needed due to ActiveRecord::Associations::Association#target
+          # name conflict
+          target = mock(:target)
+          target.stub(:target).and_return("mock")
+
+          pi = FactoryGirl.build(:provider_image)
+          pi.target_image.target = target
+          pi.should_receive(:populate_factory_fields)
+          pi.save
+        end
+
+        it "should add factory fields to provider image" do
+          status_detail = mock(:status_detail)
+          status_detail.stub(:activity).and_return("Building")
+
+          pi = FactoryGirl.build(:provider_image)
+          pi.send(:populate_factory_fields, FactoryGirl
+                                .build(:image_factory_provider_image,
+                                       :status_detail => status_detail))
+
+          pi.factory_id.should == "4cc3b024-5fe7-4b0b-934b-c5d463b990b0"
+          pi.status.should == "New"
+          pi.status_detail.should == "Building"
+          pi.progress.should == "10"
+          pi.provider.should == "MockSphere"
+          pi.external_image_id.should == "mock-123456"
+          pi.provider_account_id.should == "mock-account-123"
+        end
+
+      end
+    end
+  end
+end

data/spec/models/target_image_spec.rb

@@ -0,0 +1,75 @@
+require 'spec_helper'
+
+module Tim
+  describe TargetImage do
+
+    #TODO FIX: A bug in RSpec V2.1 means that any_instance propogates across context therefore we are  stubbing each 
+    #target_image.   This bug is fixed in RSpec V2.6 
+    describe "Model relationships" do
+      it 'should have one images version' do
+        target_image = TargetImage.new
+        target_image.stub(:create_factory_target_image).and_return(true)
+        target_image.image_version = ImageVersion.new
+        target_image.save!
+        TargetImage.find(target_image).image_version.should == target_image.image_version
+      end
+
+      it 'should have many provider images' do
+        target_image = TargetImage.new
+        target_image.stub(:create_factory_target_image).and_return(true)
+        ProviderImage.any_instance.stub(:create_factory_provider_image).and_return(true)
+        2.times do
+          target_image.provider_images << ProviderImage.new
+        end
+        target_image.save!
+        TargetImage.find(target_image).provider_images.size.should == 2
+      end
+    end
+
+    describe "Dummy model relationships" do
+      it "should have one provider type" do
+        target_image = TargetImage.new
+        target_image.stub(:create_factory_target_image).and_return(true)
+        target_image.provider_type = ProviderType.new
+        target_image.save!
+        TargetImage.find(target_image).provider_type.should == target_image.provider_type
+      end
+    end
+
+    describe "ImageFactory interactions" do
+      describe "Successful Requests" do
+        before(:each) do
+          ImageFactory::TargetImage.any_instance.stub(:save!)
+        end
+
+        it "should create factory provider image and populate fields" do
+          # Needed due to ActiveRecord::Associations::Association#target
+          # name conflict
+          target = mock(:target)
+          target.stub(:target).and_return("mock")
+
+          ti = FactoryGirl.build(:target_image_with_full_tree)
+          ti.target = target
+          ti.should_receive(:populate_factory_fields)
+          ti.save
+        end
+
+        it "should add factory fields to provider image" do
+          status_detail = mock(:status_detail)
+          status_detail.stub(:activity).and_return("Building")
+
+          ti = FactoryGirl.build(:target_image_with_full_tree)
+          ti.send(:populate_factory_fields, FactoryGirl
+                                .build(:image_factory_target_image,
+                                       :status_detail => status_detail))
+
+          ti.factory_id.should == "4cc3b024-5fe7-4b0b-934b-c5d463b990b0"
+          ti.status.should == "NEW"
+          ti.status_detail.should == "Building"
+          ti.progress.should == "0"
+        end
+
+      end
+    end
+  end
+end