Skip to content

Commit

Permalink
Comment reviews
Browse files Browse the repository at this point in the history
Signed-off-by: Jorge Bescos Gascon <[email protected]>
  • Loading branch information
jbescos committed Aug 10, 2022
1 parent 5bfa401 commit 00240b0
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 2 deletions.
24 changes: 24 additions & 0 deletions tests/integration/property-check/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -109,4 +109,28 @@
</plugin>
</plugins>
</build>

<profiles>
<profile>
<id>jdk19+</id>
<activation>
<jdk>[19,)</jdk>
</activation>
<build>
<pluginManagement>
<plugins>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
<configuration>
<excludes>
<exclude>**/SystemPropertiesConfigurationModelTest.java</exclude>
</excludes>
</configuration>
</plugin>
</plugins>
</pluginManagement>
</build>
</profile>
</profiles>
</project>
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@
import org.glassfish.jersey.server.oauth1.OAuth1ServerProperties;
import org.glassfish.jersey.servlet.ServletProperties;
import org.glassfish.jersey.test.TestProperties;
import org.junit.Ignore;
import org.junit.Test;

public class SystemPropertiesConfigurationModelTest {
Expand Down Expand Up @@ -78,7 +77,6 @@ public void allPropertyClassLoaded() throws IOException {
}

@Test
@Ignore("System.setSecurityManager is deprecated in JDK 19 and there is no replacement")
public void propertyLoadedWhenSecurityException() {
final String TEST_STRING = "test";
SecurityManager sm = System.getSecurityManager();
Expand Down

0 comments on commit 00240b0

Please sign in to comment.