chore: remove InventoryID from AdvisoryUpdateEvent

This field is no longer required as part of the AdvisoryUpdateEvent
structure. Updated tests accordingly. Added a TODO for future system
context changes in notifications.

Author: katarinazaprazna

base/mqueue/advisory_update_event.go

@@ -12,7 +12,6 @@ import (
 type AdvisoryUpdateEvent struct {
 	RhAccountID int                    `json:"rh_account_id"`
 	WorkspaceID uuid.UUID              `json:"workspace_id"`
-	InventoryID uuid.UUID              `json:"inventory_id"`
 	AdvisoryIDs []int64                `json:"advisory_ids"`
 	ProducedAt  types.Rfc3339Timestamp `json:"produced_at"`
 }

base/mqueue/advisory_update_event_test.go

@@ -20,7 +20,6 @@ func TestAdvisoryUpdateEventMarshal(t *testing.T) {
 	event := AdvisoryUpdateEvent{
 		RhAccountID: 1,
 		WorkspaceID: testWorkspaceID,
-		InventoryID: uuid.New(),
 		AdvisoryIDs: []int64{101, 202, 303},
 		ProducedAt:  testNow,
 	}
@@ -33,7 +32,6 @@ func TestAdvisoryUpdateEventMarshal(t *testing.T) {
 	assert.NoError(t, err)
 	assert.Equal(t, event.RhAccountID, parsed.RhAccountID)
 	assert.Equal(t, event.WorkspaceID, parsed.WorkspaceID)
-	assert.Equal(t, event.InventoryID, parsed.InventoryID)
 	assert.Equal(t, event.AdvisoryIDs, parsed.AdvisoryIDs)
 	assert.NotNil(t, parsed.ProducedAt)
 }
@@ -45,14 +43,12 @@ func TestAdvisoryUpdateEventsWriteEvents(t *testing.T) {
 		{
 			RhAccountID: 1,
 			WorkspaceID: testWorkspaceID,
-			InventoryID: uuid.New(),
 			AdvisoryIDs: []int64{100, 200},
 			ProducedAt:  testNow,
 		},
 		{
 			RhAccountID: 2,
 			WorkspaceID: testWorkspaceID,
-			InventoryID: uuid.New(),
 			AdvisoryIDs: []int64{300},
 			ProducedAt:  testNow,
 		},
@@ -69,7 +65,6 @@ func TestAdvisoryUpdateEventsWriteEvents(t *testing.T) {
 	assert.NoError(t, err)
 	assert.Equal(t, events[0].RhAccountID, firstEvent.RhAccountID)
 	assert.Equal(t, events[0].WorkspaceID, firstEvent.WorkspaceID)
-	assert.Equal(t, events[0].InventoryID, firstEvent.InventoryID)
 	assert.Equal(t, events[0].AdvisoryIDs, firstEvent.AdvisoryIDs)
 	assert.NotNil(t, firstEvent.ProducedAt)
 
@@ -78,7 +73,6 @@ func TestAdvisoryUpdateEventsWriteEvents(t *testing.T) {
 	assert.NoError(t, err)
 	assert.Equal(t, events[1].RhAccountID, secondEvent.RhAccountID)
 	assert.Equal(t, events[1].WorkspaceID, secondEvent.WorkspaceID)
-	assert.Equal(t, events[1].InventoryID, secondEvent.InventoryID)
 	assert.Equal(t, events[1].AdvisoryIDs, secondEvent.AdvisoryIDs)
 	assert.NotNil(t, secondEvent.ProducedAt)
 }

base/notification/notification.go

@@ -15,6 +15,10 @@ const (
 	Application = "patch"
 )
 
+// TODO: Remove Context after migrating to the new aggregator component
+// Advisories apply to multiple systems, so for aggregated notifications, system-specific context is unnecessary
+// See: https://redhat.atlassian.net/browse/RHINENG-26543
+
 type Context struct {
 	InventoryID string      `json:"inventory_id"`
 	DisplayName string      `json:"display_name"`