> This is a simple change to throw jtreg.SkippedException if no JavaScript > engine is available to run the TestSearchScript test. In my previous attempt > I tried to use the SkippedException class already present elsewhere in the > test tree, not realizing I couuld just copy/create the exception in the > javadoc test library. > > Note that while the test is skipped correctly if no JS engine is available, > the test will currently fail in the presence of a JS engine such as GraalJS. > This is fixed by JDK-8244535.
Hannes Wallnöfer has updated the pull request incrementally with one additional commit since the last revision: Use existing SkippedException class in TestSearchScript.java ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/1412/files - new: https://git.openjdk.java.net/jdk/pull/1412/files/d6bc3b79..02ab014b Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1412&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1412&range=00-01 Stats: 39 lines in 3 files changed: 5 ins; 34 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/1412.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1412/head:pull/1412 PR: https://git.openjdk.java.net/jdk/pull/1412