-
Notifications
You must be signed in to change notification settings - Fork 20
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
Implement All Launches #28
Implement All Launches #28
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That was great! Can you please just fix the conflict?
Thank you very much @ludamillion
spec/spacex/launches_spec.rb
Outdated
@@ -2,8 +2,11 @@ | |||
|
|||
require 'spec_helper' | |||
|
|||
RSpec.shared_examples 'Merah Putih' do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this wasn't suppose to be here. 😃
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hah, yes that was a left over. Fixing that and the Rubocop issues now.
- Add `all` method to `SPACEX::Launches` to query all launches endpoint - Update `SPACEX::BaseRequest` to handle a response body which is an array
12fe3f4
to
1c243bc
Compare
Pull Request Test Coverage Report for Build 41
💛 - Coveralls |
For some reason VCR was reporting that there weren't interactions recorded for certain API calls (ones that I hadn't touched). Temporarily turning the record mode to `:new_episodes` and re-recording seemed to have addressed the issue. This solution did however require updating the tests with re-recorded API calls.
all
method toSPACEX::Launches
to query all launches endpointSPACEX::BaseRequest
to handle a response body which is an array