hammer_cli 0.0.18 → 0.1.0

data/test/unit/output/dsl_test.rb

@@ -20,15 +20,15 @@ describe HammerCLI::Output::Dsl do
     dsl.fields.length.must_equal 0
   end
 
-  context "fields" do
-    it "should create DataField as default field type" do
+  describe "fields" do
+    it "should create Field as default field type" do
       dsl.build do
         field :f, "F"
       end
-      first_field.class.must_equal Fields::DataField
+      first_field.class.must_equal Fields::Field
     end
 
-    it "should create DataField of desired type" do
+    it "should create field of desired type" do
       dsl.build do
         field :f, "F", CustomFieldType
       end
@@ -56,7 +56,7 @@ describe HammerCLI::Output::Dsl do
     end
   end
 
-  context "custom fields" do
+  describe "custom fields" do
 
     let(:options) {{:a => 1, :b => 2}}
 
@@ -76,7 +76,7 @@ describe HammerCLI::Output::Dsl do
 
   end
 
-  context "path definition" do
+  describe "path definition" do
 
     it "from appends to path" do
       dsl.build do
@@ -114,6 +114,74 @@ describe HammerCLI::Output::Dsl do
   end
 
 
+  describe "label" do
+
+    it "creates field of type Label" do
+      dsl.build do
+        label "Label"
+      end
+      first_field.class.must_equal Fields::Label
+    end
+
+    it "allows to define subfields with dsl" do
+      dsl.build do
+        label "Label" do
+          field :a, "A"
+          field :b, "B"
+        end
+      end
+
+      first_field.fields.map(&:label).must_equal ["A", "B"]
+    end
+
+    it "sets correct path to subfields" do
+      dsl.build do
+        from :nest do
+          label "Label" do
+            field :a, "A"
+            field :b, "B"
+          end
+        end
+      end
+
+      first_field.fields.map(&:path).must_equal [[:a], [:b]]
+    end
+
+  end
+
+
+  describe "collection" do
+
+    it "creates field of type Collection" do
+      dsl.build do
+        collection :f, "F"
+      end
+      first_field.class.must_equal Fields::Collection
+    end
+
+    it "allows to define subfields with dsl" do
+      dsl.build do
+        collection :nest, "Label" do
+          field :a, "A"
+          field :b, "B"
+        end
+      end
+
+      first_field.fields.map(&:label).must_equal ["A", "B"]
+    end
+
+    it "sets correct path to subfields" do
+      dsl.build do
+        collection :nest, "Label" do
+          field :a, "A"
+          field :b, "B"
+        end
+      end
+
+      first_field.fields.map(&:path).must_equal [[:a], [:b]]
+    end
+
+  end
 
 end
 

data/test/unit/output/fields_test.rb

@@ -1,97 +1,128 @@
 require File.join(File.dirname(__FILE__), '../test_helper')
 
 
+
 describe Fields::Field do
 
-  let(:field) { Fields::Field.new }
+  let(:label) { "Some Label" }
+  let(:path) { [:address, :city, :name] }
+  let(:field) { Fields::Field.new :label => label, :path => path, :some => :parameter }
+  let(:blank_field) { Fields::Field.new :label => label, :path => path, :hide_blank => true }
 
-  describe "get_value" do
-    it "should exist" do
-      assert field.respond_to? :get_value
-    end
 
-    it "should take data as a parameter" do
-      field.method(:get_value).arity.must_equal 1
-    end
+  it "stores label from constructor" do
+    field.label.must_equal label
   end
 
-end
+  it "stores path from constructor" do
+    field.path.must_equal path
+  end
 
-describe Fields::LabeledField do
+  it "default path should be empty array" do
+    Fields::Field.new.path.must_equal []
+  end
 
-  let(:label) { "Some Label" }
-  let(:field) { Fields::LabeledField.new :label => label }
+  describe "parameters" do
 
-  context "labels" do
-    it "has label" do
-      assert field.respond_to? :label
+    it "returns all parameters passed to a filed" do
+      expected_params = {
+        :label => label,
+        :path => path,
+        :some => :parameter
+      }
+      field.parameters.must_equal expected_params
     end
 
-    it "is possible to set label in constructor" do
-      field.label.must_equal label
-    end
   end
 
-end
+  describe "display?" do
 
-describe Fields::DataField do
+    context "blank is allowed" do
 
-  let(:symbol_data) {{
-    :name => "John Doe",
-    :email => "john.doe@example.com",
-    :address => {
-      :city => {
-        :name => "New York",
-        :zip => "1234"
-      }
-    }
-  }}
-
-  let(:string_data) {{
-    "name" => "Eric Doe",
-    "email" => "eric.doe@example.com",
-    "address" => {
-      "city" => {
-        "name" => "Boston",
-        "zip" => "6789"
-      }
-    }
-  }}
+      it "returns false the value is nil" do
+        field.display?(nil).must_equal true
+      end
 
-  let(:label) { "Some Label" }
-  let(:path) { [:address, :city, :name] }
-  let(:field) { Fields::DataField.new :label => label, :path => path }
+      it "returns true when there is some data under the path" do
+        field.display?("some value").must_equal true
+      end
+    end
 
-  it "stores label from constructor" do
-    field.label.must_equal label
-  end
 
-  it "stores path from constructor" do
-    field.path.must_equal path
-  end
+    context "blank is not allowed" do
+
+      it "returns false the value is nil" do
+        blank_field.display?(nil).must_equal false
+      end
+
+      it "returns true when there is some data under the path" do
+        blank_field.display?("some value").must_equal true
+      end
+    end
 
-  it "default path should be empty array" do
-    Fields::DataField.new.path.must_equal []
   end
 
-  context "getting data" do
+  describe "hide_blank?" do
 
-    it "should pick correct value" do
-      field.get_value(symbol_data).must_equal symbol_data[:address][:city][:name]
+    it "defaults to false" do
+      Fields::Field.new.hide_blank?.must_equal false
     end
 
-    it "should pick correct value independent of key type" do
-      field.get_value(string_data).must_equal string_data["address"]["city"]["name"]
+    it "can be set to true in the constructor" do
+      blank_field.hide_blank?.must_equal true
     end
 
-    it "should pick correct value even if data contains empty key (#3352)" do
-      string_data[''] = {}
-      field.get_value(string_data).must_equal string_data["address"]["city"]["name"]
+  end
+
+end
+
+
+describe Fields::ContainerField do
+
+  describe "display?" do
+
+    context "blank is allowed" do
+      let(:field) { Fields::ContainerField.new :label => "Label" }
+
+      it "returns false the value is nil" do
+        field.display?(nil).must_equal true
+      end
+
+      it "returns false the value is empty array" do
+        field.display?([]).must_equal true
+      end
+
+      it "returns false the value is empty hash" do
+        field.display?({}).must_equal true
+      end
+
+      it "returns true when there is some data under the path" do
+        field.display?(["some value"]).must_equal true
+      end
+
     end
 
+    context "blank is not allowed" do
+      let(:field) { Fields::ContainerField.new :label => "Label", :hide_blank => true }
 
-  end
+      it "returns false the value is nil" do
+        field.display?(nil).must_equal false
+      end
 
-end
+      it "returns false the value is empty array" do
+        field.display?([]).must_equal false
+      end
 
+      it "returns false the value is empty hash" do
+        field.display?({}).must_equal false
+      end
 
+      it "returns true when there is some data under the path" do
+        field.display?(["some value"]).must_equal true
+      end
+
+    end
+
+  end
+
+end

data/test/unit/output/formatters_test.rb

@@ -91,3 +91,50 @@ describe HammerCLI::Output::Formatters::ListFormatter do
     formatter.format('some string').must_equal 'some string'
   end
 end
+
+
+describe HammerCLI::Output::Formatters::KeyValueFormatter do
+
+  let(:params) {
+    {
+      :name => "Name",
+      "value" => "Value",
+    }
+  }
+
+  it "serializes the value" do
+    formatter = HammerCLI::Output::Formatters::KeyValueFormatter.new
+    formatter.format(params).must_equal 'Name => Value'
+  end
+
+  it "returns empty string when the input is nil" do
+    formatter = HammerCLI::Output::Formatters::KeyValueFormatter.new
+    formatter.format(nil).must_equal ''
+  end
+
+  it "returns empty string value when the input is not a hash" do
+    formatter = HammerCLI::Output::Formatters::KeyValueFormatter.new
+    formatter.format('some string').must_equal ''
+  end
+end
+
+
+
+describe HammerCLI::Output::Formatters::LongTextFormatter do
+
+  it "prepends new line" do
+    formatter = HammerCLI::Output::Formatters::LongTextFormatter.new
+    formatter.format("Some\nmultiline\ntext").must_equal "\n  Some\n  multiline\n  text"
+  end
+
+  it "accepts nil" do
+    formatter = HammerCLI::Output::Formatters::LongTextFormatter.new
+    formatter.format(nil).must_equal "\n  "
+  end
+
+  it "enables to switch indentation off" do
+    formatter = HammerCLI::Output::Formatters::LongTextFormatter.new(:indent => false)
+    formatter.format("Some\nmultiline\ntext").must_equal "\nSome\nmultiline\ntext"
+  end
+
+end

