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

Add more JVM tests for sealed classes

    XMLWordPrintable

Details

    • 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

            hseigel Harold Seigel (Inactive)
            hseigel Harold Seigel (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: