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

Tests fail under RSpec 3.2 due to Airborne#initialize needing to accept arguments #37

Merged
merged 1 commit into from
Feb 26, 2015

Conversation

jgwmaxwell
Copy link
Contributor

Since the gem wasn't using the constructor arguments before, I've just altered StubHelper to accept splat arguments and then ignored them. This turns the test suite green, however long-term people might want to change this behaviour to something more contextual.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.71% when pulling 1f66c42 on boardiq:bugfix/initialize-arguments into 0493e4c on brooklynDev:master.

@brooklynDev
Copy link
Owner

Interesting, I just upgraded RSpec on my local machine and sure enough the current version of airbornes test all fail. Did RSpec change the way modules are created now when they're included? I tried looking around online but couldn't find anything. In any event, I'll pull merge this in a few, thanks!

brooklynDev added a commit that referenced this pull request Feb 26, 2015
Tests fail under RSpec 3.2 due to Airborne#initialize needing to accept arguments
@brooklynDev brooklynDev merged commit 40a8106 into brooklynDev:master Feb 26, 2015
@jgwmaxwell jgwmaxwell deleted the bugfix/initialize-arguments branch March 3, 2015 12:00
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.

3 participants