Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update RSpec #108

Merged
merged 7 commits into from Feb 18, 2015
Merged

Update RSpec #108

merged 7 commits into from Feb 18, 2015

Conversation

mikeastock
Copy link
Collaborator

This PR upgrades RSpec from 2 to 3.

This conversion is done by Transpec 3.0.7 with the following command:
    transpec --force --convert stub_with_hash --convert example_group --explicit-spec-type

* 169 conversions
    from: obj.should
      to: expect(obj).to

* 55 conversions
    from: == expected
      to: eq(expected)

* 44 conversions
    from: obj.should_not
      to: expect(obj).not_to

* 24 conversions
    from: describe 'something' { }
      to: RSpec.describe 'something' { }

* 14 conversions
    from: describe 'some controller' { }
      to: describe 'some controller', :type => :controller { }

* 9 conversions
    from: describe 'some model' { }
      to: describe 'some model', :type => :model { }

* 8 conversions
    from: obj.stub(:message)
      to: allow(obj).to receive(:message)

* 4 conversions
    from: be_true
      to: be_truthy

* 4 conversions
    from: expect(obj).to have_at_least(n).errors_on(...)
      to: expect(obj.errors_on(...).size).to be >= n

* 2 conversions
    from: be_false
      to: be_falsey

* 2 conversions
    from: lambda { }.should
      to: expect { }.to

* 2 additions
      of: RSpec.configure { |c| c.expose_dsl_globally = false }

* 1 conversion
    from: obj.should_receive(:message)
      to: expect(obj).to receive(:message)

For more details: https://github.com/yujinakayama/transpec#supported-conversions
@mikeastock
Copy link
Collaborator Author

@sudara I went ahead and removed the mp3-info gem bump.

@wioux let me know if this is ready to go or if something is missing

wioux added a commit that referenced this pull request Feb 18, 2015
@wioux wioux merged commit 512bb4b into master Feb 18, 2015
@wioux
Copy link
Collaborator

wioux commented Feb 18, 2015

Looks good to me. Thanks!

@mikeastock mikeastock deleted the update_rspec branch February 26, 2017 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants