Re: [cu-dev] leading slash in @WebServlet in TCK tests

[Petr Aubrecht <aubrecht@xxxxxxxxxxxx>]

https://github.com/eclipse-ee4j/concurrency-api/commit/11cb74c

    package rename and final changes

update copyright

rename packages again

Use central ContextServiceDefinitionServlet

Some other random example:

https://github.com/eclipse-ee4j/concurrency-api/commit/b75689b

Enable arquillian on transaction tests

Update spec tests to use Arquillian

Update api tests to use Arquillian

Petr

On 3/22/22 13:12, arjan tijms wrote:

Hi,

Just wondering, what did the commit comment said that removed that leading slash?

On Tuesday, March 22, 2022, Petr Aubrecht <aubrecht@xxxxxxxxxxxx> wrote:

Hi,

I'm looking at the TCK tests for Concurrent and one of the interesting things is removing leading slash in the @WebServlet annotation. Even the existing ones were removed.

E.g.

@WebServlet("/ManagedThreadFactoryDefinitionServlet")

was changed to

@WebServlet("ManagedThreadFactoryDefinitionServlet")

I looked at the spec, but it tells nothing. All examples contain the leading slash.

https://github.com/jakartaee/servlet/blob/master/spec/src/main/asciidoc/servlet-spec-body.adoc#811-webservlet

Can you point me to some place, which explains, why it should work without it? I know, that it is not required for @Path, but it is a different spec.

Thank you

Petr

_______________________________________________
cu-dev mailing list
cu-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/cu-dev