hammer_cli 0.0.18 → 0.1.0

data/locale/zanata.xml

@@ -0,0 +1,29 @@
+<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+<config xmlns="http://zanata.org/namespace/config/">
+    <url>https://translate.zanata.org/zanata/</url> 
+    <project>satellite6-hammer-cli</project> 
+    <project-version>6.0</project-version>
+    <project-type>podir</project-type>
+    <locales>
+        <locale>fr</locale>
+        <locale>it</locale>
+        <locale>ja</locale>
+        <locale>ko</locale>
+        <locale>gu</locale>
+        <locale>hi</locale>
+        <locale>mr</locale>
+        <locale>or</locale>
+        <locale>ru</locale>
+        <locale>te</locale>
+        <locale>pa</locale>
+        <locale>kn</locale>
+        <locale>de</locale>
+        <locale>es</locale>
+        <locale>gl</locale>
+        <locale map-from="pt_BR">pt-BR</locale>
+        <locale map-from="bn">bn-IN</locale>
+        <locale map-from="ta">ta-IN</locale>
+        <locale map-from="zh_CN">zh-Hans-CN</locale>
+        <locale map-from="zh_TW">zh-Hant-TW</locale>      
+    </locales>
+</config>

data/test/unit/apipie/command_test.rb

@@ -1,15 +1,20 @@
 require File.join(File.dirname(__FILE__), '../test_helper')
-require File.join(File.dirname(__FILE__), 'fake_api')
 
 
 describe HammerCLI::Apipie::Command do
 
-  class ParentCommand < HammerCLI::Apipie::Command
+  class TestCommand < HammerCLI::Apipie::Command
+    def self.resource_config
+      { :apidoc_cache_dir => 'test/unit/fixtures/apipie', :apidoc_cache_name => 'architectures' }
+    end
+  end
+
+  class ParentCommand < TestCommand
     action :show
   end
 
-  class CommandA < HammerCLI::Apipie::Command
-    resource FakeApi::Resources::Architecture, :index
+  class CommandA < TestCommand
+    resource :architectures, :index
 
     class CommandB < ParentCommand
     end
@@ -18,17 +23,20 @@ describe HammerCLI::Apipie::Command do
   class CommandC < CommandA
   end
 
-
   let(:ctx) { { :adapter => :silent, :interactive => false } }
-  let(:cmd_class) { HammerCLI::Apipie::Command.dup }
+  let(:cmd_class) { TestCommand.dup }
   let(:cmd) { cmd_class.new("", ctx) }
 
+  before :each do
+    HammerCLI::Connection.drop_all
+  end
+
   context "setting identifiers" do
 
     let(:option_switches) { cmd_class.declared_options.map(&:switches).sort }
     let(:option_attribute_names) { cmd_class.declared_options.map(&:attribute_name).sort }
 
-    class Cmd1 < HammerCLI::Apipie::Command
+    class Cmd1 < TestCommand
       identifiers :id, :name, :label
       apipie_options
     end
@@ -107,31 +115,31 @@ describe HammerCLI::Apipie::Command do
   context "setting resources" do
 
     it "should set resource and action together" do
-      cmd_class.resource FakeApi::Resources::Architecture, :index
+      cmd_class.resource :architectures, :index
 
-      cmd.resource.resource_class.must_equal FakeApi::Resources::Architecture
-      cmd_class.resource.resource_class.must_equal FakeApi::Resources::Architecture
+      cmd.resource.name.must_equal :architectures
+      cmd_class.resource.name.must_equal :architectures
 
       cmd.action.must_equal :index
       cmd_class.action.must_equal :index
     end
 
     it "should set resource alone" do
-      cmd_class.resource FakeApi::Resources::Architecture
+      cmd_class.resource :architectures
 
-      cmd.resource.resource_class.must_equal FakeApi::Resources::Architecture
-      cmd_class.resource.resource_class.must_equal FakeApi::Resources::Architecture
+      cmd.resource.name.must_equal :architectures
+      cmd_class.resource.name.must_equal :architectures
 
       cmd.action.must_equal nil
       cmd_class.action.must_equal nil
     end
 
     it "should set resource and action alone" do
