test/jdk/jdk/jfr/api/consumer/TestRecordingFile.java
author egahlin
Tue, 22 May 2018 23:27:54 +0200
changeset 50225 2942ae532175
parent 50113 caf115bb98ad
child 51214 67736b4846a0
permissions -rw-r--r--
8201331: Better test cleanup for jdk/jfr/api/consumer/TestRecordingFile.java Reviewed-by: tbell, mgronlun
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     1
/*
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     2
 * Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     4
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     7
 * published by the Free Software Foundation.  Oracle designates this
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     8
 * particular file as subject to the "Classpath" exception as provided
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     9
 * by Oracle in the LICENSE file that accompanied this code.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    10
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    11
 * This code is distributed in the hope that it will be useful, but WITHOUT
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    12
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    13
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    14
 * version 2 for more details (a copy is included in the LICENSE file that
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    15
 * accompanied this code).
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    16
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    17
 * You should have received a copy of the GNU General Public License version
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    18
 * 2 along with this work; if not, write to the Free Software Foundation,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    19
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    20
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    21
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    22
 * or visit www.oracle.com if you need additional information or have any
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    23
 * questions.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    24
 */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    25
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    26
package jdk.jfr.api.consumer;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    27
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    28
import java.io.FileNotFoundException;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    29
import java.io.FileWriter;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    30
import java.io.IOException;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    31
import java.io.RandomAccessFile;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    32
import java.nio.file.Files;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    33
import java.nio.file.Path;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    34
import java.nio.file.Paths;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    35
import java.util.Arrays;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    36
import java.util.HashSet;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    37
import java.util.List;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    38
import java.util.StringJoiner;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    39
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    40
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    41
import jdk.jfr.Event;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    42
import jdk.jfr.EventType;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    43
import jdk.jfr.FlightRecorder;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    44
import jdk.jfr.Name;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    45
import jdk.jfr.Recording;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    46
import jdk.jfr.Registered;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    47
import jdk.jfr.consumer.RecordedEvent;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    48
import jdk.jfr.consumer.RecordingFile;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    49
import jdk.test.lib.Asserts;
50225
2942ae532175 8201331: Better test cleanup for jdk/jfr/api/consumer/TestRecordingFile.java
egahlin
parents: 50113
diff changeset
    50
import jdk.test.lib.Utils;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    51
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    52
/*
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    53
 * @test
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    54
 * @summary Verifies that all methods in RecordingFIle are working
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    55
 * @key jfr
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    56
 * @library /test/lib
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    57
 * @run main/othervm jdk.jfr.api.consumer.TestRecordingFile
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    58
 */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    59
