methadone 1.9.5 → 2.0.0

data/test/integration/test_version.rb

@@ -0,0 +1,21 @@
+require_relative "base_integration_test"
+
+include FileUtils
+
+class TestVersion < BaseIntegrationTest
+  test_that "--help shows the gem version" do
+    Given { methadone "newgem" }
+    When { @stdout, _, = run_app "newgem", "--help" }
+    Then {
+      assert_match(/v\d+\.\d+\.\d+/, @stdout)
+    }
+  end
+
+  test_that "--version shows the gem version" do
+    Given { methadone "newgem" }
+    When { @stdout, _, = run_app "newgem", "--version" }
+    Then {
+      assert_match(/v\d+\.\d+\.\d+/, @stdout)
+    }
+  end
+end

data/test/{execution_strategy → unit/execution_strategy}/test_jvm.rb

@@ -53,9 +53,9 @@ module ExecutionStrategy
         @results = @strategy.run_command(@command)
       }
       Then {
-        @results[0].should == @stdout
-        @results[1].should == @stderr
-        @results[2].exitstatus.should == @exitstatus
+        @results[0].should be == @stdout
+        @results[1].should be == @stderr
+        @results[2].exitstatus.should be == @exitstatus
       }
     end
 
@@ -65,7 +65,7 @@ module ExecutionStrategy
         @klass = @strategy.exception_meaning_command_not_found
       }
       Then {
-        @klass.should == NativeException
+        @klass.should be == NativeException
       }
     end
 

data/test/{execution_strategy → unit/execution_strategy}/test_open_3.rb

@@ -23,8 +23,8 @@ module ExecutionStrategy
         @results = @strategy.run_command(@command)
       }
       Then {
-        @results[0].should == @stdout
-        @results[1].should == @stderr
+        @results[0].should be == @stdout
+        @results[1].should be == @stderr
         @results[2].should be @status
       }
     end
@@ -46,8 +46,8 @@ module ExecutionStrategy
         @results = @strategy.run_command(@command)
       }
       Then {
-        @results[0].should == @stdout
-        @results[1].should == @stderr
+        @results[0].should be == @stdout
+        @results[1].should be == @stderr
         @results[2].should be @status
       }
     end
@@ -58,7 +58,7 @@ module ExecutionStrategy
         @klass = @strategy.exception_meaning_command_not_found
       }
       Then {
-        @klass.should == Errno::ENOENT
+        @klass.should be == Errno::ENOENT
       }
     end
 

data/test/{execution_strategy → unit/execution_strategy}/test_open_4.rb

@@ -33,8 +33,8 @@ module ExecutionStrategy
         @results = @strategy.run_command(@command)
       }
       Then {
-        @results[0].should == @stdout
-        @results[1].should == @stderr
+        @results[0].should be == @stdout
+        @results[1].should be == @stderr
         @results[2].should be @status
       }
     end
@@ -62,8 +62,8 @@ module ExecutionStrategy
         @results = @strategy.run_command(@command)
       }
       Then {
-        @results[0].should == @stdout
-        @results[1].should == @stderr
+        @results[0].should be == @stdout
+        @results[1].should be == @stderr
         @results[2].should be @status
       }
     end
@@ -74,7 +74,7 @@ module ExecutionStrategy
         @klass = @strategy.exception_meaning_command_not_found
       }
       Then {
-        @klass.should == Errno::ENOENT
+        @klass.should be == Errno::ENOENT
       }
     end
 

data/test/unit/test/test_integration_test_assertions.rb

