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
Format misc files #318
Format misc files #318
Conversation
test/stripe/error_test.exs
Outdated
@@ -10,7 +10,10 @@ defmodule Stripe.ErrorTest do | |||
error = Error.from_stripe_error(400, nil, "id") | |||
assert error.code == :bad_request | |||
assert error.extra.http_status == 400 | |||
assert error.message == "The request was unacceptable, often due to missing a required parameter." | |||
|
|||
assert error.message == |
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.
Maybe name this something like expected_message
, so you don't break in middle.
test/support/stripe_mock.ex
Outdated
def handle_info({:DOWN, os_pid, :process, _ex_pid, _reason}, %{os_pid: os_pid, opts: opts, restarting: {true, from}} = state) do | ||
def handle_info( | ||
{:DOWN, os_pid, :process, _ex_pid, _reason}, | ||
%{os_pid: os_pid, opts: opts, restarting: {true, from}} = state |
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.
They recommend doing this match as the first line of the function since it's immaterial to the function being invoked.
test/support/stripe_mock.ex
Outdated
executable = opts[:stripe_mock_path] || System.find_executable("stripe-mock") | ||
|
||
unless executable, | ||
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 would turn this into a proper block.
Changes Unknown when pulling 9da36b2 on format-misc into ** on 2.0-beta**. |
1 similar comment
Changes Unknown when pulling 9da36b2 on format-misc into ** on 2.0-beta**. |
9da36b2
to
46de987
Compare
No description provided.