Skip to content

Commit

Permalink
Merge pull request #11874 from jetty/jetty-12.0.x-11873-ServerResources
Browse files Browse the repository at this point in the history
Issue #11873 - Server resources are not found if the server is subclassed in a different package
  • Loading branch information
lachlan-roberts authored Jun 4, 2024
2 parents ba8732e + 21f8e58 commit dc28714
Show file tree
Hide file tree
Showing 2 changed files with 66 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -793,7 +793,7 @@ public Server getServer()
*/
public Resource getDefaultStyleSheet()
{
return newResource("jetty-dir.css");
return newResource("/org/eclipse/jetty/server/jetty-dir.css");
}

/**
Expand All @@ -803,7 +803,7 @@ public Resource getDefaultStyleSheet()
*/
public Resource getDefaultFavicon()
{
return newResource("favicon.ico");
return newResource("/org/eclipse/jetty/server/favicon.ico");
}

/**
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,64 @@
//
// ========================================================================
// Copyright (c) 1995 Mort Bay Consulting Pty Ltd and others.
//
// This program and the accompanying materials are made available under the
// terms of the Eclipse Public License v. 2.0 which is available at
// https://www.eclipse.org/legal/epl-2.0, or the Apache License, Version 2.0
// which is available at https://www.apache.org/licenses/LICENSE-2.0.
//
// SPDX-License-Identifier: EPL-2.0 OR Apache-2.0
// ========================================================================
//

package org.eclipse.jetty.server.subpackage;

import java.util.stream.Stream;

import org.eclipse.jetty.server.Server;
import org.junit.jupiter.params.ParameterizedTest;
import org.junit.jupiter.params.provider.Arguments;
import org.junit.jupiter.params.provider.MethodSource;

import static org.junit.jupiter.api.Assertions.assertNotNull;

public class ServerDefaultResourcesTest
{
public static Stream<Arguments> arguments()
{
return Stream.of(
new Server(),
new Server(){}
).map(Arguments::of);
}

@ParameterizedTest
@MethodSource("arguments")
public void testDefaultStyleSheet(Server server) throws Exception
{
try
{
server.start();
assertNotNull(server.getDefaultStyleSheet());
}
finally
{
server.stop();
}
}

@ParameterizedTest
@MethodSource("arguments")
public void testDefaultFavicon(Server server) throws Exception
{
try
{
server.start();
assertNotNull(server.getDefaultFavicon());
}
finally
{
server.stop();
}
}
}

0 comments on commit dc28714

Please sign in to comment.