Skip to content

Commit c994bfa

Browse files
lemniklemnik
committed
refactor(Event): rename Report -> Event
1 parent 0d46121 commit c994bfa

35 files changed

Lines changed: 368 additions & 368 deletions

‎bugsnag-spring/src/main/java/com/bugsnag/BugsnagSpringConfiguration.java‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ public class BugsnagSpringConfiguration implements InitializingBean {
2828
Callback springVersionErrorCallback() {
2929
Callback callback = new Callback() {
3030
@Override
31-
public boolean onError(Report report) {
31+
public boolean onError(Event report) {
3232
addSpringRuntimeVersion(report.getDevice());
3333
return true;
3434
}

‎bugsnag-spring/src/main/java/com/bugsnag/ExceptionClassCallback.java‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -111,9 +111,9 @@ class ExceptionClassCallback implements Callback {
111111
}
112112

113113
@Override
114-
public boolean onError(Report report) {
114+
public boolean onError(Event event) {
115115

116-
HandledState handledState = report.getHandledState();
116+
HandledState handledState = event.getHandledState();
117117

118118
// A manually-set severity takes precedence
119119
SeverityReasonType severityReasonType = handledState.calculateSeverityReasonType();
@@ -122,12 +122,12 @@ public boolean onError(Report report) {
122122
return true; // do not change delivery decision
123123
}
124124

125-
Class exceptionClass = report.getException().getClass();
125+
Class exceptionClass = event.getException().getClass();
126126

127127
if (EXCEPTION_TO_SEVERITY.containsKey(exceptionClass)) {
128128
Severity severity = EXCEPTION_TO_SEVERITY.get(exceptionClass);
129-
report.setSeverity(severity);
130-
report.setHandledState(HandledState.newInstance(
129+
event.setSeverity(severity);
130+
event.setHandledState(HandledState.newInstance(
131131
SeverityReasonType.REASON_EXCEPTION_CLASS,
132132
Collections.singletonMap("exceptionClass", exceptionClass.getSimpleName()),
133133
severity,

‎bugsnag-spring/src/main/java/com/bugsnag/SpringBootConfiguration.java‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ public class SpringBootConfiguration {
1919
Callback springBootVersionErrorCallback() {
2020
Callback callback = new Callback() {
2121
@Override
22-
public boolean onError(Report report) {
22+
public boolean onError(Event report) {
2323
addSpringRuntimeVersion(report.getDevice());
2424
return true;
2525
}

‎bugsnag-spring/src/test/java/com/bugsnag/NotifierTest.java‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111

1212
public class NotifierTest {
1313

14-
private Report report;
14+
private Event event;
1515
private Configuration config;
1616
private SessionPayload sessionPayload;
1717

@@ -23,13 +23,13 @@ public class NotifierTest {
2323
@Before
2424
public void setUp() throws Throwable {
2525
config = new Configuration("api-key");
26-
report = new Report(config, new RuntimeException());
26+
event = new Event(config, new RuntimeException());
2727
sessionPayload = new SessionPayload(Collections.<SessionCount>emptyList(), config);
2828
}
2929

3030
@Test
3131
public void testNotificationSerialisation() throws Throwable {
32-
JsonNode payload = BugsnagTestUtils.mapReportToJson(config, this.report);
32+
JsonNode payload = BugsnagTestUtils.mapReportToJson(config, this.event);
3333
JsonNode notifier = payload.get("notifier");
3434

3535
assertEquals("Bugsnag Spring", notifier.get("name").asText());

‎bugsnag-spring/src/test/java/com/bugsnag/SpringAsyncTest.java‎

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -50,41 +50,41 @@ public void setUp() {
5050
public void bugsnagNotifyWhenAsyncVoidReturnTypeException() {
5151
asyncService.throwExceptionVoid();
5252

53-
Report report = verifyAndGetReport(delivery);
53+
Event event = verifyAndGetReport(delivery);
5454

5555
// Assert that the exception was detected correctly
56-
assertEquals("Async void test", report.getExceptionMessage());
57-
assertEquals("java.lang.RuntimeException", report.getExceptionName());
56+
assertEquals("Async void test", event.getExceptionMessage());
57+
assertEquals("java.lang.RuntimeException", event.getExceptionName());
5858

5959
// Assert that the severity, severity reason and unhandled values are correct
60-
assertEquals(Severity.ERROR.getValue(), report.getSeverity());
60+
assertEquals(Severity.ERROR.getValue(), event.getSeverity());
6161
assertEquals(
6262
SeverityReasonType.REASON_UNHANDLED_EXCEPTION_MIDDLEWARE.toString(),
63-
report.getSeverityReason().getType());
63+
event.getSeverityReason().getType());
6464
assertThat(
65-
report.getSeverityReason().getAttributes(),
65+
event.getSeverityReason().getAttributes(),
6666
is(Collections.singletonMap("framework", "Spring")));
67-
assertTrue(report.getUnhandled());
67+
assertTrue(event.getUnhandled());
6868
}
6969

7070
@Test
7171
public void bugsnagNotifyWhenAsyncFutureReturnTypeException() {
7272
asyncService.throwExceptionFuture();
7373

74-
Report report = verifyAndGetReport(delivery);
74+
Event event = verifyAndGetReport(delivery);
7575

7676
// Assert that the exception was detected correctly
77-
assertEquals("Async future test", report.getExceptionMessage());
78-
assertEquals("java.lang.RuntimeException", report.getExceptionName());
77+
assertEquals("Async future test", event.getExceptionMessage());
78+
assertEquals("java.lang.RuntimeException", event.getExceptionName());
7979

8080
// Assert that the severity, severity reason and unhandled values are correct
81-
assertEquals(Severity.ERROR.getValue(), report.getSeverity());
81+
assertEquals(Severity.ERROR.getValue(), event.getSeverity());
8282
assertEquals(
8383
SeverityReasonType.REASON_UNHANDLED_EXCEPTION_MIDDLEWARE.toString(),
84-
report.getSeverityReason().getType());
84+
event.getSeverityReason().getType());
8585
assertThat(
86-
report.getSeverityReason().getAttributes(),
86+
event.getSeverityReason().getAttributes(),
8787
is(Collections.singletonMap("framework", "Spring")));
88-
assertTrue(report.getUnhandled());
88+
assertTrue(event.getUnhandled());
8989
}
9090
}

‎bugsnag-spring/src/test/java/com/bugsnag/SpringMvcTest.java‎

Lines changed: 32 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -83,21 +83,21 @@ public void setUp() {
8383
public void bugsnagNotifyWhenUncaughtControllerException() {
8484
callRuntimeExceptionEndpoint();
8585

86-
Report report = verifyAndGetReport(delivery);
86+
Event event = verifyAndGetReport(delivery);
8787

8888
// Assert that the exception was detected correctly
89-
assertEquals("Test", report.getExceptionMessage());
90-
assertEquals("java.lang.RuntimeException", report.getExceptionName());
89+
assertEquals("Test", event.getExceptionMessage());
90+
assertEquals("java.lang.RuntimeException", event.getExceptionName());
9191

9292
// Assert that the severity, severity reason and unhandled values are correct
93-
assertEquals(Severity.ERROR.getValue(), report.getSeverity());
93+
assertEquals(Severity.ERROR.getValue(), event.getSeverity());
9494
assertEquals(
9595
SeverityReasonType.REASON_UNHANDLED_EXCEPTION_MIDDLEWARE.toString(),
96-
report.getSeverityReason().getType());
96+
event.getSeverityReason().getType());
9797
assertThat(
98-
report.getSeverityReason().getAttributes(),
98+
event.getSeverityReason().getAttributes(),
9999
is(Collections.singletonMap("framework", "Spring")));
100-
assertTrue(report.getUnhandled());
100+
assertTrue(event.getUnhandled());
101101
}
102102

103103
@Test
@@ -129,14 +129,14 @@ public void noBugsnagSessionStartedWhenAutoCaptureSessionsFalse() {
129129
public void requestMetadataSetCorrectly() {
130130
callRuntimeExceptionEndpoint();
131131

132-
Report report = verifyAndGetReport(delivery);
132+
Event event = verifyAndGetReport(delivery);
133133

134134
// Check that the context is set to the HTTP method and URI of the endpoint
135-
assertEquals("GET /throw-runtime-exception", report.getContext());
135+
assertEquals("GET /throw-runtime-exception", event.getContext());
136136

137137
// Check that the request metadata is set as expected
138138
@SuppressWarnings(value = "unchecked") Map<String, Object> requestMetadata =
139-
(Map<String, Object>) report.getMetadata().get("request");
139+
(Map<String, Object>) event.getMetadata().get("request");
140140
assertEquals("http://localhost:" + randomServerPort + "/throw-runtime-exception",
141141
requestMetadata.get("url"));
142142
assertEquals("GET", requestMetadata.get("method"));
@@ -161,10 +161,10 @@ public void requestMetadataSetCorrectly() {
161161
public void springVersionSetCorrectly() {
162162
callRuntimeExceptionEndpoint();
163163

164-
Report report = verifyAndGetReport(delivery);
164+
Event event = verifyAndGetReport(delivery);
165165

166166
// Check that the Spring version is set as expected
167-
Map<String, Object> deviceMetadata = report.getDevice();
167+
Map<String, Object> deviceMetadata = event.getDevice();
168168
Map<String, Object> runtimeVersions =
169169
(Map<String, Object>) deviceMetadata.get("runtimeVersions");
170170
assertEquals(SpringVersion.getVersion(), runtimeVersions.get("springFramework"));
@@ -175,22 +175,22 @@ public void springVersionSetCorrectly() {
175175
public void unhandledTypeMismatchExceptionSeverityInfo() {
176176
callUnhandledTypeMismatchExceptionEndpoint();
177177

178-
Report report = verifyAndGetReport(delivery);
178+
Event event = verifyAndGetReport(delivery);
179179

180-
assertTrue(report.getUnhandled());
181-
assertEquals("info", report.getSeverity());
182-
assertEquals("exceptionClass", report.getSeverityReason().getType());
183-
assertThat(report.getSeverityReason().getAttributes(),
180+
assertTrue(event.getUnhandled());
181+
assertEquals("info", event.getSeverity());
182+
assertEquals("exceptionClass", event.getSeverityReason().getType());
183+
assertThat(event.getSeverityReason().getAttributes(),
184184
is(Collections.singletonMap("exceptionClass", "TypeMismatchException")));
185185
}
186186

187187
@Test
188188
public void unhandledTypeMismatchExceptionCallbackSeverity()
189189
throws IllegalAccessException, NoSuchFieldException {
190-
Report report;
190+
Event event;
191191
Callback callback = new Callback() {
192192
@Override
193-
public boolean onError(Report report) {
193+
public boolean onError(Event report) {
194194
report.setSeverity(Severity.WARNING);
195195
return true;
196196
}
@@ -201,7 +201,7 @@ public boolean onError(Report report) {
201201

202202
callUnhandledTypeMismatchExceptionEndpoint();
203203

204-
report = verifyAndGetReport(delivery);
204+
event = verifyAndGetReport(delivery);
205205
} finally {
206206
// Remove the callback via reflection so that subsequent tests do not use it
207207
Field callbacksField = Configuration.class.getDeclaredField("callbacks");
@@ -210,32 +210,32 @@ public boolean onError(Report report) {
210210
callbacks.remove(callback);
211211
}
212212

213-
assertTrue(report.getUnhandled());
214-
assertEquals("warning", report.getSeverity());
215-
assertEquals("userCallbackSetSeverity", report.getSeverityReason().getType());
213+
assertTrue(event.getUnhandled());
214+
assertEquals("warning", event.getSeverity());
215+
assertEquals("userCallbackSetSeverity", event.getSeverityReason().getType());
216216
}
217217

218218
@Test
219219
public void handledTypeMismatchExceptionUserSeverity() {
220220
callHandledTypeMismatchExceptionUserSeverityEndpoint();
221221

222-
Report report = verifyAndGetReport(delivery);
222+
Event event = verifyAndGetReport(delivery);
223223

224-
assertFalse(report.getUnhandled());
225-
assertEquals("warning", report.getSeverity());
226-
assertEquals("userSpecifiedSeverity", report.getSeverityReason().getType());
227-
assertThat(report.getSeverityReason().getAttributes(), is(Collections.EMPTY_MAP));
224+
assertFalse(event.getUnhandled());
225+
assertEquals("warning", event.getSeverity());
226+
assertEquals("userSpecifiedSeverity", event.getSeverityReason().getType());
227+
assertThat(event.getSeverityReason().getAttributes(), is(Collections.EMPTY_MAP));
228228
}
229229

230230
@Test
231231
public void handledTypeMismatchExceptionCallbackSeverity() {
232232
callHandledTypeMismatchExceptionCallbackSeverityEndpoint();
233233

234-
Report report = verifyAndGetReport(delivery);
234+
Event event = verifyAndGetReport(delivery);
235235

236-
assertFalse(report.getUnhandled());
237-
assertEquals("warning", report.getSeverity());
238-
assertEquals("userCallbackSetSeverity", report.getSeverityReason().getType());
236+
assertFalse(event.getUnhandled());
237+
assertEquals("warning", event.getSeverity());
238+
assertEquals("userCallbackSetSeverity", event.getSeverityReason().getType());
239239
}
240240

241241
private void callUnhandledTypeMismatchExceptionEndpoint() {

‎bugsnag-spring/src/test/java/com/bugsnag/SpringScheduledTaskTest.java‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -70,21 +70,21 @@ public void run() {
7070
// Run the task now and wait for it to finish
7171
scheduler.submit(exampleRunnable).get();
7272

73-
Report report = verifyAndGetReport(delivery);
73+
Event event = verifyAndGetReport(delivery);
7474

7575
// Assert that the exception was detected correctly
76-
assertEquals("Scheduled test", report.getExceptionMessage());
77-
assertEquals("java.lang.RuntimeException", report.getExceptionName());
76+
assertEquals("Scheduled test", event.getExceptionMessage());
77+
assertEquals("java.lang.RuntimeException", event.getExceptionName());
7878

7979
// Assert that the severity, severity reason and unhandled values are correct
80-
assertEquals(Severity.ERROR.getValue(), report.getSeverity());
80+
assertEquals(Severity.ERROR.getValue(), event.getSeverity());
8181
assertEquals(
8282
SeverityReasonType.REASON_UNHANDLED_EXCEPTION_MIDDLEWARE.toString(),
83-
report.getSeverityReason().getType());
83+
event.getSeverityReason().getType());
8484
assertThat(
85-
report.getSeverityReason().getAttributes(),
85+
event.getSeverityReason().getAttributes(),
8686
is(Collections.singletonMap("framework", "Spring")));
87-
assertTrue(report.getUnhandled());
87+
assertTrue(event.getUnhandled());
8888

8989
// Assert that the exception is passed to an existing exception handler
9090
ArgumentCaptor<RuntimeException> exceptionCaptor =

‎bugsnag-spring/src/test/java/com/bugsnag/TestUtils.java‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ class TestUtils {
1616
/**
1717
* Verify that a report was received, then capture and return that report
1818
*/
19-
static Report verifyAndGetReport(Delivery delivery) {
19+
static Event verifyAndGetReport(Delivery delivery) {
2020
ArgumentCaptor<Notification> notificationCaptor =
2121
ArgumentCaptor.forClass(Notification.class);
2222
verify(delivery, timeout(100).times(1)).deliver(

‎bugsnag-spring/src/test/java/com/bugsnag/testapp/springboot/TestController.java‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
import com.bugsnag.Bugsnag;
44

5-
import com.bugsnag.Report;
5+
import com.bugsnag.Event;
66
import com.bugsnag.Severity;
77
import com.bugsnag.callbacks.Callback;
88

@@ -67,8 +67,8 @@ public void handledTypeMismatchExceptionCallbackSeverity() {
6767
} catch (TypeMismatchException ex) {
6868
bugsnag.notify(ex, new Callback() {
6969
@Override
70-
public boolean onError(Report report) {
71-
report.setSeverity(Severity.WARNING);
70+
public boolean onError(Event event) {
71+
event.setSeverity(Severity.WARNING);
7272
return true;
7373
}
7474
});

0 commit comments

Comments
 (0)