data/test/unit/settings_test.rb

@@ -43,6 +43,12 @@ describe HammerCLI::Settings do
     settings.get(:b).must_equal 2
   end
 
+  it "dumps all settings" do
+    data = {:a => 1, :b => 2}
+    settings.load(data)
+    settings.dump.must_equal data
+  end
+
   it "merges settings on second load" do
     settings.load({:a => 1, :b => 2, :d => {:e => 4, :f => 5}})
     settings.load({:b => 'B', :c => 'C', :d => {:e => 'E'}})
@@ -60,7 +66,7 @@ describe HammerCLI::Settings do
     settings.get(:b).must_be_nil
   end
 
-  context "load from file" do
+  context "load from paths" do
 
     let(:config1) do
       config1 = Tempfile.new 'config'
@@ -85,22 +91,47 @@ describe HammerCLI::Settings do
     end
 
     it "loads settings from file" do
-      settings.load_from_file [config2.path, config1.path]
+      settings.load_from_paths [config2.path, config1.path]
       settings.get(:host).must_equal 'https://localhost.localdomain/'
       settings.get(:username).must_equal 'admin'
     end
 
     it "clears path history on clear invokation" do
-      settings.load_from_file [config2.path]
+      settings.load_from_paths [config2.path]
       settings.clear
       settings.path_history.must_equal []
     end
 
     it "store config path history" do
-      settings.load_from_file [config2.path, config1.path]
+      settings.load_from_paths [config2.path, config1.path]
       settings.path_history.must_equal [config1.path, config2.path]
     end
 
+    it "loads settings from dir" do
+      dir = Dir.mktmpdir
+      begin
+        File.open(File.join(dir, "cli_config.yml"), "w") do |f|
+          f.write(":param1: 'value1'\n")
+        end
+        module_dir = File.join(dir, 'cli.modules.d')
+        Dir.mkdir(module_dir)
+        File.open(File.join(module_dir, "cli_config.yml"), "w") do |f|
+          f.write(":module:\n")
+          f.write("  :param2: 'value2'\n")
+        end
+
+        settings.load_from_paths [config1.path, dir]
+
+        settings.get(:host).must_equal 'https://localhost/'
+        settings.get(:username).must_equal 'admin'
+        settings.get(:param1).must_equal 'value1'
+        settings.get(:module, :param2).must_equal 'value2'
+
+      ensure
+        FileUtils.remove_entry dir
+      end
+
+    end
   end
 end
 

data/test/unit/utils_test.rb

@@ -44,5 +44,50 @@ describe String do
 
   end
 
+  describe "indent" do
+
+    it "indents single line string" do
+      "line one".indent_with("  ").must_equal "  line one"
+    end
+
+    it "indents multi line string" do
+      "line one\nline two".indent_with("  ").must_equal "  line one\n  line two"
+    end
+
+  end
+
+  describe "interactive?" do
+
+    before :each do
+      @tty = STDOUT.tty?
+      STDOUT.stubs(:'tty?').returns(true)
+    end
+
+    after :each do
+      STDOUT.stubs(:'tty?').returns(@tty)
+    end
+
+    it "should be true when called in tty" do
+      HammerCLI::Settings.load({
+        :ui => { :interactive => nil },
+        :_params => { :interactive => nil } })
+      HammerCLI::interactive?.must_equal true
+    end
+
+    it "should by false when cli arg set" do
+      HammerCLI::Settings.load({
+        :ui => { :interactive => nil },
+        :_params => { :interactive => false } })
+      HammerCLI::interactive?.must_equal false
+    end
+
+    it "should by false when turned off in cfg" do
+      HammerCLI::Settings.load({
+        :ui => { :interactive => false },
+        :_params => { :interactive => nil } })
+      HammerCLI::interactive?.must_equal false
+    end
+  end
+
 end