hammer_cli 0.1.1 → 0.1.2

data/test/reports/TEST-constraints-HammerCLI-Validator-BaseConstraint-rejected.xml

@@ -0,0 +1,13 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuite name="constraints::HammerCLI::Validator::BaseConstraint::rejected" tests="3" time="0.001188178" failures="0" errors="0" skipped="0" assertions="3">
+  <testcase name="test_0003_should return nil when exist? returns true" time="0.000339906" assertions="1">
+  </testcase>
+  <testcase name="test_0002_should raise exception with a message" time="0.000330188" assertions="1">
+  </testcase>
+  <testcase name="test_0001_should raise exception when exist? returns true" time="0.000327038" assertions="1">
+  </testcase>
+  <system-out>
+  </system-out>
+  <system-err>
+  </system-err>
+</testsuite>

data/test/reports/TEST-constraints-HammerCLI-Validator-BaseConstraint-required.xml

@@ -0,0 +1,13 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuite name="constraints::HammerCLI::Validator::BaseConstraint::required" tests="3" time="0.001221194" failures="0" errors="0" skipped="0" assertions="2">
+  <testcase name="test_0003_should return nil when exist? returns true" time="0.000335162" assertions="1">
+  </testcase>
+  <testcase name="test_0001_should raise exception when exist? returns true" time="0.000327945" assertions="1">
+  </testcase>
+  <testcase name="test_0002_should raise exception with a message" time="0.000337518" assertions="0">
+  </testcase>
+  <system-out>
+  </system-out>
+  <system-err>
+  </system-err>
+</testsuite>

data/test/reports/TEST-constraints-HammerCLI-Validator-BaseConstraint.xml

@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuite name="constraints::HammerCLI::Validator::BaseConstraint" tests="0" time="0.000163178" failures="0" errors="0" skipped="0" assertions="0">
+  <system-out>
+  </system-out>
+  <system-err>
+  </system-err>
+</testsuite>

data/test/reports/TEST-constraints.xml

@@ -0,0 +1,7 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuite name="constraints" tests="0" time="0.000161677" failures="0" errors="0" skipped="0" assertions="0">
+  <system-out>
+  </system-out>
+  <system-err>
+  </system-err>
+</testsuite>

data/test/unit/abstract_test.rb

@@ -277,16 +277,6 @@ describe HammerCLI::AbstractCommand do
       HammerCLI::AbstractCommand.option_builder.builders.empty?.must_equal true
     end
 
-    it "can be initialized with custom builder" do
-      cmd = Class.new(HammerCLI::AbstractCommand) do
-        def self.custom_option_builders
-          [:builder_1, :builder_2]
-        end
-      end
-
-      cmd.option_builder.builders.must_equal [:builder_1, :builder_2]
-    end
-
   end
 
   describe "build options" do
@@ -304,7 +294,7 @@ describe HammerCLI::AbstractCommand do
 
     class TestBuilderCmd < HammerCLI::AbstractCommand
 
-      def self.option_builder
+      def self.create_option_builder
         TestOptionBuilder.new
       end
 
@@ -327,6 +317,20 @@ describe HammerCLI::AbstractCommand do
       TestBuilderCmd.recognised_options.map(&:description).flatten.sort.must_equal ["original_test", "test2"].sort
     end
 
+    it "passes params to the builders" do
+      @params = {:param1 => 1, :param2 => :value}
+      TestBuilderCmd.option_builder.expects(:build).with(@params).returns([])
+      TestBuilderCmd.build_options(@params)
+    end
+
+    it "accepts block" do
+      @params = {:param1 => 1, :param2 => :value}
+      TestBuilderCmd.option_builder.expects(:build).with(@params).returns([])
+      TestBuilderCmd.build_options do |params|
+        @params
+      end
+    end
+
   end
 
   it "should inherit command_name" do

data/test/unit/apipie/command_test.rb

@@ -13,6 +13,19 @@ describe HammerCLI::Apipie::Command do
     action :show
   end
 
+  class OptionCommand < TestCommand
+    resource :architectures, :create
+
+    def option_name
+      'test_name'
+    end
+
+    def option_operatingsystem_ids
+      nil
+    end
+
+  end
+
   class CommandA < TestCommand
     resource :architectures, :index
 
@@ -103,5 +116,20 @@ describe HammerCLI::Apipie::Command do
 
   end
 
-end
+  context "options" do
 
+    it "should collect method options from given options" do
+      cmd_opt = OptionCommand.new("", ctx)
+      params = cmd_opt.class.resource.action(:create).params
+      cmd_opt.method_options_for_params(params, {'option_name' => 'name'}).must_equal({"architecture" => {"name" => "name"}})
+    end
+
+    it "should collect method options from methods" do
+      cmd_opt = OptionCommand.new("", ctx)
+      params = cmd_opt.class.resource.action(:create).params
+      cmd_opt.method_options_for_params(params, {}).must_equal({"architecture"=>{"name"=>"test_name"}})
+    end
+
+  end
+
+end

data/test/unit/apipie/option_builder_test.rb

@@ -106,5 +106,27 @@ describe HammerCLI::Apipie::OptionBuilder do
     end
 
   end
