Skip to content
This repository was archived by the owner on Jan 17, 2023. It is now read-only.

Enable custom httpbin URL with HTTPBIN_BASE_URL environment variable #3748

Merged
merged 1 commit into from
Oct 12, 2016

Conversation

0xced
Copy link
Contributor

@0xced 0xced commented Oct 11, 2016

Editing the scheme and checking the HTTPBIN_BASE_URL box enables running the test suites on a default local installation of httpbin.

@0xced 0xced added this to the 3.2.0 milestone Oct 11, 2016
@0xced 0xced added the added label Oct 11, 2016
@kcharwood
Copy link
Contributor

Would it make sense to install that during the CI script to speed up the test suite? I'm assuming the test suite runs faster?

@0xced
Copy link
Contributor Author

0xced commented Oct 11, 2016

No, because it's running on http instead of https and thus breaking some tests. I tried to run it on https some time ago but failed. I’m not sure it would even be possible to get a valid certificate for 127.0.0.1 without a domain name.

@kcharwood
Copy link
Contributor

Ah good point. Didn't think about that.

Editing the scheme and checking the `HTTPBIN_BASE_URL` box enables running the test suites on a default local installation of httpbin.
@0xced 0xced added the tests label Oct 11, 2016
@codecov-io
Copy link

codecov-io commented Oct 11, 2016

Current coverage is 87.48% (diff: 100%)

Merging #3748 into master will increase coverage by <.01%

@@             master      #3748   diff @@
==========================================
  Files            45         45          
  Lines          6234       6235     +1   
  Methods        1101       1101          
  Messages          0          0          
  Branches        415        415          
==========================================
+ Hits           5454       5455     +1   
  Misses          777        777          
  Partials          3          3          

Powered by Codecov. Last update 0e947a7...5d6867c

@0xced 0xced merged commit 9627d0f into master Oct 12, 2016
@0xced 0xced deleted the httpbin-base-url branch October 12, 2016 04:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants