linux_admin 0.1.2 → 0.1.3

data/spec/registration_system_spec.rb

@@ -0,0 +1,60 @@
+require 'spec_helper'
+
+describe LinuxAdmin::RegistrationSystem do
+  context ".registration_type" do
+    it "when registered Subscription Manager" do
+      stub_registered_to_system(:sm)
+      expect(described_class.registration_type).to eq(LinuxAdmin::SubscriptionManager)
+    end
+
+    it "when registered RHN" do
+      stub_registered_to_system(:rhn)
+      expect(described_class.registration_type).to eq(LinuxAdmin::Rhn)
+    end
+
+    it "when unregistered" do
+      stub_registered_to_system(nil)
+      expect(described_class.registration_type).to eq(described_class)
+    end
+
+    it "should memoize results" do
+      described_class.should_receive(:registration_type_uncached).once.and_call_original
+      described_class.registration_type
+      described_class.registration_type
+    end
+
+    it "with reload should refresh results" do
+      described_class.should_receive(:registration_type_uncached).twice.and_call_original
+      described_class.registration_type
+      described_class.registration_type(true)
+    end
+  end
+
+  it "#registered? when unregistered" do
+    stub_registered_to_system(nil)
+    expect(described_class.registered?).to be_false
+  end
+
+  context ".method_missing" do
+    before do
+      stub_registered_to_system(:rhn)
+    end
+
+    it "exists on the subclass" do
+      expect(LinuxAdmin::RegistrationSystem.registered?).to be_true
+    end
+
+    it "does not exist on the subclass" do
+      expect { LinuxAdmin::RegistrationSystem.organizations }.to raise_error(NotImplementedError)
+    end
+
+    it "is an unknown method" do
+      expect { LinuxAdmin::RegistrationSystem.method_does_not_exist }.to be_true
+    end
+  end
+
+  def stub_registered_to_system(system)
+    LinuxAdmin::SubscriptionManager.any_instance.stub(:registered? => (system == :sm))
+    LinuxAdmin::Rhn.any_instance.stub(:registered? => data_file_path("rhn/systemid")) if system == :rhn
+  end
+end

data/spec/rhn_spec.rb

@@ -1,35 +1,31 @@
 require 'spec_helper'
 
 describe LinuxAdmin::Rhn do
-  it ".systemid_file" do
-    expect(described_class.systemid_file).to be_kind_of(String)
-  end
-
-  context ".registered?" do
+  context "#registered?" do
     it "with registered system_id" do
-      described_class.stub(:systemid_file => data_file_path("rhn/systemid"))
-      expect(described_class).to be_registered
+      described_class.any_instance.stub(:systemid_file => data_file_path("rhn/systemid"))
+      expect(described_class.new).to be_registered
     end
 
     it "with unregistered system_id" do
-      described_class.stub(:systemid_file => data_file_path("rhn/systemid.missing_system_id"))
-      expect(described_class).to_not be_registered
+      described_class.any_instance.stub(:systemid_file => data_file_path("rhn/systemid.missing_system_id"))
+      expect(described_class.new).to_not be_registered
     end
 
     it "with missing systemid file" do
-      described_class.stub(:systemid_file => data_file_path("rhn/systemid.missing_file"))
-      expect(described_class).to_not be_registered
+      described_class.any_instance.stub(:systemid_file => data_file_path("rhn/systemid.missing_file"))
+      expect(described_class.new).to_not be_registered
     end
   end
 
-  context ".register" do
+  context "#register" do
     it "no username or activation key" do
-      expect { described_class.register({}) }.to raise_error(ArgumentError)
+      expect { described_class.new.register({}) }.to raise_error(ArgumentError)
     end
 
     it "with username and password" do
