hammer_cli_foreman 0.0.11 → 0.0.12

data/lib/hammer_cli_foreman/version.rb

@@ -1,5 +1,5 @@
 module HammerCLIForeman
   def self.version
-    @version ||= Gem::Version.new '0.0.11'
+    @version ||= Gem::Version.new '0.0.12'
   end
 end

data/test/unit/commands_test.rb

@@ -0,0 +1,79 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require 'hammer_cli'
+
+
+describe HammerCLIForeman do
+
+  context "collection_to_common_format" do
+
+    let(:kind) { { "name" => "PXELinux", "id" => 1 } }
+
+    it "should convert old API format" do
+      old_format = [
+        {
+          "template_kind" => kind
+        }
+      ]
+
+      set = HammerCLIForeman.collection_to_common_format(old_format)
+      set.must_be_kind_of HammerCLI::Output::RecordCollection
+      set.first.must_equal(kind)
+    end
+
+    it "should convert common API format" do
+      common_format = [ kind ]
+
+      set = HammerCLIForeman.collection_to_common_format(common_format)
+      set.must_be_kind_of HammerCLI::Output::RecordCollection
+      set.first.must_equal(kind)
+    end
+
+    it "should convert new API format" do
+      new_format = {
+        "total" => 1,
+        "subtotal" => 1,
+        "page" => 1,
+        "per_page" => 20,
+        "search" => nil,
+        "sort" => {
+          "by" => nil,
+          "order" => nil
+        },
+        "results" => [ kind ]
+      }
+
+      set = HammerCLIForeman.collection_to_common_format(new_format)
+      set.must_be_kind_of HammerCLI::Output::RecordCollection
+      set.first.must_equal(kind)
+    end
+
+    it "should rise error on unexpected format" do
+      proc { HammerCLIForeman.collection_to_common_format('unexpected') }.must_raise RuntimeError
+    end
+
+  end
+
+
+  context "record_to_common_format" do
+
+    let(:arch) { { "name" => "x86_64", "id" => 1 } }
+
+    it "should convert old API format" do
+      old_format = {
+        "architecture" => arch
+      }
+
+      rec = HammerCLIForeman.record_to_common_format(old_format)
+      rec.must_equal(arch)
+    end
+
+    it "should convert common API format" do
+      common_format = arch
+
+      rec = HammerCLIForeman.record_to_common_format(common_format)
+      rec.must_equal(arch)
+
+    end
+
+  end
+end

data/test/unit/output/formatters_test.rb

@@ -9,6 +9,11 @@ describe HammerCLIForeman::Output::Formatters::OSNameFormatter do
     formatter = HammerCLIForeman::Output::Formatters::OSNameFormatter.new
     formatter.format({ :name => 'Fedora', :major => '19'}).must_equal 'Fedora 19'
   end
+
+  it "recovers when os is nil" do
+    formatter = HammerCLIForeman::Output::Formatters::OSNameFormatter.new
+    formatter.format(nil).must_equal nil
+  end
 end
 
 describe HammerCLIForeman::Output::Formatters::ServerFormatter do
@@ -16,4 +21,9 @@ describe HammerCLIForeman::Output::Formatters::ServerFormatter do
     formatter = HammerCLIForeman::Output::Formatters::ServerFormatter.new
     formatter.format({ :name => 'Server', :url => "URL"}).must_equal 'Server (URL)'
   end
+
+  it "recovers when server is nil" do
+    formatter = HammerCLIForeman::Output::Formatters::ServerFormatter.new
+    formatter.format(nil).must_equal nil
+  end
 end

data/test/unit/test_output_adapter.rb

@@ -3,7 +3,12 @@ require 'hammer_cli/output/adapter/abstract'
 
 class TestAdapter < HammerCLI::Output::Adapter::Abstract
 
-  def print_records(fields, data)
+  def print_record(fields, record)
+    print_collection(fields, [record].flatten(1))
+  end
+
+
+  def print_collection(fields, data)
     @separator = '#'
     puts @separator+fields.collect{|f| f.label.to_s}.join(@separator)+@separator
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hammer_cli_foreman
 version: !ruby/object:Gem::Version
-  version: 0.0.11
+  version: 0.0.12
 platform: ruby
 authors:
 - Tomáš Strachota
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-11-26 00:00:00.000000000 Z
+date: 2013-12-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hammer_cli
@@ -17,14 +17,14 @@ dependencies:
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
-        version: 0.0.10
+        version: 0.0.12
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
-        version: 0.0.10
+        version: 0.0.12
 - !ruby/object:Gem::Dependency
   name: foreman_api
   requirement: !ruby/object:Gem::Requirement
@@ -99,6 +99,7 @@ files:
 - lib/hammer_cli_foreman/version.rb
 - test/unit/apipie_resource_mock.rb
 - test/unit/architecture_test.rb
+- test/unit/commands_test.rb
 - test/unit/common_parameter_test.rb
 - test/unit/compute_resource_test.rb
 - test/unit/domain_test.rb
@@ -145,13 +146,14 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.8
+rubygems_version: 2.0.13
 signing_key: 
 specification_version: 4
 summary: Foreman commands for Hammer
 test_files:
 - test/unit/apipie_resource_mock.rb
 - test/unit/architecture_test.rb
+- test/unit/commands_test.rb
 - test/unit/common_parameter_test.rb
 - test/unit/compute_resource_test.rb
 - test/unit/domain_test.rb