jetty / jetty.project

Eclipse Jetty® - Web Container & Clients - supports HTTP/2, HTTP/1.1, HTTP/1.0, websocket, servlets, and more
https://eclipse.dev/jetty
Other
3.87k stars 1.91k forks source link

Review Disabled tests in `core` #12396

Open janbartel opened 1 month ago

janbartel commented 1 month ago

Jetty version(s) 12.1.x

Jetty Environment core

Review the following test classes that have at least one @Disabled test. Acceptable resolutions are:

janbartel commented 1 month ago

ShutdownMonitorTest see PR #12398

olamy commented 1 month ago

conscrypt test are disabled for aarch64 only because of https://github.com/google/conscrypt/issues/1051

olamy commented 1 month ago

ResourcceFactoryTest https://github.com/jetty/jetty.project/pull/12402

janbartel commented 1 month ago

ContextHandlerGetResourceTest and AllowSymLinkAliasCheckerTest #12412

janbartel commented 1 month ago

MountedPathResourceTest reviewed, and relevant JDK bug is still unresolved (see https://bugs.openjdk.org/browse/JDK-8311079) so keeping @Disabled.

olamy commented 1 month ago

LowResourcesMonitorTest PR #12417

janbartel commented 1 month ago

ResourceHandlerTest see #12418

janbartel commented 1 month ago

ResourceHandlerByteRangesTest see #12419

joakime commented 1 month ago

DebugHandlerTest see #12424

janbartel commented 3 weeks ago

ContextHandlerTest PR #12450

janbartel commented 3 weeks ago

NotAcceptingTest see PR #12451

janbartel commented 3 weeks ago

jetty-12.1.x-undisable-SecuredRedirectHandlerCodeTest see PR #12471

janbartel commented 3 weeks ago

HttpConnectionTest see PR #12472

janbartel commented 3 weeks ago

HttpChannelEventTest see PR #12477

janbartel commented 2 weeks ago

PartialRFC2616Test see #12486

janbartel commented 2 weeks ago

ExternalFastCGITest see #12487

janbartel commented 2 weeks ago

HttpClientTLSTest see #12491