test/jdk/jdk/jfr/event/metadata/TestLookForUntestedEvents.java
changeset 52704 64413aaea8ed
child 52911 e0ce50c5e220
equal deleted inserted replaced
52703:e7fdc9d9c376 52704:64413aaea8ed
       
     1 /*
       
     2  * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved.
       
     3  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
       
     4  *
       
     5  * This code is free software; you can redistribute it and/or modify it
       
     6  * under the terms of the GNU General Public License version 2 only, as
       
     7  * published by the Free Software Foundation.  Oracle designates this
       
     8  * particular file as subject to the "Classpath" exception as provided
       
     9  * by Oracle in the LICENSE file that accompanied this code.
       
    10  *
       
    11  * This code is distributed in the hope that it will be useful, but WITHOUT
       
    12  * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
       
    13  * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
       
    14  * version 2 for more details (a copy is included in the LICENSE file that
       
    15  * accompanied this code).
       
    16  *
       
    17  * You should have received a copy of the GNU General Public License version
       
    18  * 2 along with this work; if not, write to the Free Software Foundation,
       
    19  * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
       
    20  *
       
    21  * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
       
    22  * or visit www.oracle.com if you need additional information or have any
       
    23  * questions.
       
    24  */
       
    25 package jdk.jfr.event.metadata;
       
    26 
       
    27 import java.io.File;
       
    28 import java.io.IOException;
       
    29 import java.lang.reflect.Field;
       
    30 import java.nio.file.Files;
       
    31 import java.nio.file.Path;
       
    32 import java.nio.file.Paths;
       
    33 import java.util.Arrays;
       
    34 import java.util.HashSet;
       
    35 import java.util.List;
       
    36 import java.util.Set;
       
    37 import java.util.stream.Collectors;
       
    38 import java.util.stream.Stream;
       
    39 
       
    40 import jdk.jfr.EventType;
       
    41 import jdk.jfr.Experimental;
       
    42 import jdk.jfr.FlightRecorder;
       
    43 import jdk.test.lib.jfr.EventNames;
       
    44 import jdk.test.lib.Utils;
       
    45 
       
    46 /**
       
    47  * @test Check for JFR events not covered by tests
       
    48  * @key jfr
       
    49  * @requires vm.hasJFR
       
    50  * @library /test/lib /test/jdk
       
    51  * @run main jdk.jfr.event.metadata.TestLookForUntestedEvents
       
    52  */
       
    53 public class TestLookForUntestedEvents {
       
    54     private static final Path jfrTestRoot = Paths.get(Utils.TEST_SRC).getParent().getParent();
       
    55     private static final String MSG_SEPARATOR = "==========================";
       
    56     private static Set<String> jfrEventTypes = new HashSet<>();
       
    57 
       
    58     private static final Set<String> knownEventsMissingFromEventNames = new HashSet<>(
       
    59         Arrays.asList(
       
    60             // The Z* events below should be marked as experimental; see: JDK-8213966
       
    61             "ZStatisticsSampler", "ZStatisticsCounter",
       
    62             "ZPageAllocation", "ZThreadPhase"
       
    63         )
       
    64     );
       
    65 
       
    66     private static final Set<String> hardToTestEvents = new HashSet<>(
       
    67         Arrays.asList(
       
    68             "DataLoss", "IntFlag", "ReservedStackActivation",
       
    69             "DoubleFlag", "UnsignedLongFlagChanged", "IntFlagChanged",
       
    70             "UnsignedIntFlag", "UnsignedIntFlagChanged", "DoubleFlagChanged")
       
    71     );
       
    72 
       
    73     // GC uses specific framework to test the events, instead of using event names literally.
       
    74     // GC tests were inspected, as well as runtime output of GC tests.
       
    75     // The following events below are know to be covered based on that inspection.
       
    76     private static final Set<String> coveredGcEvents = new HashSet<>(
       
    77         Arrays.asList(
       
    78             "MetaspaceGCThreshold", "MetaspaceAllocationFailure", "MetaspaceOOM",
       
    79             "MetaspaceChunkFreeListSummary", "G1HeapSummary", "ParallelOldGarbageCollection",
       
    80             "OldGarbageCollection", "G1GarbageCollection", "GCPhasePause",
       
    81             "GCPhasePauseLevel1", "GCPhasePauseLevel2", "GCPhasePauseLevel3",
       
    82             "GCPhasePauseLevel4", "GCPhaseConcurrent")
       
    83     );
       
    84 
       
    85     // This is a "known failure list" for this test.
       
    86     // NOTE: if the event is not covered, a bug should be open, and bug number
       
    87     // noted in the comments for this set.
       
    88     private static final Set<String> knownNotCoveredEvents = new HashSet<>(
       
    89         // DumpReason: JDK-8213918, Shutdown: JDK-8213917
       
    90         Arrays.asList("DumpReason", "Shutdown")
       
    91     );
       
    92 
       
    93 
       
    94     public static void main(String[] args) throws Exception {
       
    95         for (EventType type : FlightRecorder.getFlightRecorder().getEventTypes()) {
       
    96             if (type.getAnnotation(Experimental.class) == null) {
       
    97                 jfrEventTypes.add(type.getName().replace("jdk.", ""));
       
    98             }
       
    99         }
       
   100 
       
   101         checkEventNamesClass();
       
   102         lookForEventsNotCoveredByTests();
       
   103     }
       
   104 
       
   105     // Look thru JFR tests to make sure JFR events are referenced in the tests
       
   106     private static void lookForEventsNotCoveredByTests() throws Exception {
       
   107         List<Path> paths = Files.walk(jfrTestRoot)
       
   108             .filter(Files::isRegularFile)
       
   109             .filter(path -> isJavaFile(path))
       
   110             .collect(Collectors.toList());
       
   111 
       
   112         Set<String> eventsNotCoveredByTest = new HashSet<>(jfrEventTypes);
       
   113         for (String event : jfrEventTypes) {
       
   114             for (Path p : paths) {
       
   115                 if (findStringInFile(p, event)) {
       
   116                     eventsNotCoveredByTest.remove(event);
       
   117                     break;
       
   118                 }
       
   119             }
       
   120         }
       
   121 
       
   122         // Account for hard-to-test, experimental and GC tested events
       
   123         eventsNotCoveredByTest.removeAll(hardToTestEvents);
       
   124         eventsNotCoveredByTest.removeAll(coveredGcEvents);
       
   125         eventsNotCoveredByTest.removeAll(knownNotCoveredEvents);
       
   126 
       
   127         if (!eventsNotCoveredByTest.isEmpty()) {
       
   128             print(MSG_SEPARATOR + " Events not covered by test");
       
   129             for (String event: eventsNotCoveredByTest) {
       
   130                 print(event);
       
   131             }
       
   132             print(MSG_SEPARATOR);
       
   133             throw new RuntimeException("Found JFR events not covered by tests");
       
   134         }
       
   135     }
       
   136 
       
   137     // Make sure all the JFR events are accounted for in jdk.test.lib.jfr.EventNames
       
   138     private static void checkEventNamesClass() throws Exception {
       
   139         // jdk.test.lib.jfr.EventNames
       
   140         Set<String> eventsFromEventNamesClass = new HashSet<>();
       
   141         for (Field f : EventNames.class.getFields()) {
       
   142             String name = f.getName();
       
   143             if (!name.equals("PREFIX")) {
       
   144                 String eventName = (String) f.get(null);
       
   145                 eventName = eventName.replace(EventNames.PREFIX, "");
       
   146                 eventsFromEventNamesClass.add(eventName);
       
   147             }
       
   148         }
       
   149 
       
   150         // Account for the events that are known to be missing from the EventNames.java
       
   151         eventsFromEventNamesClass.addAll(knownEventsMissingFromEventNames);
       
   152 
       
   153         if (!jfrEventTypes.equals(eventsFromEventNamesClass)) {
       
   154             String exceptionMsg = "Events declared in jdk.test.lib.jfr.EventNames differ " +
       
   155                          "from events returned by FlightRecorder.getEventTypes()";
       
   156             print(MSG_SEPARATOR);
       
   157             print(exceptionMsg);
       
   158             print("");
       
   159             printSetDiff(jfrEventTypes, eventsFromEventNamesClass,
       
   160                         "jfrEventTypes", "eventsFromEventNamesClass");
       
   161             print("");
       
   162 
       
   163             print("This could be because:");
       
   164             print("1) You forgot to write a unit test. Please do so in test/jdk/jdk/jfr/event/");
       
   165             print("2) You wrote a unit test, but you didn't reference the event in");
       
   166             print("   test/lib/jdk/test/lib/jfr/EventNames.java. ");
       
   167             print("3) It is not feasible to test the event, not even a sanity test. ");
       
   168             print("   Add the event name to test/lib/jdk/test/lib/jfr/EventNames.java ");
       
   169             print("   and a short comment why it can't be tested");
       
   170             print("4) The event is experimental. Please add 'experimental=\"true\"' to <Event> ");
       
   171             print("   element in metadata.xml if it is a native event, or @Experimental if it is a ");
       
   172             print("   Java event. The event will now not show up in JMC");
       
   173             System.out.println(MSG_SEPARATOR);
       
   174             throw new RuntimeException(exceptionMsg);
       
   175         }
       
   176     }
       
   177 
       
   178     // ================ Helper methods
       
   179     private static boolean isJavaFile(Path p) {
       
   180         String fileName = p.getFileName().toString();
       
   181         int i = fileName.lastIndexOf('.');
       
   182         if ( (i < 0) || (i > fileName.length()) ) {
       
   183             return false;
       
   184         }
       
   185         return "java".equals(fileName.substring(i+1));
       
   186     }
       
   187 
       
   188     private static boolean findStringInFile(Path p, String searchTerm) throws IOException {
       
   189         long c = 0;
       
   190         try (Stream<String> stream = Files.lines(p)) {
       
   191             c = stream
       
   192                 .filter(line -> line.contains(searchTerm))
       
   193                 .count();
       
   194         }
       
   195         return (c != 0);
       
   196     }
       
   197 
       
   198     private static void printSetDiff(Set<String> a, Set<String> b,
       
   199         String setAName, String setBName) {
       
   200         if (a.size() > b.size()) {
       
   201             a.removeAll(b);
       
   202             System.out.printf("Set %s has more elements than set %s:", setAName, setBName);
       
   203             System.out.println();
       
   204             printSet(a);
       
   205         } else {
       
   206             b.removeAll(a);
       
   207             System.out.printf("Set %s has more elements than set %s:", setBName, setAName);
       
   208             System.out.println();
       
   209             printSet(b);
       
   210         }
       
   211     }
       
   212 
       
   213     private static void printSet(Set<String> set) {
       
   214         for (String e : set) {
       
   215             System.out.println(e);
       
   216         }
       
   217     }
       
   218 
       
   219     private static void print(String s) {
       
   220         System.out.println(s);
       
   221     }
       
   222 }