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

gtest runner repeats test name for every single gtest assertion

    XMLWordPrintable

Details

    • b10

    Backports

      Description

        GtestParser currently prints tests multiple times that fail multiple times=???

        When gtests fail multiple/many times within the same test (that can happen if e.g. EXPECT or ASSERT are used in nested sub routines, since ASSERT only cancels the outermost subroutine, and EXPECT does nothing at all, only prints the message), the test logs produced by the jtreg gtest runner get flooded with endlessly repeated test names. This is especially annoying in GHA logs.

        Example: the following code:

        ```
        static void testtesttest2 () {
          for (int i = 0; i < 100; i++) {
          EXPECT_TRUE(false);
          }
        }

        static void testtesttest () {
          testtesttest2();
        }

        TEST_VM(os, xtesttestttest) {
          testtesttest();
        }
        ```

        produces this output where the failing test name is repeated for every error:

        ```
        description=file\:/shared/projects/openjdk/jdk-jdk/source/test/hotspot/jtreg/gtest/GTestWrapper.java
        elapsed=60516 0\:01\:00.516
        end=Wed Jan 17 14\:11\:52 CET 2024
        environment=regtest
        execStatus=Failed. Execution failed\: `main' threw exception\: java.lang.AssertionError\: gtest execution failed; exit code \= 2. the failed tests\: [os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm, os\:\:xtesttestttest_vm]
        ```

        I would like it to print the failing test just once.

        Attachments

          Issue Links

            Activity

              People

                szaldana Sonia Zaldana Calles
                stuefe Thomas Stuefe
                Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: