Skip to content
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

Furoshiki tests make actual HTTP requests #4

Open
PragTob opened this issue Jun 24, 2013 · 4 comments
Open

Furoshiki tests make actual HTTP requests #4

PragTob opened this issue Jun 24, 2013 · 4 comments

Comments

@PragTob
Copy link
Member

PragTob commented Jun 24, 2013

The tests seem to make real HTTP requests, which is not always the best way to handle things. Maybe we should think about using something like the vcr/vcr gem.

@PragTob
Copy link
Member Author

PragTob commented Jun 24, 2013

This of course here is somewhat related to #5

@plexus
Copy link
Member

plexus commented Jun 24, 2013

On 24.06.2013 12:00, Tobias Pfeiffer wrote:

The tests seem to make real HTTP requests, which is not always the
best way to handle things. Maybe we should think about using something
like the vcr/vcr gem.


Reply to this email directly or view it on GitHub
#4.

What service is it calling? I've worked with VCR before, it's pretty
cool but if the interactions aren't too complex I prefer to write a
test/mock service.

@PragTob
Copy link
Member Author

PragTob commented Jun 24, 2013

Well in that case it is downloading a huge ass zip file... see #5 - maybe for tests we should take that from the local file system (I think it does so after the first test run) however that's not so cool for Travis.

On the other hand the real request is also cool, as it makes us notice if the URL is broken during test runs... so umm as always lots of viable options :-)

@wasnotrice
Copy link
Member

What would I do without you two to keep me honest? :)

I'd like to find some solution that does test the whole process end-to-end, but that maybe allows us to skip the actual download on Travis runs. As we add packaging options, the number of potential downloads will increase.

Maybe that end-to-end spec would be something we run locally before pushing to Travis, but not on Travis? We could also have a spec that checks for the existence of the necessary URLs without downloading files (HEAD requests?). This would still require the network, but not much, and it's a nice documentation of the URLs we need to be available.

I don't know enough about Travis to propose a solution for handling that large file. It seems like if we need to use a file, we will have to download it, unless Travis offers us a place to host it.

@jasonrclark jasonrclark changed the title Furoshiki tests make actuall HTTP requests Furoshiki tests make actual HTTP requests Dec 30, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants