On Tue, 18 Jun 2024 06:31:37 GMT, SendaoYan <s...@openjdk.org> wrote:

>> Hi all,
>> Testcase 
>> `test/jdk/java/util/ResourceBundle/Control/MissingResourceCauseTestRun.java` 
>> run fails with root user privileged. I think it's necessary to skip this 
>> testcase when user is root.
>> Why run the jtreg test by root user? It's because during rpmbuild process 
>> for linux distribution of JDK, root user is the default user to build the 
>> openjdk, also is the default user to run the `make test-tier1`, this PR make 
>> this testcase more robustness.
>> The change has been verified, only change the testcase, no risk.
>
> SendaoYan has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   add a whitespace before if

test/jdk/java/util/ResourceBundle/Control/MissingResourceCauseTestRun.java line 
54:

> 52: import jdk.test.lib.Utils;
> 53: import jdk.test.lib.process.ProcessTools;
> 54: import jdk.test.lib.Platform;

It would be beneficial to add this issue's bug ID to the Jtreg `@bug` tag.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19732#discussion_r1643972916

Reply via email to