On Wed, 30 Nov 2022 20:44:30 GMT, Naoto Sato <na...@openjdk.org> wrote:

>> This is to allow Console to be used even when it is not attached to the 
>> platform provided terminal, such as the case when the standard input is 
>> redirected. `System.console()` now returns a Console implementation based on 
>> `jdk.internal.le` terminal by default, or jshell implementation if 
>> available. A corresponding CSR has been drafted.
>
> Naoto Sato has updated the pull request incrementally with two additional 
> commits since the last revision:
> 
>  - Adds a test
>  - Removed JavaIOAccess.charset() which is no longer needed

A general point is that it's difficult to test Console and it looks like we 
don't have many existing tests in test/jdk/java/io/Console. The test that is 
there doesn't seem to exercise most of the methods. I wonder should improve 
this and have the tests run with `--limit-modules java.base` so they run with 
the default the is in java.base rather than the provider in jdk.intenal.le.

-------------

PR: https://git.openjdk.org/jdk/pull/11421

Reply via email to