[lib] Validate DM operations
ClosedPublic
Actions

Authored by tomek on Sep 26 2024, 3:14 AM.

Details

Reviewers

kamil
will

Commits

rCOMM23ee267213b4: [lib] Validate DM operations

Summary

Add an explicit validation with some logs.

https://linear.app/comm/issue/ENG-9190/alternative-solution-update-canbeprocessed-functions

Depends on D13464

Test Plan

Clicked through the app and verified that e.g. sending text messages, reacting, and creating sidebars work.
Created some invalid operations from code and checked if it is discarded.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tomek created this revision.Sep 26 2024, 3:14 AM

Herald added a subscriber: ashoat. · View Herald TranscriptSep 26 2024, 3:14 AM

Harbormaster completed remote builds in B31873: Diff 44591.Sep 26 2024, 3:31 AM

tomek requested review of this revision.Sep 26 2024, 3:31 AM

tomek added a child revision: D13490: [lib] Handle a case where viewer is added to a thread while being its member.Sep 26 2024, 4:16 AM

kamil accepted this revision.Sep 26 2024, 7:35 AM

This revision is now accepted and ready to land.Sep 26 2024, 7:35 AM

Rebase

Harbormaster completed remote builds in B31897: Diff 44622.Sep 27 2024, 3:39 AM

Closed by commit rCOMM23ee267213b4: [lib] Validate DM operations (authored by tomek). · Explain WhySep 27 2024, 3:54 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM23ee267213b4: [lib] Validate DM operations.

Revision Contents
Changeset List

Path

Size

lib/

shared/

dm-ops/

add-members-spec.js

6 lines

add-viewer-to-thread-members-spec.js

6 lines

change-thread-read-status-spec.js

8 lines

change-thread-settings-spec.js

10 lines

change-thread-subscription.js

6 lines

create-entry-spec.js

6 lines

create-sidebar-spec.js

6 lines

create-thread-spec.js

8 lines

6 lines

3 lines

6 lines

6 lines

6 lines

5 lines

remove-members-spec.js

6 lines

send-edit-message-spec.js

6 lines

send-multimedia-message-spec.js

8 lines

send-reaction-message-spec.js

6 lines

send-text-message-spec.js

6 lines

update-relationship-spec.js

6 lines

Diff 44626

View Options

lib/shared/dm-ops/add-members-spec.js

Show First 20 Lines • Show All 177 Lines • ▼ Show 20 Lines
export const IntroTipRouteName = 'IntroTip';		export const IntroTipRouteName = 'IntroTip';
export const CommunityDrawerTipRouteName = 'CommunityDrawerTip';		export const CommunityDrawerTipRouteName = 'CommunityDrawerTip';
export const HomeTabTipRouteName = 'HomeTabTip';		export const HomeTabTipRouteName = 'HomeTabTip';
export const MutedTabTipRouteName = 'MutedTabTip';		export const MutedTabTipRouteName = 'MutedTabTip';
export const NUXTipOverlayBackdropRouteName = 'NUXTipOverlayBackdrop';		export const NUXTipOverlayBackdropRouteName = 'NUXTipOverlayBackdrop';
export const QRAuthNavigatorRouteName = 'QRAuthNavigator';		export const QRAuthNavigatorRouteName = 'QRAuthNavigator';
export const QRAuthNotPrimaryDeviceRouteName = 'QRAuthNotPrimaryDevice';		export const QRAuthNotPrimaryDeviceRouteName = 'QRAuthNotPrimaryDevice';
export const ConnectSecondaryDeviceRouteName = 'ConnectSecondaryDevice';		export const ConnectSecondaryDeviceRouteName = 'ConnectSecondaryDevice';
		export const SecondaryDeviceConnectedRouteName = 'SecondaryDeviceConnected';