public class TestRecordingFile {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    60
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    61
    static class TestEvent1 extends Event {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    62
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    63
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    64
    static class TestEvent2 extends Event {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    65
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    66
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    67
    static class TestEvent3 extends Event {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    68
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    69
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    70
    private static String TEST_CLASS_BASE = "TestRecordingFile$TestEvent";
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    71
    private final static int EVENT_COUNT = 3;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    72
    private final static int HEADER_SIZE = 68;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    73
    private final static long METADATA_OFFSET = 24;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    74
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    75
    public static void main(String[] args) throws Throwable {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    76
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    77
        // create some recording data
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    78
        Recording r = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    79
        r.enable(TestEvent1.class).withoutStackTrace();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    80
        r.enable(TestEvent2.class).withoutStackTrace();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    81
        r.enable(TestEvent3.class).withoutStackTrace();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    82
        r.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    83
        TestEvent1 t1 = new TestEvent1();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    84
        t1.commit();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    85
        TestEvent2 t2 = new TestEvent2();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    86
        t2.commit();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    87
        TestEvent3 t3 = new TestEvent3();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    88
        t3.commit();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    89
        r.stop();
50225
2942ae532175 8201331: Better test cleanup for jdk/jfr/api/consumer/TestRecordingFile.java
egahlin
parents: 50113
diff changeset
    90
        Path valid = Utils.createTempFile("three-event-recording", ".jfr");
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    91
        r.dump(valid);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    92
        r.close();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    93
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    94
        Path brokenWithZeros = createBrokenWIthZeros(valid);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    95
        Path brokenMetadata = createBrokenMetadata(valid);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    96
        // prepare event sets
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    97
        testNewRecordingFile(valid, brokenWithZeros);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    98
        testIterate(valid, brokenWithZeros);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    99
        testReadAllEvents(valid, brokenWithZeros);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   100
        testReadEventTypes(valid, brokenMetadata);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   101
        testClose(valid);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   102
        testReadEventTypesMultiChunk();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   103
        testReadEventTypeWithUnregistration(false, false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   104
        testReadEventTypeWithUnregistration(false, true);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   105
        testReadEventTypeWithUnregistration(true, false);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   106
        testReadEventTypeWithUnregistration(true, true);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   107
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   108
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   109
    private static void testReadEventTypeWithUnregistration(boolean disk, boolean background) throws Exception {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   110
        FlightRecorder.register(Event1.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   111
        FlightRecorder.register(Event2.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   112
        FlightRecorder.register(Event3.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   113
        Recording backgrundRecording = new Recording();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   114
        if (disk) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   115
            backgrundRecording.setToDisk(disk);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   116
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   117
        if (background) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   118
            backgrundRecording.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   119
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   120
        recordAndVerify(disk, background,new int[] {1,2, 3}, new int[] {});
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   121
        FlightRecorder.unregister(Event2.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   122
        recordAndVerify(disk, background,  new int[] {1, 3}, new int[] {2});
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   123
        FlightRecorder.unregister(Event1.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   124
        FlightRecorder.register(Event2.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   125
        recordAndVerify(disk,background, new int[] {2, 3}, new int[] {1});
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   126
        FlightRecorder.unregister(Event3.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   127
        FlightRecorder.register(Event3.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   128
        FlightRecorder.unregister(Event2.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   129
        FlightRecorder.unregister(Event3.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   130
        FlightRecorder.register(Event1.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   131
        FlightRecorder.unregister(Event1.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   132
        FlightRecorder.register(Event1.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   133
        FlightRecorder.register(Event2.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   134
        recordAndVerify(disk, background,new int[] {1, 2}, new int[] {3});
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   135
        if (background) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   136
            backgrundRecording.close();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   137
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   138
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   139
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   140
    private static void recordAndVerify(boolean disk, boolean background,  int[] shouldExist, int[] shouldNotExist) throws Exception {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   141
        StringJoiner sb  = new StringJoiner("-");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   142
        for (int i = 0; i <shouldExist.length; i++) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   143
            sb.add(Integer.toString(shouldExist[i]));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   144
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   145
        System.out.println("Verifying recordings: disk=" + disk + " background=" + background);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   146
        System.out.println("Should exist: " + Arrays.toString(shouldExist));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   147
        System.out.println("Should not exist: " + Arrays.toString(shouldNotExist));
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   148
50225
2942ae532175 8201331: Better test cleanup for jdk/jfr/api/consumer/TestRecordingFile.java
egahlin
parents: 50113
diff changeset
   149
        Path p = Utils.createTempFile(sb.toString(), ".jfr");
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   150
        System.out.println("Filename: " + p);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   151
        try (Recording r = new Recording()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   152
            r.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   153
            r.stop();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   154
            r.dump(p);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   155
            try (RecordingFile f = new RecordingFile(p)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   156
                List<EventType> types = f.readEventTypes();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   157
                for (int i = 0; i< shouldExist.length; i++) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   158
                    assertHasEventType(types, "Event" + shouldExist[i]);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   159
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   160
                for (int i = 0; i< shouldNotExist.length; i++) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   161
                    assertMissingEventType(types, "Event" + shouldNotExist[i]);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   162
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   163
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   164
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   165
        System.out.println();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   166
        System.out.println();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   167
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   168
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   169
    @Registered(false)
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   170
    @Name("Event1")
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   171
    private static class Event1 extends Event {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   172
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   173
    @Registered(false)
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   174
    @Name("Event2")
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   175
    private static class Event2 extends Event {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   176
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   177
    @Registered(false)
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   178
    @Name("Event3")
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   179
    private static class Event3 extends Event {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   180
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   181
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   182
    private static void testReadEventTypesMultiChunk() throws Exception {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   183
50225
2942ae532175 8201331: Better test cleanup for jdk/jfr/api/consumer/TestRecordingFile.java
egahlin
parents: 50113
diff changeset
   184
        Path twoEventTypes = Utils.createTempFile("two-event-types", ".jfr");
2942ae532175 8201331: Better test cleanup for jdk/jfr/api/consumer/TestRecordingFile.java
egahlin
parents: 50113
diff changeset
   185
        Path threeEventTypes = Utils.createTempFile("three-event-types", ".jfr");
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   186
       try (Recording r1 = new Recording()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   187
           r1.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   188
           FlightRecorder.register(Event1.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   189
           try (Recording r2 = new Recording()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   190
               r2.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   191
               FlightRecorder.register(Event2.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   192
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   193
               // Ensure that metadata are written twice.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   194
               try (Recording rotator = new Recording()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   195
                   rotator.start();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   196
                   rotator.stop();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   197
               }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   198
               r2.stop();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   199
               r2.dump(twoEventTypes);;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   200
           }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   201
           FlightRecorder.register(Event3.class);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   202
           r1.stop();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   203
           r1.dump(threeEventTypes);;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   204
       }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   205
       try (RecordingFile f = new RecordingFile(twoEventTypes)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   206
           List<EventType> types = f.readEventTypes();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   207
           assertUniqueEventTypes(types);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   208
           assertHasEventType(types, "Event1");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   209
           assertHasEventType(types, "Event2");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   210
           assertMissingEventType(types, "Event3");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   211
       }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   212
       try (RecordingFile f = new RecordingFile(twoEventTypes)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   213
           List<EventType> types = f.readEventTypes();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   214
           assertUniqueEventTypes(types);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   215
           assertHasEventType(types, "Event1");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   216
           assertHasEventType(types, "Event2");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   217
           assertMissingEventType(types, "Event3");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   218
       }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   219
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   220
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   221
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   222
    private static void assertMissingEventType(List<EventType> types,String name) throws Exception {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   223
        EventType type = findEventType(types, name);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   224
        if (type != null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   225
            throw new Exception("Found unexpected event type " + name);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   226
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   227
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   228
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   229
    private static void assertHasEventType(List<EventType> types,String name) throws Exception {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   230
        EventType type = findEventType(types, name);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   231
        if (type == null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   232
            throw new Exception("Missing event type " + name);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   233
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   234
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   235
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   236
    private static EventType findEventType(List<EventType> types, String name) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   237
        for (EventType t : types) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   238
            if (t.getName().equals(name)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   239
                return t;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   240
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   241
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   242
        return null;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   243
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   244
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   245
    private static void assertUniqueEventTypes(List<EventType> types) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   246
        HashSet<Long> ids = new HashSet<>();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   247
        for (EventType type : types) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   248
            ids.add(type.getId());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   249
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   250
        Asserts.assertEquals(types.size(), ids.size(), "Event types repeated. " + types);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   251
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   252
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   253
    private static Path createBrokenWIthZeros(Path valid) throws Exception {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   254
        try {
50225
2942ae532175 8201331: Better test cleanup for jdk/jfr/api/consumer/TestRecordingFile.java
egahlin
parents: 50113
diff changeset
   255
            Path broken = Utils.createTempFile("broken-events", ".jfr");
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   256
            Files.delete(broken);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   257
            Files.copy(valid, broken);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   258
            RandomAccessFile raf = new RandomAccessFile(broken.toFile(), "rw");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   259
            raf.seek(HEADER_SIZE);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   260
            int size = (int) Files.size(broken);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   261
            byte[] ones = new byte[size - HEADER_SIZE];
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   262
            for (int i = 0; i < ones.length; i++) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   263
                ones[i] = (byte) 0xFF;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   264
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   265
            raf.write(ones, 0, ones.length);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   266
            raf.close();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   267
            return broken;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   268
        } catch (IOException ioe) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   269
            throw new Exception("Could not produce a broken file " + valid, ioe);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   270
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   271
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   272
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   273
    private static Path createBrokenMetadata(Path valid) throws Exception {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   274
        try {
50225
2942ae532175 8201331: Better test cleanup for jdk/jfr/api/consumer/TestRecordingFile.java
egahlin
parents: 50113
diff changeset
   275
            Path broken = Utils.createTempFile("broken-metadata", ".jfr");
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   276
            Files.delete(broken);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   277
            Files.copy(valid, broken);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   278
            RandomAccessFile raf = new RandomAccessFile(broken.toFile(), "rw");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   279
            raf.seek(METADATA_OFFSET);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   280
            long metadataOffset = raf.readLong();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   281
            raf.seek(metadataOffset);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   282
            raf.writeLong(Long.MAX_VALUE);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   283
            raf.writeLong(Long.MAX_VALUE);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   284
            raf.close();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   285
            return broken;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   286
        } catch (IOException ioe) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   287
            throw new Exception("Could not produce a broken EventSet from file " + valid, ioe);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   288
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   289
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   290
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   291
    private static void testReadEventTypes(Path valid, Path broken) throws Exception {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   292
        try (RecordingFile validFile = new RecordingFile(valid)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   293
            List<EventType> types = validFile.readEventTypes();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   294
            if (types.size() < EVENT_COUNT) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   295
                throw new Exception("Expected at least " + EVENT_COUNT + " event type but got " + types.toString());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   296
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   297
            int counter = 0;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   298
            for (Class<?> testClass : Arrays.asList(TestEvent1.class, TestEvent2.class, TestEvent3.class)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   299
                for (EventType t : types) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   300
                    if (t.getName().equals(testClass.getName())) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   301
                        counter++;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   302
                    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   303
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   304
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   305
            if (counter != 3) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   306
                throw new Exception("Returned incorrect event types");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   307
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   308
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   309
        try (RecordingFile brokenFile = new RecordingFile(broken)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   310
            brokenFile.readEventTypes();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   311
            throw new Exception("Expected IOException when getting Event Types from broken recording");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   312
        } catch (IOException ise) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   313
            // OK
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   314
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   315
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   316
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   317
    private static void testNewRecordingFile(Path valid, Path broken) throws Exception {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   318
        try (RecordingFile r = new RecordingFile(null)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   319
            throw new Exception("Expected NullPointerException");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   320
        } catch (NullPointerException npe) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   321
            // OK
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   322
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   323
        try (RecordingFile r = new RecordingFile(Paths.get("hjhjsdfhkjshdfkj.jfr"))) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   324
            throw new Exception("Expected FileNotFoundException");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   325
        } catch (FileNotFoundException npe) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   326
            // OK
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   327
        }
50225
2942ae532175 8201331: Better test cleanup for jdk/jfr/api/consumer/TestRecordingFile.java
egahlin
parents: 50113
diff changeset
   328
        Path testFile = Utils.createTempFile("test-empty-file", ".jfr");
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   329
        try (RecordingFile r = new RecordingFile(testFile)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   330
            throw new Exception("Expected IOException if file is empty");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   331
        } catch (IOException e) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   332
            // OK
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   333
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   334
        FileWriter fr = new FileWriter(testFile.toFile());
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   335
        fr.write("whatever");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   336
        fr.close();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   337
        try (RecordingFile r = new RecordingFile(Paths.get("hjhjsdfhkjshdfkj.jfr"))) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   338
            throw new Exception("Expected IOException if magic is incorrect");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   339
        } catch (IOException e) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   340
            // OK
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   341
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   342
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   343
        try (RecordingFile r = new RecordingFile(valid)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   344
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   345
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   346
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   347
    private static void testClose(Path valid) throws Exception {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   348
        RecordingFile f = new RecordingFile(valid);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   349
        f.close();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   350
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   351
        try {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   352
            f.readEvent();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   353
            throw new Exception("Should not be able to read event from closed recording");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   354
        } catch (IOException e) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   355
            if (!e.getMessage().equals("Stream Closed")) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   356
                throw new Exception("Expected 'Stream Closed' in exception message for a closed stream. Got '" + e.getMessage() +"'.");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   357
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   358
            // OK
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   359
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   360
        try {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   361
            f.readEventTypes();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   362
            throw new Exception("Should not be able to read event from closed recording");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   363
        } catch (IOException e) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   364
            if (!e.getMessage().equals("Stream Closed")) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   365
                throw new Exception("Expected 'Stream Closed' in exception message for a closed stream. Got '" + e.getMessage() +"'.");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   366
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   367
            // OK
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   368
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   369
        // close twice
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   370
        f.close();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   371
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   372
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   373
    private static void testIterate(Path valid, Path broken) throws Exception {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   374
        try (RecordingFile validFile = new RecordingFile(valid)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   375
            for (int i = 0; i < EVENT_COUNT; i++) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   376
                if (!validFile.hasMoreEvents()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   377
                    throw new Exception("Not all events available");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   378
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   379
                RecordedEvent r = validFile.readEvent();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   380
                if (r == null) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   381
                    throw new Exception("Missing event");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   382
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   383
                if (!r.getEventType().getName().contains(TEST_CLASS_BASE)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   384
                    throw new Exception("Incorrect event in recording file " + r);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   385
                }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   386
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   387
            if (validFile.hasMoreEvents()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   388
                throw new Exception("Should not be more than " + EVENT_COUNT + " in recording");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   389
            }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   390
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   391
        try (RecordingFile brokenFile = new RecordingFile(broken)) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   392
            brokenFile.readEvent();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   393
            throw new Exception("Expected IOException for broken recording");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   394
        } catch (IOException ise) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   395
            // OK
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   396
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   397
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   398
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   399
    private static void testReadAllEvents(Path valid, Path broken) throws Exception {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   400
        try {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   401
            RecordingFile.readAllEvents(broken);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   402
            throw new Exception("Expected IOException when reading all events for broken recording");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   403
        } catch (IOException ioe) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   404
            // OK as expected
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   405
        }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   406
    }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   407
}