Skip to content
This repository has been archived by the owner. It is now read-only.
Permalink
Browse files
8268602: a couple runtime/os tests don't check exit code
Reviewed-by: dholmes
  • Loading branch information
iignatev committed Jun 11, 2021
1 parent da043e9 commit cce8da2c60812873b63de4e65bc992bc6d825fe1
Showing 2 changed files with 4 additions and 2 deletions.
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2016, 2018, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2016, 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
@@ -87,6 +87,7 @@ public static void main(String[] args) throws Exception {
System.out.println("Final command line: " +
ProcessTools.getCommandLine(pb));
OutputAnalyzer output = ProcessTools.executeProcess(pb);
output.shouldHaveExitValue(0);
output.shouldContain(SUCCESS_STRING);
}
}
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2017, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2017, 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
@@ -47,6 +47,7 @@ public static void main(String[] args) throws Exception {
"-version");

OutputAnalyzer output = new OutputAnalyzer(pb.start());
output.shouldHaveExitValue(0);
output.shouldContain(SUCCESS_STRING);
}
}

1 comment on commit cce8da2

@openjdk-notifier
Copy link

@openjdk-notifier openjdk-notifier bot commented on cce8da2 Jun 11, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.