export type RootParamList = {		export type RootParamList = {
+LoggedOutModal: void,		+LoggedOutModal: void,
+App: void,		+App: void,
+ThreadPickerModal: ThreadPickerModalParams,		+ThreadPickerModal: ThreadPickerModalParams,
+AddUsersModal: AddUsersModalParams,		+AddUsersModal: AddUsersModalParams,
+CustomServerModal: CustomServerModalParams,		+CustomServerModal: CustomServerModalParams,
+ColorSelectorModal: ColorSelectorModalParams,		+ColorSelectorModal: ColorSelectorModalParams,
▲ Show 20 Lines • Show All 160 Lines • ▼ Show 20 Lines	export type UserProfileBottomSheetParamList = {
+UserProfileAvatarModal: UserProfileAvatarModalParams,		+UserProfileAvatarModal: UserProfileAvatarModalParams,
+UserRelationshipTooltipModal: UserRelationshipTooltipModalParams,		+UserRelationshipTooltipModal: UserRelationshipTooltipModalParams,
};		};

export type QRAuthNavigatorParamList = {		export type QRAuthNavigatorParamList = {
+SecondaryDeviceQRCodeScanner: void,		+SecondaryDeviceQRCodeScanner: void,
+QRAuthNotPrimaryDevice: void,		+QRAuthNotPrimaryDevice: void,
+ConnectSecondaryDevice: ConnectSecondaryDeviceParams,		+ConnectSecondaryDevice: ConnectSecondaryDeviceParams,
		+SecondaryDeviceConnected: void,
};		};

export type ScreenParamList = {		export type ScreenParamList = {
...RootParamList,		...RootParamList,
...OverlayParamList,		...OverlayParamList,
...TabParamList,		...TabParamList,
...ChatParamList,		...ChatParamList,
...ChatTopTabsParamList,		...ChatTopTabsParamList,
▲ Show 20 Lines • Show All 50 Lines • Show Last 20 Lines

View Options

lib/shared/dm-ops/add-viewer-to-thread-members-spec.js

Show First 20 Lines • Show All 177 Lines • ▼ Show 20 Lines
export const IntroTipRouteName = 'IntroTip';		export const IntroTipRouteName = 'IntroTip';
export const CommunityDrawerTipRouteName = 'CommunityDrawerTip';		export const CommunityDrawerTipRouteName = 'CommunityDrawerTip';
export const HomeTabTipRouteName = 'HomeTabTip';		export const HomeTabTipRouteName = 'HomeTabTip';
export const MutedTabTipRouteName = 'MutedTabTip';		export const MutedTabTipRouteName = 'MutedTabTip';
export const NUXTipOverlayBackdropRouteName = 'NUXTipOverlayBackdrop';		export const NUXTipOverlayBackdropRouteName = 'NUXTipOverlayBackdrop';
export const QRAuthNavigatorRouteName = 'QRAuthNavigator';		export const QRAuthNavigatorRouteName = 'QRAuthNavigator';
export const QRAuthNotPrimaryDeviceRouteName = 'QRAuthNotPrimaryDevice';		export const QRAuthNotPrimaryDeviceRouteName = 'QRAuthNotPrimaryDevice';
export const ConnectSecondaryDeviceRouteName = 'ConnectSecondaryDevice';		export const ConnectSecondaryDeviceRouteName = 'ConnectSecondaryDevice';
		export const SecondaryDeviceConnectedRouteName = 'SecondaryDeviceConnected';

export type RootParamList = {		export type RootParamList = {
+LoggedOutModal: void,		+LoggedOutModal: void,
+App: void,		+App: void,
+ThreadPickerModal: ThreadPickerModalParams,		+ThreadPickerModal: ThreadPickerModalParams,
+AddUsersModal: AddUsersModalParams,		+AddUsersModal: AddUsersModalParams,
+CustomServerModal: CustomServerModalParams,		+CustomServerModal: CustomServerModalParams,
+ColorSelectorModal: ColorSelectorModalParams,		+ColorSelectorModal: ColorSelectorModalParams,
▲ Show 20 Lines • Show All 160 Lines • ▼ Show 20 Lines	export type UserProfileBottomSheetParamList = {
+UserProfileAvatarModal: UserProfileAvatarModalParams,		+UserProfileAvatarModal: UserProfileAvatarModalParams,
+UserRelationshipTooltipModal: UserRelationshipTooltipModalParams,		+UserRelationshipTooltipModal: UserRelationshipTooltipModalParams,
};		};

export type QRAuthNavigatorParamList = {		export type QRAuthNavigatorParamList = {
+SecondaryDeviceQRCodeScanner: void,		+SecondaryDeviceQRCodeScanner: void,
+QRAuthNotPrimaryDevice: void,		+QRAuthNotPrimaryDevice: void,
+ConnectSecondaryDevice: ConnectSecondaryDeviceParams,		+ConnectSecondaryDevice: ConnectSecondaryDeviceParams,
		+SecondaryDeviceConnected: void,
};		};

export type ScreenParamList = {		export type ScreenParamList = {
...RootParamList,		...RootParamList,
...OverlayParamList,		...OverlayParamList,
...TabParamList,		...TabParamList,
...ChatParamList,		...ChatParamList,
...ChatTopTabsParamList,		...ChatTopTabsParamList,
▲ Show 20 Lines • Show All 50 Lines • Show Last 20 Lines

View Options

lib/shared/dm-ops/change-thread-read-status-spec.js

Show First 20 Lines • Show All 177 Lines • ▼ Show 20 Lines
export const IntroTipRouteName = 'IntroTip';		export const IntroTipRouteName = 'IntroTip';
export const CommunityDrawerTipRouteName = 'CommunityDrawerTip';		export const CommunityDrawerTipRouteName = 'CommunityDrawerTip';
export const HomeTabTipRouteName = 'HomeTabTip';		export const HomeTabTipRouteName = 'HomeTabTip';
export const MutedTabTipRouteName = 'MutedTabTip';		export const MutedTabTipRouteName = 'MutedTabTip';
export const NUXTipOverlayBackdropRouteName = 'NUXTipOverlayBackdrop';		export const NUXTipOverlayBackdropRouteName = 'NUXTipOverlayBackdrop';
export const QRAuthNavigatorRouteName = 'QRAuthNavigator';		export const QRAuthNavigatorRouteName = 'QRAuthNavigator';
export const QRAuthNotPrimaryDeviceRouteName = 'QRAuthNotPrimaryDevice';		export const QRAuthNotPrimaryDeviceRouteName = 'QRAuthNotPrimaryDevice';
export const ConnectSecondaryDeviceRouteName = 'ConnectSecondaryDevice';		export const ConnectSecondaryDeviceRouteName = 'ConnectSecondaryDevice';
		export const SecondaryDeviceConnectedRouteName = 'SecondaryDeviceConnected';

export type RootParamList = {		export type RootParamList = {
+LoggedOutModal: void,		+LoggedOutModal: void,
+App: void,		+App: void,
+ThreadPickerModal: ThreadPickerModalParams,		+ThreadPickerModal: ThreadPickerModalParams,
+AddUsersModal: AddUsersModalParams,		+AddUsersModal: AddUsersModalParams,
+CustomServerModal: CustomServerModalParams,		+CustomServerModal: CustomServerModalParams,
+ColorSelectorModal: ColorSelectorModalParams,		+ColorSelectorModal: ColorSelectorModalParams,
▲ Show 20 Lines • Show All 160 Lines • ▼ Show 20 Lines	export type UserProfileBottomSheetParamList = {
+UserProfileAvatarModal: UserProfileAvatarModalParams,		+UserProfileAvatarModal: UserProfileAvatarModalParams,
+UserRelationshipTooltipModal: UserRelationshipTooltipModalParams,		+UserRelationshipTooltipModal: UserRelationshipTooltipModalParams,
};		};

export type QRAuthNavigatorParamList = {		export type QRAuthNavigatorParamList = {
+SecondaryDeviceQRCodeScanner: void,		+SecondaryDeviceQRCodeScanner: void,
+QRAuthNotPrimaryDevice: void,		+QRAuthNotPrimaryDevice: void,
+ConnectSecondaryDevice: ConnectSecondaryDeviceParams,		+ConnectSecondaryDevice: ConnectSecondaryDeviceParams,
		+SecondaryDeviceConnected: void,
};		};

export type ScreenParamList = {		export type ScreenParamList = {
...RootParamList,		...RootParamList,
...OverlayParamList,		...OverlayParamList,
...TabParamList,		...TabParamList,
...ChatParamList,		...ChatParamList,
...ChatTopTabsParamList,		...ChatTopTabsParamList,
▲ Show 20 Lines • Show All 50 Lines • Show Last 20 Lines

View Options

lib/shared/dm-ops/change-thread-settings-spec.js

Show First 20 Lines • Show All 177 Lines • ▼ Show 20 Lines
export const IntroTipRouteName = 'IntroTip';		export const IntroTipRouteName = 'IntroTip';
export const CommunityDrawerTipRouteName = 'CommunityDrawerTip';		export const CommunityDrawerTipRouteName = 'CommunityDrawerTip';
export const HomeTabTipRouteName = 'HomeTabTip';		export const HomeTabTipRouteName = 'HomeTabTip';
export const MutedTabTipRouteName = 'MutedTabTip';		export const MutedTabTipRouteName = 'MutedTabTip';
export const NUXTipOverlayBackdropRouteName = 'NUXTipOverlayBackdrop';		export const NUXTipOverlayBackdropRouteName = 'NUXTipOverlayBackdrop';
export const QRAuthNavigatorRouteName = 'QRAuthNavigator';		export const QRAuthNavigatorRouteName = 'QRAuthNavigator';
export const QRAuthNotPrimaryDeviceRouteName = 'QRAuthNotPrimaryDevice';		export const QRAuthNotPrimaryDeviceRouteName = 'QRAuthNotPrimaryDevice';
export const ConnectSecondaryDeviceRouteName = 'ConnectSecondaryDevice';		export const ConnectSecondaryDeviceRouteName = 'ConnectSecondaryDevice';
		export const SecondaryDeviceConnectedRouteName = 'SecondaryDeviceConnected';

export type RootParamList = {		export type RootParamList = {
+LoggedOutModal: void,		+LoggedOutModal: void,
+App: void,		+App: void,
+ThreadPickerModal: ThreadPickerModalParams,		+ThreadPickerModal: ThreadPickerModalParams,
+AddUsersModal: AddUsersModalParams,		+AddUsersModal: AddUsersModalParams,
+CustomServerModal: CustomServerModalParams,		+CustomServerModal: CustomServerModalParams,
+ColorSelectorModal: ColorSelectorModalParams,		+ColorSelectorModal: ColorSelectorModalParams,
▲ Show 20 Lines • Show All 160 Lines • ▼ Show 20 Lines	export type UserProfileBottomSheetParamList = {
+UserProfileAvatarModal: UserProfileAvatarModalParams,		+UserProfileAvatarModal: UserProfileAvatarModalParams,
+UserRelationshipTooltipModal: UserRelationshipTooltipModalParams,		+UserRelationshipTooltipModal: UserRelationshipTooltipModalParams,
};		};

export type QRAuthNavigatorParamList = {		export type QRAuthNavigatorParamList = {
+SecondaryDeviceQRCodeScanner: void,		+SecondaryDeviceQRCodeScanner: void,
+QRAuthNotPrimaryDevice: void,		+QRAuthNotPrimaryDevice: void,
+ConnectSecondaryDevice: ConnectSecondaryDeviceParams,		+ConnectSecondaryDevice: ConnectSecondaryDeviceParams,
		+SecondaryDeviceConnected: void,
};		};

export type ScreenParamList = {		export type ScreenParamList = {
...RootParamList,		...RootParamList,
...OverlayParamList,		...OverlayParamList,
...TabParamList,		...TabParamList,
...ChatParamList,		...ChatParamList,
...ChatTopTabsParamList,		...ChatTopTabsParamList,
▲ Show 20 Lines • Show All 50 Lines • Show Last 20 Lines