search

unclebob / fitnesse

FitNesse -- The Acceptance Test Wiki
fitnesse.org
Other
2.02k stars 712 forks source link

Official JAVA 17 (LTS) support #1338

Closed HoneyBadgerSeeker closed 2 weeks ago

HoneyBadgerSeeker commented 2 years ago

Is your feature request related to a problem? Please describe. No, not for now, it seems working fine with Java 17.

Describe the solution you'd like The new LTS version of Java is available (Java 17). FitNesse should be officially compatible with it.

The FitNesse download page, state that

Ensure you have a supported Java version installed (either version 8 or 11, versions >= 15 are not supported).

enexusde commented 2 years ago

Using RestFixtures a

Could not complete testing: java.lang.UnsupportedOperationException: The Security Manager is deprecated and will be removed in a future release

Exception is thrown.

fhoeben commented 2 years ago

@enexusde is the exception thrown from RestFixture code (if so you should probably file the issue at https://github.com/smartrics/RestFixture), or from FitNesse itself?

If the error comes from FitNesse, can you provide a stack trace where the exception is thrown and preferably a reproduction path using the only code present in this project?

tcnh commented 2 years ago

I've seen that same exception, but only with a Java 18 JRE. That's where it was removed from the runtime.

enexusde commented 2 years ago

@fhoeben From FitNesse

lehtmets commented 1 year ago

I saw this waring with Java 17 (17.0.6 temurin) WARNING: A terminally deprecated method in java.lang.System has been called WARNING: System::setSecurityManager has been called by fitnesse.slim.instructions.SystemExitSecurityManager (file:/Users/ml/.gradle/caches/modules-2/files-2.1/org.fitnesse/fitnesse/20221102/b5ce9b528912742591c26c6893e8e091aeed543a/fitnesse-20221102.jar) WARNING: Please consider reporting this to the maintainers of fitnesse.slim.instructions.SystemExitSecurityManager WARNING: System::setSecurityManager will be removed in a future release

unexist commented 3 weeks ago

I stumbled upon this due to the exact error messages. This is fixed in the recent version, as can be seen in the Changelog:

https://github.com/unclebob/fitnesse/releases/tag/20240707

And the matching PR:

https://github.com/unclebob/fitnesse/pull/1513