-      cmd_class.resource FakeApi::Resources::Architecture
+      cmd_class.resource :architectures
       cmd_class.action :index
 
-      cmd.resource.resource_class.must_equal FakeApi::Resources::Architecture
-      cmd_class.resource.resource_class.must_equal FakeApi::Resources::Architecture
+      cmd.resource.name.must_equal :architectures
+      cmd_class.resource.name.must_equal :architectures
 
       cmd.action.must_equal :index
       cmd_class.action.must_equal :index
@@ -145,25 +153,34 @@ describe HammerCLI::Apipie::Command do
 
     it "looks up resource in the class' modules" do
       cmd_b = CommandA::CommandB.new("", ctx)
-      cmd_b.resource.resource_class.must_equal FakeApi::Resources::Architecture
-      cmd_b.class.resource.resource_class.must_equal FakeApi::Resources::Architecture
+      cmd_b.resource.name.must_equal :architectures
+      cmd_b.class.resource.name.must_equal :architectures
     end
 
     it "looks up resource in the superclass" do
       cmd_c = CommandC.new("", ctx)
-      cmd_c.resource.resource_class.must_equal FakeApi::Resources::Architecture
-      cmd_c.class.resource.resource_class.must_equal FakeApi::Resources::Architecture
+      cmd_c.resource.name.must_equal :architectures
+      cmd_c.class.resource.name.must_equal :architectures
     end
   end
 
   context "apipie generated options" do
 
+    class DocumentedCommand < HammerCLI::Apipie::Command
+      def self.resource_config
+        { :apidoc_cache_dir => 'test/unit/fixtures/apipie', :apidoc_cache_name => 'documented' }
+      end
+    end
+
+    let(:cmd_class) { DocumentedCommand.dup }
+    let(:cmd) { cmd_class.new("", ctx) }
+
     context "with one simple param" do
 
       let(:option) { cmd_class.declared_options[0] }
 
       before :each do
-        cmd_class.resource FakeApi::Resources::Documented, :index
+        cmd_class.resource :documented, :index
         cmd_class.apipie_options
       end
 
@@ -186,7 +203,7 @@ describe HammerCLI::Apipie::Command do
 
     context "required options" do
       before :each do
-        cmd_class.resource FakeApi::Resources::Documented, :create
+        cmd_class.resource :documented, :create
         cmd_class.apipie_options
       end
 
@@ -199,7 +216,7 @@ describe HammerCLI::Apipie::Command do
 
     context "with hash params" do
       before :each do
-        cmd_class.resource FakeApi::Resources::Documented, :create
+        cmd_class.resource :documented, :create
         cmd_class.apipie_options
       end
 
@@ -214,7 +231,7 @@ describe HammerCLI::Apipie::Command do
 
     context "array params" do
       before :each do
-        cmd_class.resource FakeApi::Resources::Documented, :create
+        cmd_class.resource :documented, :create
         cmd_class.apipie_options
       end
 
@@ -244,7 +261,7 @@ describe HammerCLI::Apipie::Command do
 
     context "filtering options" do
       before :each do
-        cmd_class.resource FakeApi::Resources::Documented, :create
+        cmd_class.resource :documented, :create
       end
 
       it "should skip filtered options" do

data/test/unit/apipie/read_command_test.rb

@@ -1,10 +1,15 @@
 require File.join(File.dirname(__FILE__), '../test_helper')
-require File.join(File.dirname(__FILE__), 'fake_api')
 
 
 describe HammerCLI::Apipie::ReadCommand do
 
-  let(:cmd_class) { HammerCLI::Apipie::ReadCommand.dup }
+  class TestReadCommand < HammerCLI::Apipie::ReadCommand
+    def self.resource_config
+      { :apidoc_cache_dir => 'test/unit/fixtures/apipie', :apidoc_cache_name => 'architectures' }
+    end
+  end
+
+  let(:cmd_class) { TestReadCommand.dup }
   let(:cmd) { cmd_class.new("", { :adapter => :silent, :interactive => false }) }
   let(:cmd_run) { cmd.run([]) }
 
@@ -16,11 +21,9 @@ describe HammerCLI::Apipie::ReadCommand do
   context "resource defined" do
 
     before :each do
-      cmd_class.resource FakeApi::Resources::Architecture, "some_action"
-
-      arch = FakeApi::Resources::Architecture.new
-      arch.expects(:some_action).returns([])
-      FakeApi::Resources::Architecture.stubs(:new).returns(arch)
+      HammerCLI::Connection.drop_all
+      ApipieBindings::API.any_instance.stubs(:call).returns([])
+      cmd_class.resource :architectures, :index
     end
 
     it "should perform a call to api when resource is defined" do

data/test/unit/apipie/write_command_test.rb

@@ -1,11 +1,14 @@
 require File.join(File.dirname(__FILE__), '../test_helper')
-require File.join(File.dirname(__FILE__), 'fake_api')
 
 describe HammerCLI::Apipie::WriteCommand do
 
+  class TestWriteCommand < HammerCLI::Apipie::WriteCommand
+    def self.resource_config
+      { :apidoc_cache_dir => 'test/unit/fixtures/apipie', :apidoc_cache_name => 'architectures' }
+    end
+  end
 
-  let(:ctx) { { :interactive => false } }
-  let(:cmd) { HammerCLI::Apipie::WriteCommand.new("", ctx) }
+  let(:cmd) { TestWriteCommand.new("") }
   let(:cmd_run) { cmd.run([]) }
 
   it "should raise exception when no action is defined" do
@@ -16,11 +19,9 @@ describe HammerCLI::Apipie::WriteCommand do
   context "resource defined" do
 
     before :each do
-      cmd.class.resource FakeApi::Resources::Architecture, "some_action"
-
-      arch = FakeApi::Resources::Architecture.new
-      arch.expects(:some_action).returns([])
-      FakeApi::Resources::Architecture.stubs(:new).returns(arch)
+      HammerCLI::Connection.drop_all
+      ApipieBindings::API.any_instance.stubs(:call).returns([])
+      cmd.class.resource :architectures, :index
     end
 
     it "should perform a call to api when resource is defined" do

data/test/unit/completer_test.rb

@@ -2,40 +2,198 @@ require File.join(File.dirname(__FILE__), 'test_helper')
 require 'tempfile'
 
 
-describe HammerCLI::CompleterLine do
+describe HammerCLI::CompleterWord do
+
+
+  describe "quote" do
+    it "returns empty string for empty word" do
+      word = HammerCLI::CompleterWord.new('')
+      word.quote.must_equal ""
+    end
+
+    it "returns empty string for word without quotes" do
+      word = HammerCLI::CompleterWord.new('word')
+      word.quote.must_equal ""
+    end
+
+    it "recognizes double quotes" do
+      word = HammerCLI::CompleterWord.new('"word')
+      word.quote.must_equal '"'
+    end
+
+    it "recognizes single quotes" do
+      word = HammerCLI::CompleterWord.new('\'word')
+      word.quote.must_equal "'"
+    end
+  end
 
-  let(:unfinished_line) { "architecture list --name arch" }
-  let(:finished_line) { "architecture list --name arch " }
+  describe "quoted?" do
+    it "returns false for an unquoted word" do
+      word = HammerCLI::CompleterWord.new('word')
+      word.quoted?.must_equal false
+    end
+
+    it "returns true for double quotes" do
+      word = HammerCLI::CompleterWord.new('"word')
+      word.quoted?.must_equal true
+    end
+
+    it "returns true for single quotes" do
+      word = HammerCLI::CompleterWord.new('\'word')
+      word.quoted?.must_equal true
+    end
+  end
+
+  describe "complete?" do
+    it "considers a word without quotes complete" do
+      word = HammerCLI::CompleterWord.new('word')
+      word.complete?.must_equal false
+    end
+
+    it "considers a word without quotes ending with space incomplete" do
+      word = HammerCLI::CompleterWord.new('word ')
+      word.complete?.must_equal true
+    end
+
+    it "considers open double quotes incomplete" do
+      word = HammerCLI::CompleterWord.new('"word')
+      word.complete?.must_equal false
+    end
+
+    it "considers open double quotes with spaces incomplete" do
+      word = HammerCLI::CompleterWord.new('"word ')
+      word.complete?.must_equal false
+    end
+
+    it "considers closed double quotes complete" do
+      word = HammerCLI::CompleterWord.new('"word"')
+      word.complete?.must_equal true
+    end
+
+    it "considers open single quotes incomplete" do
+      word = HammerCLI::CompleterWord.new('\'word')
+      word.complete?.must_equal false
+    end
+
+    it "considers open single quotes with spaces incomplete" do
+      word = HammerCLI::CompleterWord.new('\'word ')
+      word.complete?.must_equal false
+    end
+
+    it "considers closed single quotes complete" do
+      word = HammerCLI::CompleterWord.new('\'word\'')
+      word.complete?.must_equal true
+    end
+  end
+
+end
+
+describe HammerCLI::CompleterLine do
 
   context "splitting words" do
 
     it "should split basic line" do
-      line = HammerCLI::CompleterLine.new(finished_line)
+      line = HammerCLI::CompleterLine.new("architecture  list --name arch")
       line.must_equal ["architecture",  "list", "--name", "arch"]
     end
 
     it "should split basic line with space at the end" do
-      line = HammerCLI::CompleterLine.new(finished_line)
+      line = HammerCLI::CompleterLine.new("architecture  list --name arch  ")
       line.must_equal ["architecture",  "list", "--name", "arch"]
     end
 
+    it "should split on equal sign" do
+      line = HammerCLI::CompleterLine.new("--name=arch")
+      line.must_equal ["--name", "arch"]
+    end
+
+    it "should split when last character is equal sign" do
+      line = HammerCLI::CompleterLine.new("--name=")
+      line.must_equal ["--name"]
+    end
+
+    it "should split on equal sign when quotes are used" do
+      line = HammerCLI::CompleterLine.new("--name='arch' ")
+      line.must_equal ["--name", "arch"]
+    end
+
+    it "should split line with single quotes" do
+      line = HammerCLI::CompleterLine.new("--name 'arch' ")
+      line.must_equal ["--name", "arch"]
+    end
+
+    it "should split line with double quotes" do
+      line = HammerCLI::CompleterLine.new("--name \"arch\"")
+      line.must_equal ["--name", "arch"]
+    end
+
+    it "should split line with single quotes and space between" do
+      line = HammerCLI::CompleterLine.new("--name 'ar ch '")
+      line.must_equal ["--name", "ar ch "]
+    end
+
+    it "should split line with one single quote and space between" do
+      line = HammerCLI::CompleterLine.new("--name 'ar ch ")
+      line.must_equal ["--name", "ar ch "]
+    end
+
+    it "should split line with double quotes and space between" do
+      line = HammerCLI::CompleterLine.new("--name \"ar ch \"")
+      line.must_equal ["--name", "ar ch "]
+    end
+
+    it "should split line with one double quote and space between" do
+      line = HammerCLI::CompleterLine.new("--name \"ar ch ")
+      line.must_equal ["--name", "ar ch "]
+    end
+
   end
 
-  context "last word finished" do
+  context "line complete" do
+
+    it "should recongize incomplete line" do
+      line = HammerCLI::CompleterLine.new("architecture  list --name arch")
+      line.complete?.must_equal false
+    end
+
+    it "should recongize complete line" do
+      line = HammerCLI::CompleterLine.new("architecture  list --name arch  ")
+      line.complete?.must_equal true
+    end
+
+    it "should recongize complete line that ends with quotes" do
+      line = HammerCLI::CompleterLine.new("--name 'ar ch'")
+      line.complete?.must_equal true
+    end
 
-    it "should recongize unfinished line" do
-      line = HammerCLI::CompleterLine.new(unfinished_line)
-      line.finished?.must_equal false
+    it "should recongize complete line that ends with quotes followed by space" do
+      line = HammerCLI::CompleterLine.new("--name 'ar ch' ")
+      line.complete?.must_equal true
     end
 
-    it "should recongize finished line" do
-      line = HammerCLI::CompleterLine.new(finished_line)
-      line.finished?.must_equal true
+    it "should recongize complete line that ends with double quotes" do
+      line = HammerCLI::CompleterLine.new("--name \"ar ch\"")
+      line.complete?.must_equal true
     end
 
-    it "should recongize empty line as finished" do
+    it "should recongize one quote as incomplete" do
+      line = HammerCLI::CompleterLine.new("--name '")
+      line.complete?.must_equal false
+    end
+
+    it "should recongize one quote followed by space as incomplete" do
+      line = HammerCLI::CompleterLine.new("--name ' ")
+      line.complete?.must_equal false
+    end
+
+    it "should recongize one double quote as incomplete" do
+      line = HammerCLI::CompleterLine.new("--name \"")
+      line.complete?.must_equal false
+    end
+
+    it "should recongize empty line as complete" do
       line = HammerCLI::CompleterLine.new("")
-      line.finished?.must_equal true
+      line.complete?.must_equal true
     end
 
   end
@@ -55,7 +213,7 @@ describe HammerCLI::Completer do
       end
 
       def complete(val)
-        ["small ", "tall "]
+        ["small ", "tall ", "smel ly"]
       end
 
     end
@@ -103,6 +261,9 @@ describe HammerCLI::Completer do
   let(:completer) { HammerCLI::Completer.new(FakeMainCmd) }
 
   context "command completion" do
+
+    let(:ape_completions) { ["makkak ", "malpa ", "orangutan ", "--hairy ", "--weight ", "--height ", "-h ", "--help "] }
+
     it "should offer all available commands" do
       completer.complete("").sort.must_equal ["anabolic ", "ape ", "apocalypse ", "beast ", "-h ", "--help "].sort
     end
@@ -120,30 +281,54 @@ describe HammerCLI::Completer do
     end
 
     it "should offer all available subcommands and options" do
-      completer.complete("ape ").sort.must_equal ["makkak ", "malpa ", "orangutan ", "--hairy ", "--weight ", "--height ", "-h ", "--help "].sort
+      completer.complete("ape ").sort.must_equal ape_completions.sort
     end
 
     it "should offer all available subcommands and options even if a flag has been passed" do
-      completer.complete("ape --hairy ").sort.must_equal ["makkak ", "malpa ", "orangutan ", "--hairy ", "--weight ", "--height ", "-h ", "--help "].sort
+      completer.complete("ape --hairy ").sort.must_equal ape_completions.sort
     end
 
     it "should offer all available subcommands and options even if an option has been passed" do
-      completer.complete("ape --weight 12kg ").sort.must_equal ["makkak ", "malpa ", "orangutan ", "--hairy ", "--weight ", "--height ", "-h ", "--help "].sort
+      completer.complete("ape --weight 12kg ").sort.must_equal ape_completions.sort
     end
 
     it "should offer all available subcommands and options even if an egual sign option has been passed" do
-      completer.complete("ape --weight=12kg ").sort.must_equal ["makkak ", "malpa ", "orangutan ", "--hairy ", "--weight ", "--height ", "-h ", "--help "].sort
+      completer.complete("ape --weight=12kg ").sort.must_equal ape_completions.sort
+    end
+
+    it "should offer all available subcommands and options when quoted value was passed" do
+      completer.complete("ape --weight '12 kg' ").sort.must_equal ape_completions.sort
+    end
+
+    it "should offer all available subcommands and options when double quoted value was passed" do
+      completer.complete("ape --weight \"12 kg\" ").sort.must_equal ape_completions.sort
+    end
+
+    it "should offer all available subcommands and options when quoted value with equal sign was passed" do
+      completer.complete("ape --weight='12 kg' ").sort.must_equal ape_completions.sort
     end
   end
 
 
   context "option value completion" do
     it "should complete option values" do
-      completer.complete("ape --height ").sort.must_equal ["small ", "tall "].sort
+      completer.complete("ape --height ").sort.must_equal ["small ", "tall ", "smel ly"].sort
+    end
+
+    it "should complete option values when equal sign is used" do
+      completer.complete("ape --height=").sort.must_equal ["small ", "tall ", "smel ly"].sort
     end
 
     it "should complete option values" do
-      completer.complete("ape --height s").must_equal ["small "]
+      completer.complete("ape --height s").must_equal ["small ", "smel ly"]
+    end
+
+    it "should complete quoted option values" do
+      completer.complete("ape --height 's").must_equal ["'small' ", "'smel ly"]
+    end
+
+    it "should complete quoted option values" do
+      completer.complete("ape --height 'smel l").must_equal ["'smel ly"]
     end
   end