-      described_class.should_receive(:run).once.with("rhnreg_ks", {:params=>{"--username="=>"SomeUser", "--password="=>"SomePass", "--proxy="=>"1.2.3.4", "--proxyUser="=>"ProxyUser", "--proxyPassword="=>"ProxyPass", "--serverUrl="=>"192.168.1.1"}}).and_return(0)
-      described_class.register(
+      described_class.any_instance.should_receive(:run).once.with("rhnreg_ks", {:params=>{"--username="=>"SomeUser", "--password="=>"SomePass", "--proxy="=>"1.2.3.4", "--proxyUser="=>"ProxyUser", "--proxyPassword="=>"ProxyPass", "--serverUrl="=>"192.168.1.1"}}).and_return(0)
+      described_class.new.register(
         :username       => "SomeUser",
         :password       => "SomePass",
         :proxy_address  => "1.2.3.4",
@@ -40,8 +36,8 @@ describe LinuxAdmin::Rhn do
     end
 
     it "with activation key" do
-      described_class.should_receive(:run).once.with("rhnreg_ks", {:params=>{"--activationkey="=>"123abc", "--proxy="=>"1.2.3.4", "--proxyUser="=>"ProxyUser", "--proxyPassword="=>"ProxyPass", "--serverUrl="=>"192.168.1.1"}}).and_return(0)
-      described_class.register(
+      described_class.any_instance.should_receive(:run).once.with("rhnreg_ks", {:params=>{"--activationkey="=>"123abc", "--proxy="=>"1.2.3.4", "--proxyUser="=>"ProxyUser", "--proxyPassword="=>"ProxyPass", "--serverUrl="=>"192.168.1.1"}}).and_return(0)
+      described_class.new.register(
         :activationkey  => "123abc",
         :proxy_address  => "1.2.3.4",
         :proxy_username => "ProxyUser",
@@ -51,14 +47,14 @@ describe LinuxAdmin::Rhn do
     end
   end
 
-  context ".subscribe" do
+  context "#subscribe" do
     it "without arguments" do
-      expect { described_class.subscribe({}) }.to raise_error(ArgumentError)
+      expect { described_class.new.subscribe({}) }.to raise_error(ArgumentError)
     end
 
     it "with pools" do
-      described_class.should_receive(:run).once.with("rhn-channel -a", {:params=>[["--user=", "SomeUser"], ["--password=", "SomePass"], ["--channel=", 123], ["--channel=", 456]]})
-      described_class.subscribe({
+      described_class.any_instance.should_receive(:run).once.with("rhn-channel -a", {:params=>[["--user=", "SomeUser"], ["--password=", "SomePass"], ["--channel=", 123], ["--channel=", 456]]})
+      described_class.new.subscribe({
         :username => "SomeUser",
         :password => "SomePass",
         :pools    => [123, 456]

data/spec/spec_helper.rb

@@ -18,6 +18,14 @@ RSpec.configure do |config|
   # the seed, which is printed after each run.
   #     --seed 1234
   config.order = 'random'
+
+  config.before do
+    Kernel.stub(:spawn).and_raise("Spawning is not permitted in specs.  Please change your spec to use expectations/stubs.")
+  end
+
+  config.after do
+    clear_caches
+  end
 end
 
 def data_file_path(to)
@@ -28,6 +36,10 @@ def sample_output(to)
   File.read(data_file_path(to))
 end
 
+def clear_caches
+  LinuxAdmin::RegistrationSystem.instance_variable_set(:@registration_type, nil)
+end
+
 class LinuxAdmin
   module Distros
     # simply alias test distro to redhat distro for time being

data/spec/subscription_manager_spec.rb

@@ -1,31 +1,31 @@
 require 'spec_helper'
 
 describe LinuxAdmin::SubscriptionManager do
-  context ".registered?" do
+  context "#registered?" do
     it "system with subscription-manager commands" do
-      described_class.should_receive(:run).once.with("subscription-manager identity", {:return_exitstatus=>true}).and_return(0)
-      expect(described_class.registered?).to be_true
+      described_class.any_instance.should_receive(:run).once.with("subscription-manager identity", {:return_exitstatus=>true}).and_return(0)
+      expect(described_class.new.registered?).to be_true
     end
 
     it "system without subscription-manager commands" do
-      described_class.should_receive(:run).once.with("subscription-manager identity", {:return_exitstatus=>true}).and_return(255)
-      expect(described_class.registered?).to be_false
+      described_class.any_instance.should_receive(:run).once.with("subscription-manager identity", {:return_exitstatus=>true}).and_return(255)
+      expect(described_class.new.registered?).to be_false
     end
   end
 
-  it ".refresh" do
-    described_class.should_receive(:run).once.with("subscription-manager refresh").and_return(0)
-    described_class.refresh
+  it "#refresh" do
+    described_class.any_instance.should_receive(:run).once.with("subscription-manager refresh").and_return(0)
+    described_class.new.refresh
   end
 
-  context ".register" do
+  context "#register" do
     it "no username" do
-      expect { described_class.register }.to raise_error(ArgumentError)
+      expect { described_class.new.register }.to raise_error(ArgumentError)
     end
 
     it "with username and password" do
-      described_class.should_receive(:run).once.with("subscription-manager register", {:params=>{"--username="=>"SomeUser", "--password="=>"SomePass", "--org="=>"IT", "--proxy="=>"1.2.3.4", "--proxyuser="=>"ProxyUser", "--proxypassword="=>"ProxyPass", "--serverurl="=>"192.168.1.1"}}).and_return(0)
-      described_class.register(
+      described_class.any_instance.should_receive(:run).once.with("subscription-manager register", {:params=>{"--username="=>"SomeUser", "--password="=>"SomePass", "--org="=>"IT", "--proxy="=>"1.2.3.4", "--proxyuser="=>"ProxyUser", "--proxypassword="=>"ProxyPass", "--serverurl="=>"192.168.1.1"}}).and_return(0)
+      described_class.new.register(
         :username       => "SomeUser",
         :password       => "SomePass",
         :org            => "IT",
@@ -37,14 +37,14 @@ describe LinuxAdmin::SubscriptionManager do
     end
   end
 
-  it ".subscribe" do
-    described_class.should_receive(:run).once.with("subscription-manager attach", {:params=>[["--pool", 123], ["--pool", 456]]})
-    described_class.subscribe({:pools => [123, 456]})
+  it "#subscribe" do
+    described_class.any_instance.should_receive(:run).once.with("subscription-manager attach", {:params=>[["--pool", 123], ["--pool", 456]]})
+    described_class.new.subscribe({:pools => [123, 456]})
   end
 
-  it ".available_subscriptions" do
-    described_class.should_receive(:run).once.with("subscription-manager list --all --available", :return_output => true).and_return(sample_output("subscription_manager/output_list_all_available"))
-    expect(described_class.available_subscriptions).to eq({
+  it "#available_subscriptions" do
+    described_class.any_instance.should_receive(:run).once.with("subscription-manager list --all --available", :return_output => true).and_return(sample_output("subscription_manager/output_list_all_available"))
+    expect(described_class.new.available_subscriptions).to eq({
       "82c042fca983889b10178893f29b06e3" => {
         :subscription_name => "Example Subscription",
         :sku               => "SER0123",
@@ -91,4 +91,9 @@ describe LinuxAdmin::SubscriptionManager do
       },
     })
   end
+
+  it "#organizations" do
+    described_class.any_instance.should_receive(:run).once.with("subscription-manager orgs", {:params=>{"--username="=>"SomeUser", "--password="=>"SomePass", "--proxy="=>"1.2.3.4", "--proxyuser="=>"ProxyUser", "--proxypassword="=>"ProxyPass", "--serverurl="=>"192.168.1.1"}, :return_output => true}).and_return(sample_output("subscription_manager/output_orgs"))
+    expect(described_class.new.organizations({:username=>"SomeUser", :password=>"SomePass", :proxy_address=>"1.2.3.4", :proxy_username=>"ProxyUser", :proxy_password=>"ProxyPass", :server_url=>"192.168.1.1"})).to eq({"SomeOrg"=>{:name=>"SomeOrg", :key=>"1234567"}})
+  end
 end

data/spec/volume_group_spec.rb

@@ -0,0 +1,108 @@
+require 'spec_helper'
+
+describe LinuxAdmin::VolumeGroup do
+  before(:each) do
+    LinuxAdmin::Distro.stub(:local => LinuxAdmin::Distros::Test.new)
+
+    @groups = <<eos
+vg_foobar:r/w:772:-1:0:2:2:-1:0:1:1:12058624:32768:368:368:0:tILZUF-IspH-H90I-pT5j-vVFl-b76L-zWx3CW
+eos
+  end
+
+  after(:each) do
+    # reset local copies of volumes / groups
+    LinuxAdmin::LogicalVolume.instance_variable_set(:@lvs, nil)
+    LinuxAdmin::PhysicalVolume.instance_variable_set(:@pvs, nil)
+    described_class.instance_variable_set(:@vgs, nil)
+  end
+
+  describe "#attach_to" do
+    it "uses lvextend" do
+      lv = LinuxAdmin::LogicalVolume.new :name => 'lv'
+      vg = described_class.new :name => 'vg'
+      vg.should_receive(:run).
+         with(vg.cmd(:lvextend),
+              :params => ['lv', 'vg'])
+      vg.attach_to(lv)
+    end
+
+    it "returns self" do
+      lv = LinuxAdmin::LogicalVolume.new :name => 'lv'
+      vg = described_class.new :name => 'vg'
+      vg.stub(:run)
+      vg.attach_to(lv).should == vg
+    end
+  end
+
+  describe "#extend_with" do
+    it "uses vgextend" do
+      vg = described_class.new :name => 'vg'
+      pv = LinuxAdmin::PhysicalVolume.new :device_name => '/dev/hda'
+      vg.should_receive(:run).
+         with(vg.cmd(:vgextend),
+              :params => ['vg', '/dev/hda'])
+      vg.extend_with(pv)
+    end
+
+    it "assigns volume group to physical volume" do
+      vg = described_class.new :name => 'vg'
+      pv = LinuxAdmin::PhysicalVolume.new :device_name => '/dev/hda'
+      vg.stub(:run)
+      vg.extend_with(pv)
+      pv.volume_group.should == vg
+    end
+
+    it "returns self" do
+      vg = described_class.new :name => 'vg'
+      pv = LinuxAdmin::PhysicalVolume.new :device_name => '/dev/hda'
+      vg.stub(:run)
+      vg.extend_with(pv).should == vg
+    end
+  end
+
+  describe "#create" do
+    before(:each) do
+      @pv = LinuxAdmin::PhysicalVolume.new :device_name => '/dev/hda'
+    end
+
+    it "uses vgcreate" do
+      described_class.instance_variable_set(:@vgs, [])
+      described_class.should_receive(:run).
+                              with(LinuxAdmin.cmd(:vgcreate),
+                                   :params => ['vg', '/dev/hda'])
+      described_class.create 'vg', @pv
+    end
+
+    it "returns new volume group" do
+      described_class.stub(:run => "")
+      vg = described_class.create 'vg', @pv
+      vg.should be_an_instance_of(described_class)
+      vg.name.should == 'vg'
+    end
+
+    it "adds volume group to local registry" do
+      described_class.stub(:run => "")
+      vg = described_class.create 'vg', @pv
+      described_class.scan.should include(vg)
+    end
+  end
+
+  describe "#scan" do
+    it "uses vgdisplay" do
+      described_class.should_receive(:run).
+                              with(LinuxAdmin.cmd(:vgdisplay),
+                                   :return_output => true,
+                                   :params => { '-c' => nil}).
+                                 and_return(@groups)
+      described_class.scan
+    end
+
+    it "returns local volume groups" do
+      described_class.should_receive(:run).and_return(@groups)
+      vgs = described_class.scan
+
+      vgs[0].should be_an_instance_of(described_class)
+      vgs[0].name.should == 'vg_foobar'
+    end
+  end
+end

data/spec/yum_spec.rb

@@ -107,7 +107,7 @@ describe LinuxAdmin::Yum do
 
   context ".version_available" do
     it "no packages" do
-      expect { described_class.version_available }.to raise_error
+      expect { described_class.version_available }.to raise_error(ArgumentError)
     end
 
     it "with packages" do

metadata

@@ -1,15 +1,18 @@
 --- !ruby/object:Gem::Specification
 name: linux_admin
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
   prerelease: 
 platform: ruby
 authors:
 - Brandon Dunne
+- Jason Frey
+- Mo Morsi
+- Joe Rafaniello
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-07-11 00:00:00.000000000 Z
+date: 2013-08-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -149,7 +152,10 @@ description: ! '
 
 '
 email:
-- brandondunne@hotmail.com
+- bdunne@redhat.com
+- fryguy9@gmail.com
+- mmorsi@redhat.com
+- jrafanie@redhat.com
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -165,15 +171,19 @@ files:
 - lib/linux_admin/common.rb
 - lib/linux_admin/disk.rb
 - lib/linux_admin/distro.rb
+- lib/linux_admin/exceptions.rb
 - lib/linux_admin/fstab.rb
 - lib/linux_admin/logical_volume.rb
 - lib/linux_admin/partition.rb
-- lib/linux_admin/rhn.rb
+- lib/linux_admin/physical_volume.rb
+- lib/linux_admin/registration_system.rb
+- lib/linux_admin/registration_system/rhn.rb
+- lib/linux_admin/registration_system/subscription_manager.rb
 - lib/linux_admin/rpm.rb
 - lib/linux_admin/service.rb
-- lib/linux_admin/subscription_manager.rb
 - lib/linux_admin/system.rb
 - lib/linux_admin/version.rb
+- lib/linux_admin/volume_group.rb
 - lib/linux_admin/yum.rb
 - linux_admin.gemspec
 - spec/common_spec.rb
@@ -181,6 +191,7 @@ files:
 - spec/data/rhn/systemid.missing_system_id
 - spec/data/rpm/cmd_output_for_list_installed
 - spec/data/subscription_manager/output_list_all_available
+- spec/data/subscription_manager/output_orgs
 - spec/data/yum/first.repo
 - spec/data/yum/output_repoquery
 - spec/data/yum/second.repo
@@ -188,13 +199,17 @@ files:
 - spec/distro_spec.rb
 - spec/fstab_spec.rb
 - spec/linux_admin_spec.rb
+- spec/logical_volume_spec.rb
 - spec/partition_spec.rb
+- spec/physical_volume_spec.rb
+- spec/registration_system_spec.rb
 - spec/rhn_spec.rb
 - spec/rpm_spec.rb
 - spec/service_spec.rb
 - spec/spec_helper.rb
 - spec/subscription_manager_spec.rb
 - spec/system_spec.rb
+- spec/volume_group_spec.rb
 - spec/yum_spec.rb
 homepage: http://github.com/ManageIQ/linux_admin
 licenses:
@@ -227,6 +242,7 @@ test_files:
 - spec/data/rhn/systemid.missing_system_id
 - spec/data/rpm/cmd_output_for_list_installed
 - spec/data/subscription_manager/output_list_all_available
+- spec/data/subscription_manager/output_orgs
 - spec/data/yum/first.repo
 - spec/data/yum/output_repoquery
 - spec/data/yum/second.repo
@@ -234,11 +250,15 @@ test_files:
 - spec/distro_spec.rb
 - spec/fstab_spec.rb
 - spec/linux_admin_spec.rb
+- spec/logical_volume_spec.rb
 - spec/partition_spec.rb
+- spec/physical_volume_spec.rb
+- spec/registration_system_spec.rb
 - spec/rhn_spec.rb
 - spec/rpm_spec.rb
 - spec/service_spec.rb
 - spec/spec_helper.rb
 - spec/subscription_manager_spec.rb
 - spec/system_spec.rb
+- spec/volume_group_spec.rb
 - spec/yum_spec.rb

data/lib/linux_admin/subscription_manager.rb

@@ -1,61 +0,0 @@
-require 'date'
-
-class LinuxAdmin
-  class SubscriptionManager < LinuxAdmin
-    def self.registered?
-      run("subscription-manager identity", :return_exitstatus => true) == 0
-    end
-
-    def self.refresh
-      run("subscription-manager refresh")
-    end
-
-    def self.register(options)
-      raise ArgumentError, "username and password are required" unless options[:username] && options[:password]
-      cmd = "subscription-manager register"
-
-      params = {"--username=" => options[:username], "--password=" => options[:password]}
-      params.merge!(proxy_params(options))
-      params["--org="]        = options[:org]         if options[:server_url] && options[:org]
-      params["--serverurl="]  = options[:server_url]  if options[:server_url]
-
-      run(cmd, :params => params)
-    end
-
-    def self.subscribe(options)
-      cmd    = "subscription-manager attach"
-      pools  = options[:pools].collect {|pool| ["--pool", pool]}
-      params = proxy_params(options).to_a + pools
-
-      run(cmd, :params => params)
-    end
-
-    def self.available_subscriptions
-      out = run("subscription-manager list --all --available", :return_output => true)
-
-      out.split("\n\n").each_with_object({}) do |subscription, subscriptions_hash|
-        hash = {}
-        subscription.each_line do |line|
-          # Strip the header lines if they exist
-          next if (line.start_with?("+---") && line.end_with?("---+\n")) || line.strip == "Available Subscriptions"
-
-          key, value = line.split(":", 2)
-          hash[key.strip.downcase.tr(" -", "_").to_sym] = value.strip
-        end
-        hash[:ends] = Date.strptime(hash[:ends], "%m/%d/%Y")
-
-        subscriptions_hash[hash[:pool_id]] = hash
-      end
-    end
-
-    private
-
-    def self.proxy_params(options)
-      config = {}
-      config["--proxy="]          = options[:proxy_address]   if options[:proxy_address]
-      config["--proxyuser="]      = options[:proxy_username]  if options[:proxy_username]
-      config["--proxypassword="]  = options[:proxy_password]  if options[:proxy_password]
-      config
-    end
-  end
-end