hammer_cli 0.11.0 → 0.12.0

data/test/reports/TEST-HammerCLI-ShellHistory-loading-old-history.xml

@@ -0,0 +1,11 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuite name="HammerCLI::ShellHistory::loading old history" tests="2" time="0.000591805" failures="0" errors="0" skipped="0" assertions="2">
+  <testcase name="test_0001_skips loading if the file does not exist" time="0.000228018" assertions="1">
+  </testcase>
+  <testcase name="test_0002_preseeds readline's history" time="0.000185292" assertions="1">
+  </testcase>
+  <system-out>
+  </system-out>
+  <system-err>
+  </system-err>
+</testsuite>

data/test/reports/TEST-HammerCLI-ShellHistory-saving-history.xml

@@ -0,0 +1,15 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuite name="HammerCLI::ShellHistory::saving history" tests="4" time="0.000956469" failures="0" errors="0" skipped="0" assertions="4">
+  <testcase name="test_0001_creates history file if it does not exist" time="0.00022899" assertions="1">
+  </testcase>
+  <testcase name="test_0003_appends to readline's history" time="0.000185984" assertions="1">
+  </testcase>
+  <testcase name="test_0002_appends history to the given file" time="0.000169983" assertions="1">
+  </testcase>
+  <testcase name="test_0004_doesn't save exit command" time="0.000154798" assertions="1">
+  </testcase>
+  <system-out>
+  </system-out>
+  <system-err>
+  </system-err>
+</testsuite>

data/test/reports/TEST-HammerCLI-ShellHistory.xml

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

data/test/reports/TEST-MiniTest-Spec.xml

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

data/test/reports/TEST-String-camelize.xml

@@ -0,0 +1,11 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuite name="String::camelize" tests="2" time="0.000337952" failures="0" errors="0" skipped="0" assertions="2">
+  <testcase name="test_0001_should camelize string with underscores" time="0.0001091" assertions="1">
+  </testcase>
+  <testcase name="test_0002_should not camelize string with dashes" time="5.1678e-05" assertions="1">
+  </testcase>
+  <system-out>
+  </system-out>
+  <system-err>
+  </system-err>
+</testsuite>

data/test/reports/TEST-String-formatting.xml

@@ -0,0 +1,17 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuite name="String::formatting" tests="5" time="0.00052804" failures="0" errors="0" skipped="0" assertions="5">
+  <testcase name="test_0003_should replace named parameters" time="0.000120653" assertions="1">
+  </testcase>
+  <testcase name="test_0002_should replace positional parameters" time="5.0694e-05" assertions="1">
+  </testcase>
+  <testcase name="test_0004_should replace named parameters with string keys" time="5.4219e-05" assertions="1">
+  </testcase>
+  <testcase name="test_0005_should replace named parameters marked with curly brackets" time="5.3821e-05" assertions="1">
+  </testcase>
+  <testcase name="test_0001_should not fail without expected parameters" time="5.197e-05" assertions="1">
+  </testcase>
+  <system-out>
+  </system-out>
+  <system-err>
+  </system-err>
+</testsuite>

data/test/reports/TEST-String-indent.xml

@@ -0,0 +1,11 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuite name="String::indent" tests="2" time="0.000342954" failures="0" errors="0" skipped="0" assertions="2">
+  <testcase name="test_0002_indents multi line string" time="0.000108296" assertions="1">
+  </testcase>
+  <testcase name="test_0001_indents single line string" time="4.4031e-05" assertions="1">
+  </testcase>
+  <system-out>
+  </system-out>
+  <system-err>
+  </system-err>
+</testsuite>

data/test/reports/TEST-String-interactive-.xml

@@ -0,0 +1,13 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuite name="String::interactive?" tests="3" time="0.001375686" failures="0" errors="0" skipped="0" assertions="3">
+  <testcase name="test_0001_should be true when called in tty" time="0.00042329" assertions="1">
+  </testcase>
+  <testcase name="test_0002_should by false when cli arg set" time="0.000403856" assertions="1">
+  </testcase>
+  <testcase name="test_0003_should by false when turned off in cfg" time="0.000364321" assertions="1">
+  </testcase>
+  <system-out>
+  </system-out>
+  <system-err>
+  </system-err>
+</testsuite>

data/test/reports/TEST-String.xml

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

data/test/reports/TEST-constraints-HammerCLI-Validator-AllConstraint-exist-.xml

@@ -0,0 +1,13 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuite name="constraints::HammerCLI::Validator::AllConstraint::exist?" tests="3" time="0.000590622" failures="0" errors="0" skipped="0" assertions="3">
+  <testcase name="test_0002_should return true when all the options exist" time="0.000196018" assertions="1">
+  </testcase>
+  <testcase name="test_0001_should return true when no options are passed" time="0.000113036" assertions="1">
+  </testcase>
+  <testcase name="test_0003_should return false when one of the options is missing" time="9.6605e-05" assertions="1">
+  </testcase>
+  <system-out>
+  </system-out>
+  <system-err>
+  </system-err>
+</testsuite>

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

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

data/test/reports/TEST-constraints-HammerCLI-Validator-AnyConstraint-exist-.xml

@@ -0,0 +1,13 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuite name="constraints::HammerCLI::Validator::AnyConstraint::exist?" tests="3" time="0.000586351" failures="0" errors="0" skipped="0" assertions="3">
+  <testcase name="test_0002_should return true when one of the options exist" time="0.000170553" assertions="1">
+  </testcase>
+  <testcase name="test_0001_should return true when no options are passed" time="0.000112862" assertions="1">
+  </testcase>
+  <testcase name="test_0003_should return false when all the options are missing" time="0.000111747" assertions="1">
+  </testcase>
+  <system-out>
+  </system-out>
+  <system-err>
+  </system-err>
+</testsuite>

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

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

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

@@ -0,0 +1,9 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<testsuite name="constraints::HammerCLI::Validator::BaseConstraint::exist?" tests="1" time="0.000343672" failures="0" errors="0" skipped="0" assertions="1">
+  <testcase name="test_0001_throws not implemented error" time="0.000171895" assertions="1">
+  </testcase>
+  <system-out>
+  </system-out>
+  <system-err>
+  </system-err>
+</testsuite>

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

@@ -1,7 +1,6 @@
 require File.join(File.dirname(__FILE__), 'test_helper')
 require 'tempfile'
 
-
 describe HammerCLI::AbstractCommand do
 
   context "output" do
@@ -285,6 +284,16 @@ describe HammerCLI::AbstractCommand do
 
   end
 
+  describe "#options" do
+    class TestOptionSourceCommand < HammerCLI::AbstractCommand;end
+
+    it "allows overriding option sources in a command" do
+      cmd = TestOptionSourceCommand.new('')
+      cmd.expects(:option_sources).returns([])
+      cmd.send(:all_options)
+    end
+  end
+
   describe "option builder" do
 
     it "uses builder container as default" do

data/test/unit/apipie/option_builder_test.rb

@@ -97,6 +97,12 @@ describe HammerCLI::Apipie::OptionBuilder do
       numeric_option = options.find {|o| o.attribute_name == HammerCLI.option_accessor_name("numeric_param") }
       numeric_option.value_formatter.class.must_equal HammerCLI::Options::Normalizers::Number
     end
+
+    it "should set number normalizer for integer" do
+      integer_option = options.find {|o| o.attribute_name == HammerCLI.option_accessor_name("integer_param") }
+      integer_option.value_formatter.class.must_equal HammerCLI::Options::Normalizers::Number
+    end
+
   end
 
 

data/test/unit/defaults_test.rb

@@ -13,12 +13,19 @@ describe HammerCLI::Defaults do
   describe '#add_defaults_to_conf' do
     it "Should add a default param to defaults file, without a provider" do
       defaults_result = @defaults.add_defaults_to_conf({"organization_id"=> 3}, nil)
-      assert_equal 3, defaults_result[:defaults][:organization_id][:value]
+      assert_equal '3', defaults_result[:defaults][:organization_id][:value]
+    end
+
+    it "adds values from hammer shell" do
+      key = HammerCLI::CompleterWord.new("organization_id")
+      value = HammerCLI::CompleterWord.new("3")
+      defaults_result = @defaults.add_defaults_to_conf({ key => value }, nil)
+      assert_equal '3', defaults_result[:defaults][:organization_id][:value]
     end
 
     it "Should update dashed default when underscored default is set" do
       defaults_result = @defaults.add_defaults_to_conf({"location-id"=> 3}, nil)