+
+  context "aliasing resources" do
+    let(:action) {api.resource(:documented).action(:action_with_ids)}
+    let(:builder_options) { {:resource_mapping => {:organization => 'company', 'compute_resource' => :compute_provider}} }
+
+    it "renames options" do
+      # builder_options[:resource_mapping] = {:organization => 'company', 'compute_resource' => :compute_provider}
+      options.map(&:long_switch).sort.must_equal ["--company-id", "--company-ids", "--compute-provider-id", "--name"]
+    end
+
+    it "renames option types" do
+      # builder_options[:resource_mapping] = {:organization => 'company', 'compute_resource' => :compute_provider}
+      options.map(&:type).sort.must_equal ["COMPANY_ID", "COMPANY_IDS", "COMPUTE_PROVIDER_ID", "NAME"]
+    end
+
+    it "keeps option accessor the same" do
+      # builder_options[:resource_mapping] = {:organization => 'company', 'compute_resource' => :compute_provider}
+      options.map(&:attribute_name).sort.must_equal HammerCLI.option_accessor_name("compute_resource_id", "name", "organization_id", "organization_ids")
+    end
+
+  end
+
 end
 

data/test/unit/fixtures/apipie/documented.json

@@ -26,6 +26,50 @@
                         ],
                         "full_description": ""
                     },
+                    {
+                        "name": "action_with_ids",
+                        "examples": [],
+                        "errors": [],
+                        "params": [
+                            {
+                                "name": "name",
+                                "allow_null": false,
+                                "full_name": "name",
+                                "validator": "Must be String",
+                                "expected_type": "string",
+                                "description": "",
+                                "required": false
+                            },
+                            {
+                                "name": "organization_id",
+                                "allow_null": false,
+                                "full_name": "organization_id",
+                                "validator": "Must be number",
+                                "expected_type": "number",
+                                "description": "",
+                                "required": false
+                            },
+                            {
+                                "name": "organization_ids",
+                                "allow_null": false,
+                                "full_name": "organization_ids",
+                                "validator": "Array of numbers",
+                                "expected_type": "Array",
+                                "description": "",
+                                "required": false
+                            },
+                            {
+                                "name": "compute_resource_id",
+                                "allow_null": false,
+                                "full_name": "compute_resource_id",
+                                "validator": "Must be number",
+                                "expected_type": "number",
+                                "description": "",
+                                "required": false
+                            }
+                        ],
+                        "full_description": ""
+                    },
                     {
                         "name": "create",
                         "examples": [],

data/test/unit/options/normalizers_test.rb

@@ -171,6 +171,41 @@ describe HammerCLI::Options::Normalizers do
 
   end
 
+  describe 'enumlist' do
+
+    let (:formatter) { HammerCLI::Options::Normalizers::EnumList.new ['1', '2', 'a', 'b'] }
+
+    it "should return array of values when the values are allowed" do
+      formatter.format("a,b,1").must_equal(['a', 'b', '1'])
+    end
+
+    it "should raise argument error when any of the values isn't in the list" do
+      proc { formatter.format("c,d") }.must_raise ArgumentError
+      proc { formatter.format('1,x') }.must_raise ArgumentError
+    end
+
+    it "should remove duplicate values" do
+      formatter.format("a,a,a,a,a").must_equal ['a']
+    end
+
+    it "should not change order of the values" do
+      formatter.format("a,b,2,1").must_equal ['a', 'b', '2', '1']
+    end
+
+    it "should return empty array on nil" do
+      formatter.format(nil).must_equal []
+    end
+
+    it "should return empty array on empty string" do
+      formatter.format("").must_equal []
+    end
+
+    it "should list allowed values in description" do
+      formatter.description.must_equal("Any combination (comma separated list) of ''1', '2', 'a', 'b''")
+    end
+
+  end
+
   describe 'datetime' do
 
     let(:formatter) { HammerCLI::Options::Normalizers::DateTime.new }

data/test/unit/output/adapter/csv_test.rb

@@ -16,13 +16,16 @@ describe HammerCLI::Output::Adapter::CSValues do
       :started_at => "2000"
     }]}
 
-
     it "should print column name" do
-      proc { adapter.print_collection(fields, data) }.must_output(/.*Name,Started At.*/, "")
+      out, err = capture_io { adapter.print_collection(fields, data) }
+      out.must_match /.*Name,Started At.*/
+      err.must_match //
     end
 
     it "should print field value" do
-      proc { adapter.print_collection(fields, data) }.must_output(/.*John Doe.*/, "")
+      out, err = capture_io { adapter.print_collection(fields, data) }
+      out.must_match /.*John Doe.*/
+      err.must_match //
     end
 
     context "handle ids" do
@@ -30,11 +33,15 @@ describe HammerCLI::Output::Adapter::CSValues do
       let(:fields) {
         [field_name, field_id]
       }
+      let(:data) { HammerCLI::Output::RecordCollection.new [{
+        :name => "John Doe",
+        :some_id => "2000"
+      }]}
 
       it "should ommit column of type Id by default" do
         out, err = capture_io { adapter.print_collection(fields, data) }
         out.wont_match(/.*Id.*/)
