Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8246337

Add more JVM tests for sealed classes

    Details

    • Type: Enhancement
    • Status: Resolved
    • Priority: P4
    • Resolution: Fixed
    • Affects Version/s: 15
    • Fix Version/s: 16
    • Component/s: hotspot
    • Subcomponent:
    • Resolved In Build:
      b04

      Description

      More JVM tests are needed for sealed classes.

      From dholmes as part of sealed classes code review:

      On testing:

      test/hotspot/jtreg/runtime/modules/SealedModuleTest.java
      test/hotspot/jtreg/runtime/sealedClasses/SealedUnnamedModuleTest.java
      test/hotspot/jtreg/runtime/sealedClasses/SealedUnnamedModuleIntfTest.java

      You don't seem to have coverage of the full test matrix. For the combination of "same module or not" x "same package or not" x "public or not", there should be 8 test cases: 3 failures and 5 successes. Then you also have "listed in permits clause" versus "not listed in permits clause".

      Then you have all that for classes and interfaces.

        Attachments

          Activity

            People

            • Assignee:
              hseigel Harold Seigel
              Reporter:
              hseigel Harold Seigel
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: