Skip to content
This repository has been archived by the owner. It is now read-only.
Permalink
Browse files
8271223: two runtime/ClassFile tests don't check exit code
Reviewed-by: dholmes
  • Loading branch information
iignatev committed Jul 27, 2021
1 parent b76a838 commit cea7bc2dea8e179425db19998665f85c0b124d71
Showing with 4 additions and 2 deletions.
  1. +2 −1 test/hotspot/jtreg/runtime/ClassFile/JsrRewriting.java
  2. +2 −1 test/hotspot/jtreg/runtime/ClassFile/OomWhileParsingRepeatedJsr.java
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2011, 2020, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2011, 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -74,6 +74,7 @@ public static void main(String[] args) throws Exception {
className);

output = new OutputAnalyzer(pb.start());
output.shouldNotHaveExitValue(0);
String[] expectedMsgs = {
"java.lang.LinkageError",
"java.lang.NoSuchMethodError",
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2011, 2020, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2011, 2021, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -73,6 +73,7 @@ public static void main(String[] args) throws Exception {
className );

output = new OutputAnalyzer(pb.start());
output.shouldNotHaveExitValue(0);
output.shouldContain("Cannot reserve enough memory");
}
}

1 comment on commit cea7bc2

@openjdk-notifier

This comment has been minimized.

Copy link

@openjdk-notifier openjdk-notifier bot commented on cea7bc2 Jul 27, 2021

Please sign in to comment.