@@ -0,0 +1,211 @@
+require 'base_test'
+require 'methadone'
+require 'methadone/test/integration_test_assertions'
+require 'fileutils'
+require 'tmpdir'
+
+class TestIntegrationTestAssertions < BaseTest
+  include Methadone::IntegrationTestAssertions
+  include FileUtils
+
+  def setup
+    @pwd = pwd
+    @tmpdir = Dir.mktmpdir
+    chdir @tmpdir
+  end
+
+  def teardown
+    chdir @pwd
+    rm_rf @tmpdir
+  end
+
+  test_that "assert_file works with one regexp" do
+    Given { some_file }
+    When {
+      @code = ->() { assert_file("some_file.txt", contains: /foo/) }
+    }
+    Then { refute_raises(&@code) }
+  end
+
+  test_that "assert_file works with many regexps" do
+    Given { some_file }
+    When {
+      @code = ->() { assert_file("some_file.txt", contains: [ /foo/, /bar/ ]) }
+    }
+    Then { refute_raises(&@code) }
+  end
+
+  test_that "assert_file fails if any regexp fails to match" do
+    Given { some_file }
+    When {
+      @code = ->() { assert_file("some_file.txt", contains: [ /foo/, /baz/ ]) }
+    }
+    Then { assert_raises(&@code) }
+  end
+
+  test_that "assert_banner without takes_options passes for a banner with the bin name and no '[options]'" do
+    Given {
+      @bin_name = "foobar"
+      @stdout = "Usage: foobar"
+    }
+    When {
+      @code = ->() { assert_banner(@stdout,@bin_name,takes_options: false) }
+    }
+    Then { refute_raises(&@code) }
+  end
+
+  test_that "assert_banner without takes_options fails for a banner with the bin name and '[options]'" do
+    Given {
+      @bin_name = "foobar"
+      @stdout = "Usage: foobar [options]"
+    }
+    When {
+      @code = ->() { assert_banner(@stdout,@bin_name,takes_options: false) }
+    }
+    Then { assert_raises(&@code) }
+  end
+
+  test_that "assert_banner with takes_options passes for a banner with the bin name and '[options]'" do
+    Given {
+      @bin_name = "foobar"
+      @stdout = "Usage: foobar [options]\nOptions\n  --help"
+    }
+    When {
+      @code = ->() { assert_banner(@stdout,@bin_name,takes_options: true) }
+    }
+    Then { refute_raises(&@code) }
+  end
+
+  test_that "assert_banner with takes_options fails for a banner with the bin name but no '[options]'" do
+    Given {
+      @bin_name = "foobar"
+      @stdout = "Usage: foobar\nOptions\n  --help"
+    }
+    When {
+      @code = ->() { assert_banner(@stdout,@bin_name,takes_options: true) }
+    }
+    Then { assert_raises(&@code) }
+  end
+
+  test_that "assert_banner with takes_options and takes_arguments passes for a banner with the bin name, '[options]' and the arg list" do
+    Given {
+      @bin_name = "foobar"
+      @stdout = "Usage: foobar [options] some_arg [some_other_arg]\nOptions\n  --help"
+    }
+    When {
+      @code = ->() {
+        assert_banner(@stdout,
+                      @bin_name,
+                      takes_options: true,
+                      takes_arguments: { some_arg: :required, some_other_arg: :optional }
+                     )
+      }
+    }
+    Then { refute_raises(&@code) }
+  end
+
+  test_that "assert_banner with takes_options and takes_arguments failes for a banner with the bin name, '[options]' and no arg list" do
+    Given {
+      @bin_name = "foobar"
+      @stdout = "Usage: foobar [options]\nOptions\n  --help"
+    }
+    When {
+      @code = ->() {
+        assert_banner(@stdout,
+                      @bin_name,
+                      takes_options: true,
+                      takes_arguments: { some_arg: :required, some_other_arg: :optional }
+                     )
+      }
+    }
+    Then { assert_raises(&@code) }
+  end
+
+  test_that "assert_options with one option passes when stdout contains that option" do
+    Given { @stdout = some_options }
+    When { @code = ->() { assert_option(@stdout,"--help") } }
+    Then { refute_raises(&@code) }
+  end
+
+  test_that "assert_options with many option passes when stdout contains that option" do
+    Given { @stdout = some_options }
+    When { @code = ->() { assert_option(@stdout,"-h", "--help") } }
+    Then { refute_raises(&@code) }
+  end
+
+  test_that "assert_options fails when stdout does not the option" do
+    Given { @stdout = some_options }
+    When { @code = ->() { assert_option(@stdout,"--bleorg") } }
+    Then { assert_raises(&@code) }
+  end
+
+  test_that "assert_oneline_summary passes when the stdout has at least three lines, the second of which is blank and the third of which has some words in it" do
+    Given {
+      @stdout = [
+        "Usage: foobar",
+        "",
+        "The awesome app of awesome",
+      ].join("\n")
+    }
+    When { @code = ->() { assert_oneline_summary(@stdout) } }
+    Then { refute_raises(&@code) }
+  end
+
+  test_that "assert_oneline_summary fails when the stdout has at least three lines, the second of which is blank and the third of which has only one word in it" do
+    Given {
+      @stdout = [
+        "Usage: foobar",
+        "",
+        "awesome",
+      ].join("\n")
+    }
+    When { @code = ->() { assert_oneline_summary(@stdout) } }
+    Then { assert_raises(&@code) }
+  end
+
+  test_that "assert_oneline_summary fails when the stdout has at least three lines, the second of which is not blank and the third of which has words in it" do
+    Given {
+      @stdout = [
+        "Usage: foobar",
+        "foo",
+        "awesome app of awesome",
+      ].join("\n")
+    }
+    When { @code = ->() { assert_oneline_summary(@stdout) } }
+    Then { assert_raises(&@code) }
+  end
+
+  test_that "assert_oneline_summary fails when the stdout has less than three lines" do
+    Given {
+      @stdout = [
+        "Usage: foobar",
+        "awesome app of awesome",
+      ].join("\n")
+    }
+    When { @code = ->() { assert_oneline_summary(@stdout) } }
+    Then { assert_raises(&@code) }
+  end
+
+private
+
+  def some_options
+    [
+      "-h, --help     Get Help",
+      "--version      Show the version",
+      "--[no-]output  Print output",
+    ].join("\n")
+  end
+
+  def refute_raises(&block)
+    block.()
+  rescue Exception => ex
+    assert false, "Expected block NOT to raise, but got a #{ex.class}/#{ex.message}"
+  end
+
+  def some_file
+    File.open("some_file.txt","w") do |file|
+      file.puts "foo"
+      file.puts "bar"
+    end
+  end
+end

data/test/{test_cli_logger.rb → unit/test_cli_logger.rb}

@@ -37,8 +37,8 @@ class TestCLILogger < BaseTest
     When log_all_levels
 
     Then {
-      $stdout.string.should == "debug\ninfo\n"
-      $stderr.string.should == "warn\nerror\nfatal\n"
+      $stdout.string.should be == "debug\ninfo\n"
+      $stderr.string.should be == "warn\nerror\nfatal\n"
     }
   end
 
@@ -58,8 +58,8 @@ class TestCLILogger < BaseTest
     When log_all_levels
 
     Then {
-      $stdout.string.should == ""
-      $stderr.string.should == "error\nfatal\n"
+      $stdout.string.should be == ""
+      $stderr.string.should be == "error\nfatal\n"
     }
   end
 
@@ -69,7 +69,7 @@ class TestCLILogger < BaseTest
 
     Then stdout_should_have_everything
     And {
-      $stderr.string.should == "warn\nerror\nfatal\n"
+      $stderr.string.should be == "warn\nerror\nfatal\n"
     }
   end
 
@@ -80,7 +80,7 @@ class TestCLILogger < BaseTest
 
     Then stdout_should_have_everything
     And {
-      $stderr.string.should == "fatal\n"
+      $stderr.string.should be == "fatal\n"
     }
   end
 
@@ -97,8 +97,8 @@ class TestCLILogger < BaseTest
     When log_all_levels
 
     Then {
-      @out.string.should == "debug\ninfo\nwarn\nerror\nfatal\n"
-      @err.string.should == "warn\nerror\nfatal\n"
+      @out.string.should be == "debug\ninfo\nwarn\nerror\nfatal\n"
+      @err.string.should be == "warn\nerror\nfatal\n"
     }
   end
 
@@ -108,8 +108,8 @@ class TestCLILogger < BaseTest
     When log_all_levels
 
     Then {
-      $stdout.string.should == "fatal\n"
-      $stderr.string.should == "warn\nerror\nfatal\n"
+      $stdout.string.should be == "fatal\n"
+      $stderr.string.should be == "warn\nerror\nfatal\n"
     }
   end
 
@@ -126,8 +126,8 @@ class TestCLILogger < BaseTest
     }
 
     Then {
-      $stdout.string.should match /the time.*DEBUG.*debug/
-      $stderr.string.should match /the time.*ERROR.*error/
+      $stdout.string.should match(/the time.*DEBUG.*debug/)
+      $stderr.string.should match(/the time.*ERROR.*error/)
     }
   end
 
@@ -137,8 +137,8 @@ class TestCLILogger < BaseTest
       @logger << "foo"
     }
     Then {
-      $stdout.string.should == "foo"
-      $stderr.string.should == ""
+      $stdout.string.should be == "foo"
+      $stderr.string.should be == ""
     }
   end
 
@@ -154,8 +154,8 @@ class TestCLILogger < BaseTest
       @logger.error("error")
     }
     Then {
-      $stdout.string.should == "debug\nerror\n"
-      $stderr.string.should == "ERROR_LOGGER: error\n"
+      $stdout.string.should be == "debug\nerror\n"
+      $stderr.string.should be == "ERROR_LOGGER: error\n"
     }
   end
 
@@ -211,7 +211,7 @@ class TestCLILogger < BaseTest
 
   def stdout_should_have_everything
     proc do
-      $stdout.string.should == "debug\ninfo\nwarn\nerror\nfatal\n"
+      $stdout.string.should be == "debug\ninfo\nwarn\nerror\nfatal\n"
     end
   end
 

data/test/{test_cli_logging.rb → unit/test_cli_logging.rb}

@@ -32,8 +32,8 @@ class TestCLILogging < BaseTest
     }
 
     Then {
-      $stdout.string.should == "debug\ninfo\nwarn\nerror\nfatal\n"
-      $stderr.string.should == "warn\nerror\nfatal\n"
+      $stdout.string.should be == "debug\ninfo\nwarn\nerror\nfatal\n"
+      $stderr.string.should be == "warn\nerror\nfatal\n"
     }
   end
 
@@ -43,7 +43,7 @@ class TestCLILogging < BaseTest
       @second = MyOtherClassThatLogsToStdout.new
     }
     Then {
-      @first.logger_id.should == @second.logger_id
+      @first.logger_id.should be == @second.logger_id
     }
   end
 
@@ -59,8 +59,8 @@ class TestCLILogging < BaseTest
       end
     }
     Then {
-      @logger_id.should_not == @second.logger_id
-      @first.logger_id.should == @second.logger_id
+      @logger_id.should_not be == @second.logger_id
+      @first.logger_id.should be == @second.logger_id
     }
   end
 
@@ -76,8 +76,8 @@ class TestCLILogging < BaseTest
       end
     }
     Then {
-      @logger_id.should_not == @second.logger_id
-      @first.logger_id.should == @second.logger_id
+      @logger_id.should_not be == @second.logger_id
+      @first.logger_id.should be == @second.logger_id
     }
   end
 
@@ -104,7 +104,7 @@ class TestCLILogging < BaseTest
       @app.use_option(@level)
     }
     Then {
-      @app.logger.level.should == @level
+      @app.logger.level.should be == @level
     }
   end
 
@@ -120,7 +120,7 @@ class TestCLILogging < BaseTest
       @app.change_logger(@other_logger)
     }
     Then {
-      @other_logger.level.should == @level
+      @other_logger.level.should be == @level
     }
   end
 
@@ -136,7 +136,7 @@ class TestCLILogging < BaseTest
       send_signal_and_wait_a_moment('USR2')
     }
     Then {
-      @app.logger.level.should == Logger::DEBUG
+      @app.logger.level.should be == Logger::DEBUG
     }
   end
 
@@ -153,7 +153,7 @@ class TestCLILogging < BaseTest
       @app.change_logger(@other_logger)
     }
     Then {
-      @other_logger.level.should == Logger::DEBUG
+      @other_logger.level.should be == Logger::DEBUG
     }
   end
 
@@ -169,7 +169,7 @@ class TestCLILogging < BaseTest
       send_signal_and_wait_a_moment('USR2')
     }
     Then {
-      @app.logger.level.should == Logger::INFO
+      @app.logger.level.should be == Logger::INFO
     }
   end