RFR: JDK-8296406: ChainingConstructors jtreg test reduce code duplication
    Sean Mullan 
    mullan at openjdk.org
       
    Tue Nov  8 14:05:30 UTC 2022
    
    
  
On Mon, 7 Nov 2022 12:49:21 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:
> The java/security/Exceptions/ChainingConstructors jdk jtreg test shows quite a lot of code duplication. The test code can be simplified e.g. by using generics. Additionally the test failure output was improved and a test for the constructors of CertPathBuilderException was added.
Changes requested by mullan (Reviewer).
test/jdk/java/security/Exceptions/ChainingConstructors.java line 40:
> 38:     private static Exception cause = new Exception("cause");
> 39: 
> 40:     public static void main(String[] args) throws Exception {
I would add the bugid to line 26 since you are adding an additional test for CertPathBuilderException.
test/jdk/java/security/Exceptions/ChainingConstructors.java line 208:
> 206:             throw new SecurityException("Test 17 failed");
> 207:         }
> 208: */
I don't know why this is commented out, but when I uncomment it, the test still passes, so I would include a test for `SSLException` below.
-------------
PR: https://git.openjdk.org/jdk/pull/11016
    
    
More information about the security-dev
mailing list