1
0
Fork 0
mirror of https://github.com/pry/pry.git synced 2022-11-09 12:35:05 -05:00
Commit graph

12 commits

Author SHA1 Message Date
Kyrylo Silin
96c6958cc6 specs: convert to shoulds to expects 2015-03-10 22:49:29 +02:00
Josh Cheek
144d32e1d6 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.
2014-08-10 17:37:21 -06:00
Conrad Irwin
f27e718cfe Move back to 'gist' gem 2014-04-27 22:24:20 -07:00
Robert Gleeson
74135c1890 allow for tests to be run with ease.
'bacon spec/config_spec.rb' instead of 'bacon -I spec spec/config_spec.rb'
2014-03-14 05:31:24 +01:00
John Mair
c10017f17d gist: re-add 'jist' gem dependency with :requires_gem
Oops, forgetting this meant that 'gist' command would error out with
ugly messages rather than prompting the user to 'install-command'
2013-01-17 16:52:55 +01:00
John Mair
b6b78157a5 gist: Use Pry::CodeCollector (like play/save-file)
This enables a simplified UI, i.e: gist my_file.rb instead of gist -f my_file.rb
and `gist my_method` instead of `gist -m my_method`. Unfortunately, we had to kill our
tests in the process of doing this (gist_spec.rb is just commented out) - however since play
and save-file have tests and they use the same Pry::CodeCollector object 'gist' is indirectly
tested. Nonetheless, we should re-add tests at some point
2013-01-15 20:08:32 +01:00
Kyrylo Silin
5871054c05 gist_spec: remove some repetitive code 2013-01-14 17:42:20 +02:00
☈king
9b38f33b79 Progress on gist + jruby crash 2012-12-09 23:21:18 -06:00
☈king
2104655a67 Replace gist spec globals with Pad 2012-12-09 19:19:38 -07:00
☈king
8f92f5a855 Use Jist.copy, not a Helper; fix @filename 2012-12-07 08:35:25 -06:00
☈king
c51f75a9fc DRY gist command's docs vs tests 2012-12-06 23:14:04 -06:00
☈king
27bbc4383a gist: fix filenames from REPL + refactor + test
I glommed together too much on this commit. ☹
2012-12-06 22:46:01 -06:00