Add an HTTP test client #131

lvarvel · 2014-05-20T22:23:12Z

This commit adds RspecApiDocumentation::HttpTestClient, which can be used to test any website, either on your local development box or out on the internet somewhere.

To use, just add the following line to the top of your spec:

let(:client) { RspecApiDocumentation::HttpTestClient.new(self, {host: 'https://yourtestsite.com/'}) }

This commit adds RspecApiDocumentation::HttpTestClient, which can be used to test *any* website, either on your local development box or out on the internet somewhere. To use, just add the following line to the top of your spec: ``` let(:client) { RspecApiDocumentation::HttpTestClient.new(self, {host: 'https://base.url.of.the.site.you.want.to.test.com/'}) } ```

lvarvel · 2014-05-20T22:29:00Z

Full disclosure: There's a few outstanding nits I have with the tests (the client is fine).

The output from the sinatra test app makes the tests noisy.
The way the sinatra test app is being launched may not work on Windows
There's a "sleep 3" to let the sinatra test app come up before running the specs

snuggs · 2014-05-21T19:43:07Z

Why the need for referencing the current Rspec ExampleGroup?

self

?

oestrich · 2014-05-21T19:57:41Z

rspec_api_documentation.gemspec

@@ -19,6 +19,7 @@ Gem::Specification.new do |s|
  s.add_runtime_dependency "i18n", ">= 0.1.0"
  s.add_runtime_dependency "mustache", ">= 0.99.4"
  s.add_runtime_dependency "json", ">= 1.4.6"
+  s.add_runtime_dependency "faraday", ">= 0.9.0"


We should include this similar to how webmock is required.

rspec_api_documentation/lib/rspec_api_documentation/oauth2_mac_client.rb

Line 7 in d48c626

require "webmock"

oestrich · 2014-10-08T16:54:07Z

Any updates on this?

oestrich · 2014-10-09T15:10:17Z

I merged this locally and then did some updates. Fixing my previous issue and then making it use the same StubApp as the RackTestClient.

oestrich · 2014-10-09T15:26:08Z

It also uses Capybara to launch the server so we're not doing a separate process.

oestrich reviewed May 21, 2014
View reviewed changes

oestrich merged commit c5f83ee into zipmark:master Oct 9, 2014

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Add an HTTP test client #131

Add an HTTP test client #131

Uh oh!

lvarvel commented May 20, 2014

Uh oh!

lvarvel commented May 20, 2014

Uh oh!

snuggs commented May 21, 2014

Uh oh!

oestrich May 21, 2014

Uh oh!

oestrich commented Oct 8, 2014

Uh oh!

oestrich commented Oct 9, 2014

Uh oh!

oestrich commented Oct 9, 2014

Uh oh!

Uh oh!

Add an HTTP test client #131

Add an HTTP test client #131

Uh oh!

Conversation

lvarvel commented May 20, 2014

Uh oh!

lvarvel commented May 20, 2014

Uh oh!

snuggs commented May 21, 2014

Uh oh!

oestrich May 21, 2014

Choose a reason for hiding this comment

Uh oh!

oestrich commented Oct 8, 2014

Uh oh!

oestrich commented Oct 9, 2014

Uh oh!

oestrich commented Oct 9, 2014

Uh oh!

Uh oh!