[
https://issues.apache.org/jira/browse/SHINDIG-364?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12603334#action_12603334
]
Kevin Brown commented on SHINDIG-364:
-------------------------------------
Hmm -- maybe the problem *is* the interface? Instead of HttpFetcher.fetch,
perhaps we should have HttpFetcher.get, HttpFetcher.post, HttpFetcher.head,
etc.?
> provide reusable tests for injected dependencies
> ------------------------------------------------
>
> Key: SHINDIG-364
> URL: https://issues.apache.org/jira/browse/SHINDIG-364
> Project: Shindig
> Issue Type: Bug
> Components: Gadget Rendering Server (Java)
> Reporter: Brian Eaton
> Attachments: testing-package.patch
>
>
> Problem: dependencies injected in to Shindig are expected to fulfill
> contracts that are not completely specified by their interfaces. For
> example, HttpFetcher implementations need to support GET, PUT, POST, and
> DELETE methods, and also need to return the body of HTTP error responses.
> It's easy to write test cases for this for any particular HttpFetcher
> implementation (we have them for BasicHttpFetcher), but it is slightly
> trickier to reuse those test cases against other HttpFetcher implementations.
> Quite a bit of this trickiness is introduced by maven, unfortunately.
> I'm attaching a patch that takes a first swing at artifact creation and
> renaming to allow the BasicHttpFetcherTest test cases to be reused for
> arbitrary HTTP fetchers.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
