3436
97%

Ran 04 Sep 2019 09:51PM UTC

Jobs 5

Files 97

Run time 3min

Badge

Embed ▾

pending completion

Build # 3436

Build Type

push

travis-ci

Committed by

brandur-stripe

Commit Message

Change some error tests to use `assert_raises` (#847)

Previously, we had quite a few error tests that were written like this:

``` ruby
begin
  client.execute_request(:post, "/v1/charges")
rescue Stripe::InvalidRequestError => e
  assert_equal(404, e.http_status)
  assert_equal(true, e.json_body.is_a?(Hash))
end
```

The trouble with that pattern is that although they'll _usually_ work,
the test will incorrectly pass if no error at all is thrown because the
`rescue` never activates and therefore the assertions never run.

We change them to use `assert_raises` like so:

``` ruby
e = assert_raises Stripe::InvalidRequestError do
  client.execute_request(:post, "/v1/charges")
end
assert_equal(404, e.http_status)
assert_equal(true, e.json_body.is_a?(Hash))
```

The weird part is that many of the tests were already using
`assert_raises`, so here we're just converting them all over to use the
same convention.

I've also made a few whitespace tweaks. None of them are significant,
but they were an attempt to standardize a little on the whitespace
layout of many of these tests which were similar.

Run Details

1855 of 1932 relevant lines covered (96.01%)

2437.2 hits per line

Jobs

ID	Job ID	Ran	Coverage
1	3436.1 (2.3)	04 Sep 2019 09:51PM UTC	96.12	Travis Job 3436.1
2	3436.2 (2.4)	04 Sep 2019 09:51PM UTC	96.12	Travis Job 3436.2
3	3436.3 (2.5)	04 Sep 2019 09:51PM UTC	96.12	Travis Job 3436.3
4	3436.4 (2.6)	04 Sep 2019 09:51PM UTC	96.09	Travis Job 3436.4
5	3436.5 (jruby-9.2.7.0)	04 Sep 2019 09:54PM UTC	0.0	Travis Job 3436.5

Source Files on build 3436

Detailed source file information is not available for this build.

stripe / stripe-ruby / 3436
97%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 3436

stripe / stripe-ruby / 3436 97%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 3436

stripe / stripe-ruby / 3436
97%

README BADGES
x