summaryrefslogtreecommitdiff
path: root/Rakefile
diff options
context:
space:
mode:
authorJosh Cheek <josh.cheek@gmail.com>2014-08-10 16:26:47 -0600
committerJosh Cheek <josh.cheek@gmail.com>2014-08-10 17:37:21 -0600
commit144d32e1d6ba772d900dcb9dd0b1672f761fa480 (patch)
treee05c0ee4c2b6ddd9578d80e2faa8b62fb7aa8317 /Rakefile
parentc6f8d5c632d5609402da6bc49c289798ce872238 (diff)
downloadpry-144d32e1d6ba772d900dcb9dd0b1672f761fa480.tar.gz
Switch test suite to RSpec
Removes Bacon and Mocha Reasoning explained in this comment: https://github.com/pry/pry/issues/277#issuecomment-51708712 Mostly this went smoothly. There were a few errors that I fixed along the way, e.g. tests that were failing but for various reasons still passed. Should have documented them, but didn't think about it until very near the end. But generaly, I remember 2 reasons this would happen: `lambda { raise "omg" }.should.raise(RuntimeError, /not-omg/)` will pass because the second argument is ignored by Bacon. And `1.should == 2` will return false instead of raising an error when it is not in an it block (e.g. if stuck in a describe block, that would just return false) The only one that I felt unsure about was spec/helpers/table_spec.rb `Pry::Helpers.tablify_or_one_line('head', %w(ing)).should == 'head: ing'` This is wrong, but was not failing because it was in a describe block instead of an it block. In reality, it returns `"head: ing\n"`, I updated the test to reflect this, though I don't know for sure this is the right thing to do This will fail on master until https://github.com/pry/pry/pull/1281 is merged. This makes https://github.com/pry/pry/pull/1278 unnecessary.
Diffstat (limited to 'Rakefile')
-rw-r--r--Rakefile4
1 files changed, 2 insertions, 2 deletions
diff --git a/Rakefile b/Rakefile
index 96f37f40..dc5c96d5 100644
--- a/Rakefile
+++ b/Rakefile
@@ -11,8 +11,8 @@ desc "Set up and run tests"
task :default => [:test]
def run_specs paths
- quiet = ENV['VERBOSE'] ? '' : '-q'
- exec "bacon -Ispec -rubygems #{quiet} #{paths.join ' '}"
+ format = ENV['VERBOSE'] ? '--format documentation ' : ''
+ sh "rspec #{format}#{paths.join ' '}"
end
desc "Run tests"