-        out.wont_match(/.*John Doe,.*/)
+        out.wont_match(/.*2000,.*/)
       end
 
       it "should print column of type Id when --show-ids is set" do
@@ -44,6 +51,99 @@ describe HammerCLI::Output::Adapter::CSValues do
       end
     end
 
+    context "handle fields with containers" do
+      let(:demographics) {
+        Fields::Label.new(:path => [], :label => "Demographics") do
+          from :demographics do
+            field :age, "Age"
+            field :gender, "Gender"
+            label _("Biometrics") do
+              from :biometrics do
+                field :weight, "Weight"
+                field :height, "Height"
+              end
+            end
+          end
+        end
+      }
+      let(:fields) {
+        [field_name, field_started_at, demographics]
+      }
+      let(:data) { HammerCLI::Output::RecordCollection.new [{
+        :name => "John Doe",
+        :started_at => "2000",
+        :demographics=> { :age => '22', :gender => 'm',
+                          :biometrics => { :weight => '123', :height => '155' } }
+      }]}
+
+      it "should print column names" do
+        out, err = capture_io { adapter.print_collection(fields, data) }
+        out.must_match /.*Demographics::Age,Demographics::Gender,Biometrics::Weight,Biometrics::Height*/
+        err.must_match //
+      end
+
+      it "should print data" do
+        out, err = capture_io { adapter.print_collection(fields, data) }
+        out.must_match /.*2000,22,m,123,155*/
+        err.must_match //
+      end
+    end
+
+    context "handle fields with collections" do
+      let(:items) {
+        Fields::Collection.new(:path => [:items], :label => "Items") do
+          from :item do
+            field :name, "Item Name"
+            field :quantity, "Item Quantity"
+          end
+        end
+      }
+      let(:fields) {
+        [field_name, field_started_at, items]
+      }
+      let(:data) { HammerCLI::Output::RecordCollection.new [
+        {
+          :name => "John Doe",
+          :started_at => "2000",
+          :items => [{:item => { :name => 'hammer', :quantity => '100'}}]
+        },
+        {
+          :name => "Jane Roe",
+          :started_at => "2001",
+          :items => [{:item => { :name => 'cleaver', :quantity => '1'}}, {:item => { :name => 'sledge', :quantity => '50'}}]
+        }
+      ]}
+
+      it "should print collection column name" do
+        out, err = capture_io { adapter.print_collection(fields, data) }
+
+        lines = out.split("\n")
+        lines[0].must_equal 'Name,Started At,Items::Item Name::1,Items::Item Quantity::1,Items::Item Name::2,Items::Item Quantity::2'
+
+        err.must_match //
+      end
+
+      it "should print collection data" do
+        out, err = capture_io { adapter.print_collection(fields, data) }
+        lines = out.split("\n")
+
+        lines[1].must_equal 'John Doe,2000,hammer,100,"",""'
+        lines[2].must_equal 'Jane Roe,2001,cleaver,1,sledge,50'
+
+        err.must_match //
+      end
+
+      it "should handle empty collection" do
+        out, err = capture_io { adapter.print_collection(fields, []) }
+        lines = out.split("\n")
+
+        lines[0].must_equal 'Name,Started At,Items'
+
+        err.must_match //
+      end
+
+    end
+
     context "formatters" do
       it "should apply formatters" do
         class DotFormatter < HammerCLI::Output::Formatters::FieldFormatter

data/test/unit/utils_test.rb

@@ -1,6 +1,13 @@
 require File.join(File.dirname(__FILE__), 'test_helper')
 
 
+module Constant
+  module Test
+    class X
+    end
+  end
+end
+
 describe String do
 
   context "formatting" do
@@ -56,6 +63,35 @@ describe String do
 
   end
 
+  describe "constantize" do
+
+    it "raises NameError for empty string" do
+      proc {
+        "".constantize
+      }.must_raise NameError
+    end
+
+    it "raises NameError for unknown constant" do
+      proc {
+        "UnknownClass".constantize
+      }.must_raise NameError
+    end
+
+    it "returns correct constant" do
+      "Object".constantize.must_equal Object
+    end
+
+    it "returns correct namespaced constant" do
+      "Constant::Test::X".constantize.must_equal Constant::Test::X
+    end
+  end
+
+end
+
+
+
+describe HammerCLI do
+
   describe "interactive?" do
 
     before :each do
@@ -89,5 +125,27 @@ describe String do
     end
   end
 
+
+  describe "constant_path" do
+
+    it "returns empty array for empty string" do
+      HammerCLI.constant_path("").must_equal []
+    end
+
+    it "raises NameError for unknown constant" do
+      proc {
+        HammerCLI.constant_path("UnknownClass")
+      }.must_raise NameError
+    end
+
+    it "returns single constant" do
+      HammerCLI.constant_path("Object").must_equal [Object]
+    end
+
+    it "returns correct path for namespaced constant" do
+      HammerCLI.constant_path("Constant::Test::X").must_equal [Constant, Constant::Test, Constant::Test::X]
+    end
+  end
+
 end