fix: Correctly store last event id (#99)

* fix: Correctly store last event id

* Revert LastEventID return type

When LastEventID() returns EventID (string) instead of *EventID, user
code gets simpler because it doesn't need a nil-check.

Author: kamilogorek

hub.go

@@ -210,7 +210,13 @@ func (hub *Hub) CaptureEvent(event *Event) *EventID {
 	if client == nil || scope == nil {
 		return nil
 	}
-	return client.CaptureEvent(event, nil, scope)
+	eventID := client.CaptureEvent(event, nil, scope)
+	if eventID != nil {
+		hub.lastEventID = *eventID
+	} else {
+		hub.lastEventID = ""
+	}
+	return eventID
 }
 
 // CaptureMessage calls the method of a same name on currently bound `Client` instance
@@ -221,7 +227,13 @@ func (hub *Hub) CaptureMessage(message string) *EventID {
 	if client == nil || scope == nil {
 		return nil
 	}
-	return client.CaptureMessage(message, nil, scope)
+	eventID := client.CaptureMessage(message, nil, scope)
+	if eventID != nil {
+		hub.lastEventID = *eventID
+	} else {
+		hub.lastEventID = ""
+	}
+	return eventID
 }
 
 // CaptureException calls the method of a same name on currently bound `Client` instance
@@ -232,7 +244,13 @@ func (hub *Hub) CaptureException(exception error) *EventID {
 	if client == nil || scope == nil {
 		return nil
 	}
-	return client.CaptureException(exception, &EventHint{OriginalException: exception}, scope)
+	eventID := client.CaptureException(exception, &EventHint{OriginalException: exception}, scope)
+	if eventID != nil {
+		hub.lastEventID = *eventID
+	} else {
+		hub.lastEventID = ""
+	}
+	return eventID
 }
 
 // AddBreadcrumb records a new breadcrumb.

hub_test.go

@@ -177,6 +177,19 @@ func TestLastEventID(t *testing.T) {
 	assertEqual(t, uuid, hub.LastEventID())
 }
 
+func TestLastEventIDUpdatesAfterCaptures(t *testing.T) {
+	hub, _, _ := setupHubTest()
+
+	messageID := hub.CaptureMessage("wat")
+	assertEqual(t, *messageID, hub.LastEventID())
+
+	errorID := hub.CaptureException(fmt.Errorf("wat"))
+	assertEqual(t, *errorID, hub.LastEventID())
+
+	eventID := hub.CaptureEvent(&Event{Message: "wat"})
+	assertEqual(t, *eventID, hub.LastEventID())
+}
+
 func TestLayerAccessingEmptyStack(t *testing.T) {
 	hub := &Hub{}
 	if hub.stackTop() != nil {