diff --git a/README.md b/README.md index 7c1ddef..96605a1 100644 --- a/README.md +++ b/README.md @@ -36,9 +36,6 @@ class ExampleApp { client.submitEvent( EventPluginContext.from( client.createLog("test-log").referenceId("test-reference-id").build())); - - // Submit async events using submitXXXAsync methods - client.submitLogAsync("Test log"); } } ``` @@ -98,7 +95,7 @@ client uses v4. In that case you have to explicitly declare v4 of the library in com.exceptionless exceptionless-client - 1.0-beta1 + 0.1.0 org.springframework.boot diff --git a/src/main/java/com/exceptionless/exceptionlessclient/ExceptionlessClient.java b/src/main/java/com/exceptionless/exceptionlessclient/ExceptionlessClient.java index b5d037f..037c5e9 100644 --- a/src/main/java/com/exceptionless/exceptionlessclient/ExceptionlessClient.java +++ b/src/main/java/com/exceptionless/exceptionlessclient/ExceptionlessClient.java @@ -15,6 +15,7 @@ import lombok.extern.slf4j.Slf4j; import java.time.LocalDate; +import java.time.OffsetDateTime; import java.util.Timer; import java.util.TimerTask; import java.util.concurrent.ExecutorService; @@ -176,7 +177,7 @@ public Event.EventBuilder createSessionStart() { } public Event.EventBuilder createEvent() { - return Event.builder().dataExclusions(configuration.getDataExclusions()).date(LocalDate.now()); + return Event.builder().dataExclusions(configuration.getDataExclusions()).date(OffsetDateTime.now()); } public void submitEvent(Event event) { diff --git a/src/main/java/com/exceptionless/exceptionlessclient/models/Event.java b/src/main/java/com/exceptionless/exceptionlessclient/models/Event.java index 4155b52..44aaa88 100644 --- a/src/main/java/com/exceptionless/exceptionlessclient/models/Event.java +++ b/src/main/java/com/exceptionless/exceptionlessclient/models/Event.java @@ -12,6 +12,7 @@ import lombok.EqualsAndHashCode; import java.time.LocalDate; +import java.time.OffsetDateTime; import java.util.*; import java.util.stream.Collectors; @@ -24,7 +25,7 @@ public class Event extends Model { private String type; private String source; - private LocalDate date; + private OffsetDateTime date; private Set tags; private String message; private String geo; @@ -36,7 +37,7 @@ public class Event extends Model { public Event( String type, String source, - LocalDate date, + OffsetDateTime date, Set tags, String message, String geo, @@ -47,7 +48,7 @@ public Event( Set dataExclusions) { this.type = type; this.source = source; - this.date = date == null ? LocalDate.now() : date; + this.date = date == null ? OffsetDateTime.now() : date; this.tags = tags == null ? new HashSet<>() : tags; this.message = message; this.geo = geo; @@ -167,7 +168,7 @@ public EventBuilder source(String source) { } @Override - public EventBuilder date(LocalDate date) { + public EventBuilder date(OffsetDateTime date) { super.date(date); return this; } diff --git a/src/main/java/com/exceptionless/exceptionlessclient/plugins/MergedEvent.java b/src/main/java/com/exceptionless/exceptionlessclient/plugins/MergedEvent.java index 3157e40..fe0e08a 100644 --- a/src/main/java/com/exceptionless/exceptionlessclient/plugins/MergedEvent.java +++ b/src/main/java/com/exceptionless/exceptionlessclient/plugins/MergedEvent.java @@ -6,6 +6,7 @@ import lombok.Data; import java.time.LocalDate; +import java.time.OffsetDateTime; @Data public class MergedEvent { @@ -24,7 +25,7 @@ public void incrementCount(long count) { event.setCount(event.getCount() + count); } - public void updateDate(LocalDate date) { + public void updateDate(OffsetDateTime date) { if (date.isAfter(event.getDate())) { event.setDate(date); } diff --git a/src/main/java/com/exceptionless/exceptionlessclient/submission/EventRequest.java b/src/main/java/com/exceptionless/exceptionlessclient/submission/EventRequest.java index 9c9d7a0..14b2e01 100644 --- a/src/main/java/com/exceptionless/exceptionlessclient/submission/EventRequest.java +++ b/src/main/java/com/exceptionless/exceptionlessclient/submission/EventRequest.java @@ -4,6 +4,7 @@ import lombok.Value; import java.time.LocalDate; +import java.time.OffsetDateTime; import java.util.Map; import java.util.Set; @@ -12,7 +13,7 @@ public class EventRequest { String type; String source; - LocalDate date; + OffsetDateTime date; Set tags; String message; String geo; diff --git a/src/test/java/com/exceptionless/exceptionlessclient/ExceptionlessClientTest.java b/src/test/java/com/exceptionless/exceptionlessclient/ExceptionlessClientTest.java index 4022fac..2e889c4 100644 --- a/src/test/java/com/exceptionless/exceptionlessclient/ExceptionlessClientTest.java +++ b/src/test/java/com/exceptionless/exceptionlessclient/ExceptionlessClientTest.java @@ -20,6 +20,7 @@ import org.mockito.junit.jupiter.MockitoExtension; import java.time.LocalDate; +import java.time.OffsetDateTime; import static org.assertj.core.api.Assertions.assertThat; import static org.mockito.Mockito.*; @@ -72,7 +73,7 @@ public void itCanSubmitAnExceptionEvent() { argThat( event -> event.getType().equals(EventType.ERROR.value()) - && event.getDate().equals(LocalDate.now()) + && event.getDate().toLocalDate().equals(OffsetDateTime.now().toLocalDate()) && event.getReferenceId() != null && event.getError().isPresent() && event.getEnvironmentInfo().isPresent() @@ -101,7 +102,7 @@ public void itCanSubmitUnhandledExceptionEvent() { argThat( event -> event.getType().equals(EventType.ERROR.value()) - && event.getDate().equals(LocalDate.now()) + && event.getDate().toLocalDate().equals(OffsetDateTime.now().toLocalDate()) && event.getReferenceId() != null && event.getError().isPresent() && event.getEnvironmentInfo().isPresent() @@ -123,7 +124,7 @@ public void itCanSubmitFeatureUsageEvent() { event -> event.getType().equals(EventType.USAGE.value()) && event.getSource().equals("test-feature") - && event.getDate().equals(LocalDate.now()) + && event.getDate().toLocalDate().equals(OffsetDateTime.now().toLocalDate()) && event.getReferenceId() != null && event.getEnvironmentInfo().isPresent())); assertThat(configuration.getLastReferenceIdManager().getLast()).isNotNull(); @@ -153,7 +154,7 @@ public void itCanSubmitALogEvent() { event.getType().equals(EventType.LOG.value()) && event.getSource().equals("test-log-source") && event.getMessage().equals("test-log-message") - && event.getDate().equals(LocalDate.now()) + && event.getDate().toLocalDate().equals(OffsetDateTime.now().toLocalDate()) && event.getReferenceId() != null && event.getEnvironmentInfo().isPresent() && event.getData().containsKey(EventPropertyKey.LOG_LEVEL.value()))); @@ -226,7 +227,7 @@ public void itCanSubmitANotFoundEvent() { event -> event.getType().equals(EventType.NOT_FOUND.value()) && event.getSource().equals("test-resource") - && event.getDate().equals(LocalDate.now()) + && event.getDate().toLocalDate().equals(OffsetDateTime.now().toLocalDate()) && event.getReferenceId() != null && event.getEnvironmentInfo().isPresent())); assertThat(configuration.getLastReferenceIdManager().getLast()).isNotNull(); @@ -254,7 +255,7 @@ public void itCanSubmitASessionStartEvent() { argThat( event -> event.getType().equals(EventType.SESSION.value()) - && event.getDate().equals(LocalDate.now()) + && event.getDate().toLocalDate().equals(OffsetDateTime.now().toLocalDate()) && event.getReferenceId() != null && event.getEnvironmentInfo().isPresent())); assertThat(configuration.getLastReferenceIdManager().getLast()).isNotNull(); diff --git a/src/test/java/com/exceptionless/exceptionlessclient/plugins/MergedEventTest.java b/src/test/java/com/exceptionless/exceptionlessclient/plugins/MergedEventTest.java index 8f79134..e9b59ab 100644 --- a/src/test/java/com/exceptionless/exceptionlessclient/plugins/MergedEventTest.java +++ b/src/test/java/com/exceptionless/exceptionlessclient/plugins/MergedEventTest.java @@ -9,6 +9,7 @@ import org.mockito.junit.jupiter.MockitoExtension; import java.time.LocalDate; +import java.time.OffsetDateTime; import static org.mockito.ArgumentMatchers.argThat; import static org.mockito.Mockito.times; @@ -23,7 +24,7 @@ public class MergedEventTest { public void setup() { mergedEvent = MergedEvent.builder() - .event(Event.builder().date(LocalDate.now()).build()) + .event(Event.builder().date(OffsetDateTime.now()).build()) .eventQueue(eventQueue) .build(); } @@ -38,7 +39,7 @@ public void itCanIncrementCount() { @Test public void itCanUpdateDate() { - LocalDate tomorrow = LocalDate.now().plusDays(1); + OffsetDateTime tomorrow = OffsetDateTime.now().plusDays(1); mergedEvent.updateDate(tomorrow); mergedEvent.resubmit(); @@ -47,18 +48,19 @@ public void itCanUpdateDate() { @Test public void itShouldNotUpdateDateIfSameAsTheEvent() { - mergedEvent.updateDate(LocalDate.now()); + OffsetDateTime now = OffsetDateTime.now(); + mergedEvent.updateDate(now); mergedEvent.resubmit(); - verify(eventQueue, times(1)).enqueue(argThat(event -> event.getDate().equals(LocalDate.now()))); + verify(eventQueue, times(1)).enqueue(argThat(event -> event.getDate().equals(now))); } @Test public void itShouldNotUpdateDateIfBeforeAsTheEvent() { - LocalDate yesterday = LocalDate.now().minusDays(1); + OffsetDateTime yesterday = OffsetDateTime.now().minusDays(1); mergedEvent.updateDate(yesterday); mergedEvent.resubmit(); - verify(eventQueue, times(1)).enqueue(argThat(event -> event.getDate().equals(LocalDate.now()))); + verify(eventQueue, times(1)).enqueue(argThat(event -> event.getDate().toLocalDate().equals(OffsetDateTime.now().toLocalDate()))); } }