stripe / stripe-ruby / 3388

96%

master: 97%

Pull #833

travis-ci

Do better bookkeeping when tracking state in `Thread.current`

This is largely just another cleanup patch, but does a couple main
things:

* Hoists the `last_response` value into thread state. This is a very
  minor nicety, but effectively makes `StripeClient` fully thread-safe,
  which seems like a minor nicety. Two calls to `#request` to the same
  `StripeObject` can now be executed on two different threads and their
  results won't interfere with each other.

* Moves state off one-off `Thread.current` keys and into a single one
  for the whole client which stores a new simple type of record called
  `ThreadContext`. Again, this doesn't change much, but adds some minor
  type safety and lets us document each field we expect to have in a
  thread's context.

17 of 17 new or added lines in 1 file covered. (100.0%)

1847 of 1923 relevant lines covered (96.05%)

2441.4 hits per line