boxgrinder-build 0.9.2 → 0.9.3

data/lib/boxgrinder-build/plugins/os/sl/sl-plugin.rb

@@ -0,0 +1,56 @@
+#
+# Copyright 2010 Red Hat, Inc.
+#
+# This is free software; you can redistribute it and/or modify it
+# under the terms of the GNU Lesser General Public License as
+# published by the Free Software Foundation; either version 3 of
+# the License, or (at your option) any later version.
+#
+# This software is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+# Lesser General Public License for more details.
+#
+# You should have received a copy of the GNU Lesser General Public
+# License along with this software; if not, write to the Free
+# Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+# 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+
+require 'boxgrinder-build/plugins/os/rhel/rhel-plugin'
+
+module BoxGrinder
+  class ScientificLinuxPlugin < RHELPlugin
+
+    SL_REPOS = {
+     "5" => {
+       "base" => {
+         "baseurl" => "http://ftp.scientificlinux.org/linux/scientific/#OS_VERSION#x/#BASE_ARCH#/SL/"
+        },
+       "security" => {
+         "baseurl" => "http://ftp.scientificlinux.org/linux/scientific/#OS_VERSION#x/#BASE_ARCH#/updates/security/"
+        }
+      },
+     "6" => {
+       "base" => {
+         "baseurl" => "http://ftp.scientificlinux.org/linux/scientific/#OS_VERSION#x/#BASE_ARCH#/os/"
+        },
+       "security" => {
+         "baseurl" => "http://ftp.scientificlinux.org/linux/scientific/#OS_VERSION#x/#BASE_ARCH#/updates/security/"
+        }
+      }
+    }
+
+    def after_init
+      super
+      register_supported_os('sl', ['5', '6'])
+    end
+
+    def execute(appliance_definition_file)
+      build_rhel(appliance_definition_file, SL_REPOS)
+    end
+
+  end
+end
+
+plugin :class => BoxGrinder::ScientificLinuxPlugin, :type => :os, :name => :sl, :full_name  => "Scientific Linux", :versions   => ["5", "6"]
+

data/lib/boxgrinder-build/plugins/platform/ec2/ec2-plugin.rb

@@ -27,6 +27,7 @@ module BoxGrinder
 
       register_supported_os('fedora', ['13', '14', '15'])
       register_supported_os('centos', ['5'])
+      register_supported_os('sl', ['5', '6'])
       register_supported_os('rhel', ['5', '6'])
     end
 

data/lib/boxgrinder-build/plugins/platform/ec2/src/rc_local

@@ -1,19 +1,26 @@
-curl http://169.254.169.254/2009-04-04/meta-data/public-keys/0/openssh-key 2>/dev/null >/tmp/my-key
+key=`curl http://169.254.169.254/2009-04-04/meta-data/public-keys/0/openssh-key 2>/dev/null`
 
 if [ $? -eq 0 ] ; then
-   for home in `find /home/* -maxdepth 0 -type d 2>/dev/null | tr '\n' ' '`; do
-       user=`echo $home | awk -F '/' '{ print $3 }'`
+    for home in `find /home/* -maxdepth 0 -type d 2>/dev/null | tr '\n' ' '`; do
+        user=`echo $home | awk -F '/' '{ print $3 }'`
 
-       if [ ! -d $home/.ssh ] ; then
+        if [ ! -d $home/.ssh ] ; then
             mkdir -p $home/.ssh
             chmod 700 $home/.ssh
             chown $user $home/.ssh
         fi
 
-       cat /tmp/my-key >> $home/.ssh/authorized_keys
-       chmod 600 $home/.ssh/authorized_keys
-       chown $user $home/.ssh/authorized_keys
+        actual_key=`echo $key | awk '{ print $2 }'`
 
+        if [ -f $home/.ssh/authorized_keys ]; then
+            if [ "`grep $actual_key $home/.ssh/authorized_keys | wc -l`" -gt 0 ]; then
+                continue
+            fi
+        fi
+
+        echo $key >> $home/.ssh/authorized_keys
+        chmod 600 $home/.ssh/authorized_keys
+        chown $user $home/.ssh/authorized_keys
    done
-   rm /tmp/my-key
 fi
+

data/lib/boxgrinder-build/plugins/platform/vmware/vmware-plugin.rb

@@ -21,10 +21,7 @@ require 'boxgrinder-build/plugins/base-plugin'
 module BoxGrinder
   class VMwarePlugin < BasePlugin
     def after_init
-      set_default_config_value('thin_disk', false)
-      validate_plugin_config(['type'], 'http://boxgrinder.org/tutorials/boxgrinder-build-plugins/#VMware_Platform_Plugin')
-
-      register_deliverable(:vmx    => "#{@appliance_config.name}.vmx",
+      register_deliverable(:vmx => "#{@appliance_config.name}.vmx",
                            :readme => "README")
 
       if @plugin_config['type'].eql?('personal') and @plugin_config['thin_disk']
@@ -35,6 +32,11 @@ module BoxGrinder
       end
     end
 
+    def validate
+      set_default_config_value('thin_disk', false)
+      validate_plugin_config(['type'], 'http://boxgrinder.org/tutorials/boxgrinder-build-plugins/#VMware_Platform_Plugin')
+    end
+
     def execute
       @log.info "Converting image to VMware #{@plugin_config['type']} format..."
 
@@ -79,7 +81,7 @@ module BoxGrinder
         end
       end
 
-      c             = disk_size / (h*s*512)
+      c = disk_size / (h*s*512)
       total_sectors = disk_size / 512
 
       [c.to_i, h.to_i, s.to_i, total_sectors.to_i]
@@ -188,4 +190,4 @@ module BoxGrinder
   end
 end
 
-plugin :class => BoxGrinder::VMwarePlugin, :type => :platform, :name => :vmware, :full_name  => "VMware"
+plugin :class => BoxGrinder::VMwarePlugin, :type => :platform, :name => :vmware, :full_name => "VMware"

data/rubygem-boxgrinder-build.spec

@@ -5,7 +5,7 @@
 
 Summary: A tool for creating appliances from simple plain text files
 Name: rubygem-%{gemname}
-Version: 0.9.2
+Version: 0.9.3
 Release: 1%{?dist}
 Group: Development/Languages
 License: LGPLv3+
@@ -22,8 +22,6 @@ Requires: e2fsprogs
 # Fix for rubygem-aws package
 Requires: rubygem(activesupport)
 
-# For EL5
-BuildRoot: %(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XXXXXX)
 BuildArch: noarch
 
 BuildRequires: rubygem(rake)
@@ -143,12 +141,34 @@ popd
 %files doc
 %defattr(-, root, root, -)
 %{geminstdir}/spec
+%{geminstdir}/integ
 %{geminstdir}/Rakefile
 %{geminstdir}/rubygem-%{gemname}.spec
 %{geminstdir}/%{gemname}.gemspec
 %{gemdir}/doc/%{gemname}-%{version}
 
 %changelog
+* Fri Jun 17 2011 Marc Savy <msavy@redhat.com> - 0.9.3-1
+- Upstream release: 0.9.3
+- [BGBUILD-232] boxgrinder doesn't validate config early enough
+- [BGBUILD-237] Tilde characters break creation of yum.conf
+- [BGBUILD-223] BoxGrinder hangs because qemu.wrapper does not detect x86_64 properly on CentOS 5.6
+- [BGBUILD-241] Add Scientific Linux support
+- [BGBUILD-220] Group names have spaces (to the user), this breaks schema rules for packages
+- [BGBUILD-222] Allow overwrite of uploaded ec2 image
+- [BGBUILD-225] Move PAE configuration parameter to operating system configuration
+- [BGBUILD-224] EBS Plugin Support for CentOS v5.5 and fix for non-integer EBS disk sizes
+- [BGBUILD-231] Cannot register Fedora 15 EC2 AMI with S3 delivery plugin in eu-west-1 availability zone
+- [BGBUILD-193] EBS delivery plugin timing/concurrency issues
+- [BGBUILD-247] ap-northeast-1 end-point is missing in S3 plugin (added Tokyo region)
+- [BGBUILD-251] Add ap-northeast-1 (tokyo) region for EBS plugin
+- [BGBUILD-248] Throw error in S3 plugin if invalid region is specified
+- [BGBUILD-252] rc.local script fills ~/.ssh/authorized_keys with a duplicate key every boot
+- [BGBUILD-250] EBS plugin incorrectly determines that non-US regions are not EC2 instances
+- [BGBUILD-254] Not able to deliver EBS AMIs to regions other than us-east-1
+- [BGBUILD-260] Wrong EC2 discovery causing libguestfs errors on non US regions
+- [BGBUILD-261] Decrease amount of debug log when downloading or uploading file using guestfs
+
 * Thu May 05 2011 Marek Goldmann <mgoldman@redhat.com> - 0.9.2-1
 - Upstream release: 0.9.2
 - [BGBUILD-148] Add support for building CentOS/RHEL images on Fedora

data/spec/appliance-spec.rb

@@ -68,9 +68,27 @@ module BoxGrinder
       config.size.should == 1
       config[:log].should == nil
     end
+ 
+    describe ".validate_definition" do
+      before(:each) do
+        prepare_appliance
+        @appliance.instance_variable_set(:@appliance_config, prepare_appliance_config)
+      end
 
-    describe ".create" do
+      it "should raise if we use unsupported OS" do
+        PluginManager.stub(:instance).and_return(OpenCascade.new(:plugins => {:os => {:centos => {}}}))
+        lambda {
+          @appliance.validate_definition
+        }.should raise_error(RuntimeError, "Not supported operating system selected: fedora. Make sure you have installed right operating system plugin, see http://boxgrinder.org/tutorials/boxgrinder-build-plugins/#Operating_system_plugins. Supported OSes are: centos")
+      end
 
+      it "should NOT raise if we use supported OS" do
+        PluginManager.stub(:instance).and_return(OpenCascade.new(:plugins => {:os => {:fedora => {:versions => ['11']}}}))
+        @appliance.validate_definition
+      end
+    end
+
+    describe ".create" do
       it "should prepare appliance to build" do
         prepare_appliance
 
@@ -81,6 +99,7 @@ module BoxGrinder
 
         @appliance.should_receive(:read_definition)
         @appliance.should_receive(:validate_definition)
+        @appliance.should_receive(:initialize_plugins)
         @appliance.should_not_receive(:remove_old_builds)
         @appliance.should_receive(:execute_plugin_chain)
 
@@ -96,6 +115,7 @@ module BoxGrinder
         PluginHelper.should_receive(:new).with(@config, :log => @log).and_return(plugin_helper)
 
         @appliance.should_receive(:read_definition)
+        @appliance.should_receive(:initialize_plugins)
         @appliance.should_receive(:validate_definition)
         @appliance.should_receive(:remove_old_builds)
         @appliance.should_receive(:execute_plugin_chain)
@@ -199,114 +219,135 @@ module BoxGrinder
 
     it "should remove old builds" do
       prepare_appliance
-
       @appliance.instance_variable_set(:@appliance_config, prepare_appliance_config)
+
       FileUtils.should_receive(:rm_rf).with("build/path")
       @appliance.remove_old_builds
     end
 
-    it "should build base appliance" do
-      prepare_appliance
+    describe ".execute_plugin_chain" do
+      before(:each) do
+        prepare_appliance
+        @appliance.instance_variable_set(:@appliance_config, prepare_appliance_config)
+      end
 
-      @appliance.instance_variable_set(:@appliance_config, prepare_appliance_config)
+      it "should not fail when plugin chain is empty" do
+        @appliance.instance_variable_set(:@plugin_chain, [])
+        @appliance.execute_plugin_chain
+      end
 
-      os_plugin = mock('FedoraPlugin')
-      os_plugin.should_receive(:init)
-      os_plugin.should_receive(:deliverables_exists?).and_return(false)
-      os_plugin.should_receive(:run)
-      os_plugin.should_receive(:deliverables).and_return({:disk => 'abc'})
+      it "should execute the whole plugin chain" do
+        @appliance.instance_variable_set(:@plugin_chain, [{:plugin => :plugin1, :param => 'definition'}, {:plugin => :plugin2}, {:plugin => :plugin3}])
 
-      @plugin_manager.should_receive(:plugins).and_return({:os => "something"})
-      @plugin_manager.should_receive(:initialize_plugin).once.with(:os, :fedora).and_return([os_plugin, {:class => Appliance, :type => :os, :name => :fedora, :full_name => "Fedora", :versions => ["11", "12", "13", "rawhide"]}])
+        @appliance.should_receive(:execute_plugin).ordered.with(:plugin1, 'definition')
+        @appliance.should_receive(:execute_plugin).ordered.with(:plugin2, nil)
+        @appliance.should_receive(:execute_plugin).ordered.with(:plugin3, nil)
 
-      @appliance.execute_plugin_chain
+        @appliance.execute_plugin_chain
+      end
     end
 
-    it "should not build base appliance because deliverable already exists" do
-      prepare_appliance
+    describe ".initialize_plugins" do
+      it "should prepare the plugin chain to create an appliance and convert it to VMware format" do
+        prepare_appliance(:platform => :vmware)
+        @appliance.instance_variable_set(:@appliance_config, prepare_appliance_config)
 
-      @appliance.instance_variable_set(:@appliance_config, prepare_appliance_config)
+        os_plugin = mock("OSPlugin")
+        platform_plugin = mock("PlatformPlugin", :deliverables => OpenCascade.new(:disk => 'a/disk.vmdk'))
 
-      os_plugin = mock('FedoraPlugin')
-      os_plugin.should_receive(:init)
-      os_plugin.should_receive(:deliverables_exists?).and_return(true)
-      os_plugin.should_not_receive(:run)
-      os_plugin.should_receive(:deliverables).and_return({:disk => 'abc'})
+        @plugin_manager.should_receive(:initialize_plugin).with(:os, :fedora).and_return([os_plugin, "os_plugin_info"])
+        @plugin_manager.should_receive(:initialize_plugin).with(:platform, :vmware).and_return([platform_plugin, "platform_plugin_info"])
+        @plugin_manager.should_not_receive(:initialize_plugin).with(:delivery, anything)
 
-      @plugin_manager.should_receive(:plugins).and_return({:os => "something"})
-      @plugin_manager.should_receive(:initialize_plugin).once.with(:os, :fedora).and_return([os_plugin, {:class => Appliance, :type => :os, :name => :fedora, :full_name => "Fedora", :versions => ["11", "12", "13", "rawhide"]}])
+        os_plugin.should_receive(:init).with(@config, @appliance_config, "os_plugin_info", :log => @log)
+        platform_plugin.should_receive(:init).with(@config, @appliance_config, "platform_plugin_info", :log => @log, :previous_plugin => os_plugin)
 
-      @appliance.execute_plugin_chain
-    end
+        @appliance.initialize_plugins
 
-    it "should build appliance and convert it to VMware format" do
-      prepare_appliance(:platform => :vmware)
+        @appliance.plugin_chain.size.should == 2
+        @appliance.plugin_chain.last[:plugin].deliverables.should == {:disk=>"a/disk.vmdk"}
+      end
 
-      @appliance.instance_variable_set(:@appliance_config, prepare_appliance_config)
-      @appliance.should_receive(:execute_os_plugin).and_return({})
+      it "should prepare the plugin chain to create an appliance and convert it to VMware format and deliver to S3" do
+        prepare_appliance(:platform => :vmware, :delivery => :s3)
+        @appliance.instance_variable_set(:@appliance_config, prepare_appliance_config)
 
-      platform_plugin = mock('VMware Plugin')
-      platform_plugin.should_receive(:init)
-      platform_plugin.should_receive(:deliverables_exists?).and_return(false)
-      platform_plugin.should_receive(:run)
-      platform_plugin.should_receive(:deliverables).and_return({:disk => 'abc'})
+        os_plugin = mock("OSPlugin")
+        platform_plugin = mock("PlatformPlugin")
+        delivery_plugin = mock("DeliveryPlugin", :deliverables => {})
 
-      @plugin_manager.should_receive(:plugins).and_return({:platform => "something"})
-      @plugin_manager.should_receive(:initialize_plugin).once.with(:platform, :vmware).and_return([platform_plugin, {:class => Appliance, :type => :platform, :name => :vmware, :full_name => "VMware"}])
+        @plugin_manager.should_receive(:initialize_plugin).with(:os, :fedora).and_return([os_plugin, "os_plugin_info"])
+        @plugin_manager.should_receive(:initialize_plugin).with(:platform, :vmware).and_return([platform_plugin, "platform_plugin_info"])
+        @plugin_manager.should_receive(:initialize_plugin).with(:delivery, :s3).and_return([delivery_plugin, "delivery_plugin_info"])
 
-      @appliance.execute_plugin_chain
-    end
+        os_plugin.should_receive(:init).with(@config, @appliance_config, "os_plugin_info", :log => @log)
+        platform_plugin.should_receive(:init).with(@config, @appliance_config, "platform_plugin_info", :log => @log, :previous_plugin => os_plugin)
+        delivery_plugin.should_receive(:init).with(@config, @appliance_config, "delivery_plugin_info", :log => @log, :previous_plugin => platform_plugin, :type => :s3)
 
-    it "should build appliance and convert it to VMware format because deliverable already exists" do
-      prepare_appliance(:platform => :vmware)
+        @appliance.initialize_plugins
 
-      @appliance.instance_variable_set(:@appliance_config, prepare_appliance_config)
-      @appliance.should_receive(:execute_os_plugin).and_return({})
+        @appliance.plugin_chain.size.should == 3
+        @appliance.plugin_chain.last[:plugin].deliverables.size.should == 0
+      end
+
+      it "should prepare the plugin chain to create an appliance and without conversion deliver to S3" do
+        prepare_appliance(:delivery => :s3)
+        @appliance.instance_variable_set(:@appliance_config, prepare_appliance_config)
 
-      platform_plugin = mock('VMware Plugin')
-      platform_plugin.should_receive(:init)
-      platform_plugin.should_receive(:deliverables_exists?).and_return(true)
-      platform_plugin.should_not_receive(:run)
-      platform_plugin.should_receive(:deliverables).and_return({:disk => 'abc'})
+        os_plugin = mock("OSPlugin")
+        delivery_plugin = mock("DeliveryPlugin")
 
-      @plugin_manager.should_receive(:plugins).and_return({:platform => "something"})
-      @plugin_manager.should_receive(:initialize_plugin).once.with(:platform, :vmware).and_return([platform_plugin, {:class => Appliance, :type => :platform, :name => :vmware, :full_name => "VMware"}])
+        @plugin_manager.should_receive(:initialize_plugin).with(:os, :fedora).and_return([os_plugin, "os_plugin_info"])
+        @plugin_manager.should_receive(:initialize_plugin).with(:delivery, :s3).and_return([delivery_plugin, "delivery_plugin_info"])
+        @plugin_manager.should_not_receive(:initialize_plugin).with(:platform, anything)
 
-      @appliance.execute_plugin_chain
+        os_plugin.should_receive(:init).with(@config, @appliance_config, "os_plugin_info", :log => @log)
+        delivery_plugin.should_receive(:init).with(@config, @appliance_config, "delivery_plugin_info", :log => @log, :previous_plugin => os_plugin, :type => :s3)
+
+        @appliance.initialize_plugins
+      end
     end
 
-    it "should build appliance, convert it to EC2 format and deliver it using S3 ami type" do
-      prepare_appliance(:platform => :ec2, :delivery => :ami)
+    describe ".execute_plugin" do
+      before(:each) do
+        prepare_appliance
+        @appliance.instance_variable_set(:@appliance_config, prepare_appliance_config)
+      end
 
-      @appliance.instance_variable_set(:@appliance_config, prepare_appliance_config)
-      @appliance.should_receive(:execute_os_plugin).and_return({:abc => 'def'})
-      @appliance.should_receive(:execute_platform_plugin).with({:abc => 'def'}).and_return({:def => 'ghi'})
+      it "should not execute the plugin because deliverable already exists" do
+        plugin = mock('APlugin', :deliverables_exists? => true, :plugin_info => {:name => :ec2, :type => :platform})
 
-      delivery_plugin = mock('S3 Plugin')
-      delivery_plugin.should_receive(:init)
-      delivery_plugin.should_receive(:run).with(:ami)
+        @appliance.execute_plugin(plugin)
+      end
 
-      @plugin_manager.should_receive(:plugins).and_return({:delivery => "something"})
-      @plugin_manager.should_receive(:initialize_plugin).with(:delivery, :ami).and_return([delivery_plugin, {:class => Appliance, :type => :delivery, :name => :s3, :full_name => "Amazon Simple Storage Service (Amazon S3)", :types => [:s3, :cloudfront, :ami]}])
+      it "should execute the plugin" do
+        plugin = mock('APlugin', :deliverables_exists? => false, :plugin_info => {:name => :ec2, :type => :platform})
+        plugin.should_receive(:run).with(:s3)
 
-      @appliance.execute_plugin_chain
+        @appliance.execute_plugin(plugin, :s3)
+      end
     end
 
-    it "should build appliance, convert it to EC2 format and deliver it using delivery plugin with only one delivery type" do
-      prepare_appliance(:platform => :ec2, :delivery => :same)
-
-      @appliance.instance_variable_set(:@appliance_config, prepare_appliance_config)
-      @appliance.should_receive(:execute_os_plugin).and_return({:abc => 'def'})
-      @appliance.should_receive(:execute_platform_plugin).with({:abc => 'def'}).and_return({:def => 'ghi'})
+    context "preparations" do
+      it "should return true if we have selected a platform" do
+        prepare_appliance(:platform => :vmware)
+        @appliance.platform_selected?.should == true
+      end
 
-      delivery_plugin = mock('S3 Plugin')
-      delivery_plugin.should_receive(:init)
-      delivery_plugin.should_receive(:run).with(:same)
+      it "should return false if we haven't selected a platform" do
+        prepare_appliance
+        @appliance.platform_selected?.should == false
+      end
 
-      @plugin_manager.should_receive(:plugins).and_return({:delivery => "something"})
-      @plugin_manager.should_receive(:initialize_plugin).with(:delivery, :same).and_return([delivery_plugin, {:class => Appliance, :type => :delivery, :name => :same, :full_name => "A plugin"}])
+      it "should return true if we have selected a delivery" do
+        prepare_appliance(:delivery => :s3)
+        @appliance.delivery_selected?.should == true
+      end
 
-      @appliance.execute_plugin_chain
+      it "should return false if we haven't selected a delivery" do
+        prepare_appliance
+        @appliance.delivery_selected?.should == false
+      end
     end
   end
 end

data/spec/helpers/guestfs-helper-spec.rb

@@ -247,19 +247,28 @@ module BoxGrinder
 
     describe ".hw_virtualization_available?" do
       it "should return true if HW acceleration is available" do
-        Resolv.should_receive(:getname).with("169.254.169.254").and_return("blah")
+        URI.should_receive(:parse).with('http://169.254.169.254/latest/meta-data/ami-id').and_return('parsed')
+        Net::HTTP.should_receive(:get_response).with("parsed").and_return(OpenCascade.new(:code => '404'))
         @helper.should_receive(:`).with("egrep '^flags.*(vmx|svm)' /proc/cpuinfo | wc -l").and_return("2")
         @helper.hw_virtualization_available?.should == true
       end
 
       it "should return false if no vmx flag is present" do
-        Resolv.should_receive(:getname).with("169.254.169.254").and_return("blah")
+        URI.should_receive(:parse).with('http://169.254.169.254/latest/meta-data/ami-id').and_return('parsed')
+        Net::HTTP.should_receive(:get_response).with("parsed").and_return(OpenCascade.new(:code => '404'))
         @helper.should_receive(:`).with("egrep '^flags.*(vmx|svm)' /proc/cpuinfo | wc -l").and_return("0")
         @helper.hw_virtualization_available?.should == false
       end
 
       it "should return false if we're on EC2" do
-        Resolv.should_receive(:getname).with("169.254.169.254").and_return("instance-data.ec2.internal")
+        URI.should_receive(:parse).with('http://169.254.169.254/latest/meta-data/ami-id').and_return('parsed')
+        Net::HTTP.should_receive(:get_response).with("parsed").and_return(OpenCascade.new(:code => '200'))
+        @helper.hw_virtualization_available?.should == false
+      end
+
+      it "should return false if we're NOT on EC2 and AMI id retrieval raised an exception" do
+        URI.should_receive(:parse).with('http://169.254.169.254/latest/meta-data/ami-id').and_return('parsed')
+        Net::HTTP.should_receive(:get_response).with("parsed").and_raise "Boom"
         @helper.hw_virtualization_available?.should == false
       end
     end