test(presence): Add snapshot tests for NotificationsManager - Githubissues

microsoft / FluidFramework

Library for building distributed, real-time collaborative web applications

https://fluidframework.com

MIT License

4.74k stars 536 forks source link

test(presence): Add snapshot tests for NotificationsManager #23124

Open tylerbutler opened 1 week ago

tylerbutler commented 1 week ago

Adds a snapshot test for notifications manager that validates the following:

Notification messages are sent immediately
Notifications are sent separately (not batched - note that batching/throttling are not implemented currently so this test is more relevant for the future.

msfluid-bot commented 1 week ago

⯅ @fluid-example/bundle-size-tests: +245 Bytes

Metric Name	Baseline Size	Compare Size	Size Diff
aqueduct.js	467.24 KB	467.27 KB	⯅ +35 Bytes
azureClient.js	564.01 KB	564.06 KB	⯅ +49 Bytes
connectionState.js	724 Bytes	724 Bytes	■ No change
containerRuntime.js	263.43 KB	263.45 KB	⯅ +14 Bytes
fluidFramework.js	427.22 KB	427.23 KB	⯅ +14 Bytes
loader.js	134.18 KB	134.19 KB	⯅ +14 Bytes
map.js	42.71 KB	42.71 KB	⯅ +7 Bytes
matrix.js	150.15 KB	150.16 KB	⯅ +7 Bytes
odspClient.js	529.85 KB	529.89 KB	⯅ +49 Bytes
odspDriver.js	97.88 KB	97.9 KB	⯅ +21 Bytes
odspPrefetchSnapshot.js	42.81 KB	42.83 KB	⯅ +14 Bytes
sharedString.js	166.23 KB	166.24 KB	⯅ +7 Bytes
sharedTree.js	417.68 KB	417.68 KB	⯅ +7 Bytes
Total Size	3.38 MB	3.38 MB	⯅ +245 Bytes

Baseline commit: dadfe49432801d0e982a7653a5265c5dbde0e3c1

Generated by :no_entry_sign: dangerJS against bfd49127e618ed2c81b112dea82bec734a84e044