Skip to content

Commit

Permalink
8323994: gtest runner repeats test name for every single gtest assertion
Browse files Browse the repository at this point in the history
Backport-of: 1aae980c549741cf5fc5ca51f3c299285bafa49d
  • Loading branch information
Andrew Lu committed Mar 28, 2024
1 parent f52725d commit b293856
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions test/hotspot/jtreg/gtest/GTestResultParser.java
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2019, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2019, 2024, 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
Expand Down Expand Up @@ -57,7 +57,10 @@ public GTestResultParser(Path file) {
testCase = xmlReader.getAttributeValue("", "name");
break;
case "failure":
failedTests.add(testSuite + "::" + testCase);
String failedStr = testSuite + "::" + testCase;
if (!failedTests.contains(failedStr)) {
failedTests.add(failedStr);
}
break;
default:
// ignore
Expand Down

1 comment on commit b293856

@openjdk-notifier
Copy link

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.