[resteasy-dev] netty / undertow tests in RESTEasy testsuite
Ron Sigal
rsigal at redhat.com
Tue Oct 16 16:12:29 EDT 2018
Thanks, Rebecca. You're probably right. Ok, good.
On 10/16/2018 03:18 PM, Rebecca Searls wrote:
> I understand where you are coming from not wanting to create a new
> module just for 1 test
> but I also prefer not mixing such unrelated code in with existing
> modules just to avoid it. I think a
> new module would be good here. It makes it very clear these are
> different. It does not mix specifying
> jboss-servlet-api_3.1_spec with code that does not require it. And
> over time there may be the need to add more tests for these. I vote
> new module.
>
> On Tue, Oct 16, 2018 at 1:56 PM Ron Sigal <rsigal at redhat.com
> <mailto:rsigal at redhat.com>> wrote:
>
> For RESTEASY-2041 "Refactor scopes for Servlet API jar" I want to
> write
> tests that use netty and undertow, but I'm not sure where to put
> them.
> They clearly don't go in testsuite/integration-tests, and it's not
> clear
> that they go in testsuite/unit-tests, which doesn't reference
> resteasy-netty, resteasy-netty4, or resteasy-undertow. Seems a little
> crazy to create a new module just for one test.
>
> And they can't go in resteasy-netty, resteasy-netty4, or
> resteasy-undertow, because the point of the tests is to check that
> each
> of them "exports" jboss-servlet-api_3.1_spec to other modules.
>
> I'm leaning towards testsuite/unit-tests.
>
> Any thoughts?
>
> --
> My company's smarter than your company (unless you work for Red Hat)
>
> _______________________________________________
> resteasy-dev mailing list
> resteasy-dev at lists.jboss.org <mailto:resteasy-dev at lists.jboss.org>
> https://lists.jboss.org/mailman/listinfo/resteasy-dev
>
--
My company's smarter than your company (unless you work for Red Hat)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/resteasy-dev/attachments/20181016/4ef906f8/attachment.html
More information about the resteasy-dev
mailing list