-      assert_equal 3, defaults_result[:defaults][:'location-id'][:value]
+      assert_equal '3', defaults_result[:defaults][:'location-id'][:value]
       assert_equal nil, defaults_result[:defaults][:location_id]
     end
 
@@ -26,20 +33,20 @@ describe HammerCLI::Defaults do
       let(:filepath) { File.join(File.dirname(__FILE__), '/fixtures/defaults/defaults_dashed.yml') }
       it "Should update underscored default when dashed default is set" do
         defaults_result = @defaults.add_defaults_to_conf({"location_id"=> 3}, nil)
-        assert_equal 3, defaults_result[:defaults][:location_id][:value]
+        assert_equal '3', defaults_result[:defaults][:location_id][:value]
         assert_equal nil, defaults_result[:defaults][:'location-id']
       end
 
       it "should update underscored default with option syntax when dashed default is set" do
         defaults_result = @defaults.add_defaults_to_conf({"--location-id"=> 3}, nil)
-        assert_equal 3, defaults_result[:defaults][:'location-id'][:value]
+        assert_equal '3', defaults_result[:defaults][:'location-id'][:value]
         assert_equal nil, defaults_result[:defaults][:location_id]
       end
     end
 
     it "Should add a default param to defaults file, with provider" do
       defaults_result = @defaults.add_defaults_to_conf({"location_id"=>nil}, :foreman)
-      assert_equal :foreman, defaults_result[:defaults][:location_id][:provider]
+      assert_equal 'foreman', defaults_result[:defaults][:location_id][:provider]
     end
   end
 

data/test/unit/exception_handler_test.rb

@@ -36,7 +36,7 @@ describe HammerCLI::ExceptionHandler do
   end
 
   it "should handle usage error" do
-    output.expects(:print_error).with(heading, "Error: wrong_usage\n\nSee: 'command_name --help'")
+    output.expects(:print_error).with(heading, "Error: wrong_usage\n\nSee: 'command_name --help'.")
     handler.handle_exception(Clamp::UsageError.new('wrong_usage', cmd), :heading => heading)
 
   end

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

@@ -162,7 +162,16 @@
                                 "allow_null": false,
                                 "full_name": "documented[numeric_param]",
                                 "validator": "Must be Number",
-                                "expected_type": "number",
+                                "expected_type": "numeric",
+                                "description": "",
+                                "required": false
+                            },
+                            {
+                                "name": "integer_param",
+                                "allow_null": false,
+                                "full_name": "documented[numeric_param]",
+                                "validator": "Integer",
+                                "expected_type": "numeric",
                                 "description": "",
                                 "required": false
                             }

data/test/unit/help/builder_test.rb

@@ -23,6 +23,22 @@ describe HammerCLI::Help::Builder do
     end
   end
 
+  describe 'adding an option with lower case description' do
+    it 'capitalizes the description' do
+      options = [
+        Clamp::Option::Definition.new(['--alpha-option'], 'OPT_ALPHA', 'alpha description'),
+        Clamp::Option::Definition.new(['--beta-option'], 'OPT_BETA', 'BETA description')
+      ]
+      help.add_list('Options',options)
+
+      help.string.strip.must_equal [
+        'Options:',
+        ' --alpha-option OPT_ALPHA      Alpha description',
+        ' --beta-option OPT_BETA        BETA description'
+      ].join("\n")
+    end
+  end
+
   describe 'adding text' do
     let(:content_1) {[
       'Lorem ipsum dolor sit amet, consectetur adipiscing elit.',
@@ -54,4 +70,3 @@ describe HammerCLI::Help::Builder do
     end
   end
 end
-

data/test/unit/messages_test.rb

@@ -0,0 +1,7 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+require 'hammer_cli/testing/messages'
+
+include HammerCLI::Testing::Messages
+describe "message formatting" do
+  check_all_command_messages(HammerCLI::MainCommand)
+end

data/test/unit/modules_test.rb

@@ -35,16 +35,6 @@ describe HammerCLI::Modules do
       HammerCLI::Settings.clear
       HammerCLI::Modules.names.must_equal []
     end
-
-    it "must work with old modules config" do
-      HammerCLI::Settings.clear
-      HammerCLI::Settings.load({
-        :tom => {},
-        :modules => ['hammer_cli_tom', 'hammer_cli_jerry'],
-      })
-      HammerCLI::Modules.names.must_equal ["hammer_cli_jerry", "hammer_cli_tom"]
-    end
-
   end
 
   describe "find by name" do
@@ -95,7 +85,7 @@ describe HammerCLI::Modules do
       it "must log module's name and version" do
         HammerCLI::Modules.expects(:require_module).with("hammer_cli_tom")
         HammerCLI::Modules.load("hammer_cli_tom")
-        @log_output.readline.strip.must_equal "INFO  Modules : Extension module hammer_cli_tom (0.0.1) loaded"
+        @log_output.readline.strip.must_equal "INFO  Modules : Extension module hammer_cli_tom (0.0.1) loaded."
       end
 
       it "must return true when load succeeds" do
@@ -110,7 +100,7 @@ describe HammerCLI::Modules do
     describe "module not found" do
       before :each do
         HammerCLI::Modules.stubs(:require_module).raises(LoadError)
-        @error_msg = "ERROR  Modules : Error while loading module hammer_cli_tom"
+        @error_msg = "ERROR  Modules : Error while loading module hammer_cli_tom."
       end
 
       it "must log an error if the load! fails" do
@@ -137,8 +127,8 @@ describe HammerCLI::Modules do
     describe "module runtime exception" do
       before :each do
         HammerCLI::Modules.stubs(:require_module).raises(RuntimeError)
-        @error_msg = "ERROR  Modules : Error while loading module hammer_cli_tom"
-        @warning_msg = "Warning: An error occured while loading module hammer_cli_tom"
+        @error_msg = "ERROR  Modules : Error while loading module hammer_cli_tom."
+        @warning_msg = "Warning: An error occured while loading module hammer_cli_tom."
       end
 
       it "must log an error if the load! fails" do
@@ -167,4 +157,3 @@ describe HammerCLI::Modules do
   end
 
 end
-

data/test/unit/options/normalizers_test.rb

@@ -3,7 +3,6 @@ require File.join(File.dirname(__FILE__), '../test_helper')
 
 describe HammerCLI::Options::Normalizers do
 
-
   describe 'abstract' do
 
     let(:formatter) { HammerCLI::Options::Normalizers::AbstractNormalizer.new }
@@ -14,6 +13,27 @@ describe HammerCLI::Options::Normalizers do
 
   end
 
+  describe 'default' do
+  
+    let(:formatter) { HammerCLI::Options::Normalizers::Default.new }
+  
+    it "should not change any value" do
+      formatter.format('value').must_equal 'value'
+    end
+  
+    it "should not change nil value" do
+      formatter.format(nil).must_be_nil
+      end
+    
+    it "has empty description" do
+      formatter.description.must_equal ''
+    end
+ 
+    it "has empty completion" do
+      formatter.complete('test').must_equal []
+    end
+  end
+
   describe 'list' do
 
     let(:formatter) { HammerCLI::Options::Normalizers::List.new }

data/test/unit/options/option_collector_test.rb

@@ -3,7 +3,7 @@ require_relative '../test_helper'
 describe HammerCLI::Options::OptionCollector do
   before do
     recognised_options = [mock, mock]
-    source1_result = {:option1 => 1, :option2 => nil}
+    source1_result = {:option1 => 1, :option2 => nil, :option4 => HammerCLI::NilValue}
     source2_result = source1_result.merge({:option3 => 3})
 
     source1 = mock
@@ -16,14 +16,20 @@ describe HammerCLI::Options::OptionCollector do
   end
 
   describe '#options' do
-    it 'returns options without nil values' do
-      assert_equal({:option1 => 1, :option3 => 3}, @collector.options)
+    it 'returns options without nil values but with NIL values' do
+      assert_equal({:option1 => 1, :option3 => 3, :option4 => nil}, @collector.options)
     end
   end
 
   describe '#all_options' do
     it 'returns all options' do
-      assert_equal({:option1 => 1, :option2 => nil, :option3 => 3}, @collector.all_options)
+      assert_equal({:option1 => 1, :option2 => nil, :option3 => 3, :option4 => nil}, @collector.all_options)
+    end
+  end
+  
+  describe '#all_options_raw' do
+    it 'returns all options with NIL values untranslated' do
+      assert_equal({:option1 => 1, :option2 => nil, :option3 => 3, :option4 => HammerCLI::NilValue}, @collector.all_options_raw)
     end
   end
 end

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

@@ -55,6 +55,35 @@ describe HammerCLI::Output::Adapter::CSValues do
       end
     end
 
+    context "handle headers" do
+      let(:empty_data) { HammerCLI::Output::RecordCollection.new [] }
+
+      it "should print headers by default" do
+        out, err = capture_io { adapter.print_collection(fields, data) }
+        out.must_match(/.*Name.*/)
+      end
+
+      it "should print headers by default even if there is no data" do
+        out, err = capture_io { adapter.print_collection(fields, empty_data) }
+        out.must_match(/.*Name.*/)
+      end
+
+      it "should print data only when --no-headers is set" do
+        expected_output = [
+                           "John Doe,2000",
+                           "",
+                          ].join("\n")
+        adapter = HammerCLI::Output::Adapter::CSValues.new( { :no_headers => true } )
+        proc { adapter.print_collection(fields, data) }.must_output(expected_output)
+      end
+
+      it "should print nothing when --no-headers is set but no data" do
+        expected_output = "\n"
+        adapter = HammerCLI::Output::Adapter::CSValues.new( { :no_headers => true } )
+        proc { adapter.print_collection(fields, empty_data) }.must_output(expected_output)
+      end
+    end
+
     context "handle fields with containers" do
       let(:demographics) {
         Fields::Label.new(:path => [], :label => "Demographics") do

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

@@ -96,6 +96,33 @@ describe HammerCLI::Output::Adapter::Table do
       end
     end
 
+    context "handle headers" do
+      it "should print headers by default" do
+        out, err = capture_io { adapter.print_collection(fields, data) }
+        out.must_match(/.*NAME.*/)
+      end
+
+      it "should print headers by default even if there is no data" do
+        out, err = capture_io { adapter.print_collection(fields, empty_data) }
+        out.must_match(/.*NAME.*/)
+      end
+
+      it "should print data only when --no-headers is set" do
+        expected_output = [
+                           "John Doe",
+                           "",
+                          ].join("\n")
+        adapter = HammerCLI::Output::Adapter::Table.new( { :no_headers => true } )
+        proc { adapter.print_collection(fields, data) }.must_output(expected_output)
+      end
+
+      it "should print nothing when --no-headers is set but no data" do
+        expected_output = ""
+        adapter = HammerCLI::Output::Adapter::Table.new( { :no_headers => true } )
+        proc { adapter.print_collection(fields, empty_data) }.must_output(expected_output)
+      end
+    end
+
     context "column width" do
 
       it "calculates correct width of two-column characters" do

data/test/unit/validator_test.rb

@@ -145,7 +145,7 @@ describe "constraints" do
       it "raises exception when the option is present" do
         constraint = cls.new(options, :option_a)
         e = proc{ constraint.rejected }.must_raise HammerCLI::Validator::ValidationError
-        e.message.must_equal "You can't set option --option-a"
+        e.message.must_equal "You can't set option --option-a."
       end
     end
 
@@ -158,7 +158,7 @@ describe "constraints" do
       it "raises exception when the option is present" do
         constraint = cls.new(options, :option_unset_d)
         e = proc{ constraint.required }.must_raise HammerCLI::Validator::ValidationError
-        e.message.must_equal 'Option --option-unset-d is required'
+        e.message.must_equal 'Option --option-unset-d is required.'
       end
     end
 
@@ -257,13 +257,13 @@ describe "constraints" do
       it "raises exception when none of the options is present" do
         constraint = cls.new(options, [:option_unset_d, :option_unset_e])
         e = proc{ constraint.required }.must_raise HammerCLI::Validator::ValidationError
-        e.message.must_equal 'One of options --option-unset-d, --option-unset-e is required'
+        e.message.must_equal 'One of options --option-unset-d, --option-unset-e is required.'
       end
 
       it "raises exception when more than one of the options is present" do
         constraint = cls.new(options, [:option_a, :option_b])
         e = proc{ constraint.required }.must_raise HammerCLI::Validator::ValidationError
-        e.message.must_equal 'Only one of options --option-a, --option-b can be set'
+        e.message.must_equal 'Only one of options --option-a, --option-b can be set.'
       end
     end
   end