Live location sharing - don't trigger unread counts for beacon location events

src/Unread.ts

@@ -17,6 +17,7 @@ limitations under the License.
 import { Room } from "matrix-js-sdk/src/models/room";
 import { MatrixEvent } from "matrix-js-sdk/src/models/event";
 import { EventType } from "matrix-js-sdk/src/@types/event";
+import { M_BEACON } from "matrix-js-sdk/src/@types/beacon";
 
 import { MatrixClientPeg } from "./MatrixClientPeg";
 import shouldHideEvent from './shouldHideEvent';
@@ -44,6 +45,8 @@ export function eventTriggersUnreadCount(ev: MatrixEvent): boolean {
         case EventType.RoomAliases:
         case EventType.RoomCanonicalAlias:
         case EventType.RoomServerAcl:
+        case M_BEACON.name:
+        case M_BEACON.altName:
             return false;
     }
 

test/Unread-test.ts

@@ -22,7 +22,7 @@ import {
 } from "matrix-js-sdk/src/matrix";
 
 import { haveRendererForEvent } from "../src/events/EventTileFactory";
-import { getMockClientWithEventEmitter, mockClientMethodsUser } from "./test-utils";
+import { getMockClientWithEventEmitter, makeBeaconEvent, mockClientMethodsUser } from "./test-utils";
 import { eventTriggersUnreadCount } from "../src/Unread";
 
 jest.mock("../src/events/EventTileFactory", () => ({
@@ -92,6 +92,12 @@ describe('eventTriggersUnreadCount()', () => {
         expect(haveRendererForEvent).toHaveBeenCalledWith(alicesMessage, false);
     });
 
+    it('returns false for beacon locations', () => {
+        const beaconLocationEvent = makeBeaconEvent(aliceId);
+        expect(eventTriggersUnreadCount(beaconLocationEvent)).toBe(false);
+        expect(haveRendererForEvent).not.toHaveBeenCalled();
+    });
+
     const noUnreadEventTypes = [
         EventType.RoomMember,
         EventType.RoomThirdPartyInvite,