Fix incorrectly migrated prop name in MessageSearch

Author: AhyoungRyu

src/modules/MessageSearch/__test__/MessageSearchUI.integration.test.tsx

@@ -21,7 +21,7 @@ const mockLocalizationContext = {
 };
 
 const defaultMockState = {
-  isQueryInvalid: false,
+  isInvalid: false,
   searchString: '',
   requestString: '',
   currentChannel: null,
@@ -89,7 +89,7 @@ describe('MessageSearchUI Integration Tests', () => {
   it('handles error state and retry', async () => {
     const handleRetryToConnect = jest.fn();
     renderComponent(
-      { isQueryInvalid: true, searchString: 'error query', requestString: 'error query' },
+      { isInvalid: true, searchString: 'error query', requestString: 'error query' },
       { handleRetryToConnect },
     );
     expect(screen.getByText(mockStringSet.PLACE_HOLDER__RETRY_TO_CONNECT)).toBeInTheDocument();

src/modules/MessageSearch/components/MessageSearchUI/index.tsx

@@ -35,7 +35,7 @@ export const MessageSearchUI: React.FC<MessageSearchUIProps> = ({
 }: MessageSearchUIProps) => {
   const {
     state: {
-      isQueryInvalid,
+      isInvalid,
       searchString,
       requestString,
       currentChannel,
@@ -83,7 +83,7 @@ export const MessageSearchUI: React.FC<MessageSearchUIProps> = ({
     return stringSet.NO_TITLE;
   };
 
-  if (isQueryInvalid && searchString && requestString) {
+  if (isInvalid && searchString && requestString) {
     return renderPlaceHolderError?.() || (
       <div className="sendbird-message-search">
         <PlaceHolder

src/modules/MessageSearch/context/MessageSearchProvider.tsx

@@ -28,7 +28,7 @@ export interface MessageSearchState extends MessageSearchProviderProps {
   channelUrl: string;
   allMessages: ClientSentMessages[];
   loading: boolean;
-  isQueryInvalid: boolean;
+  isInvalid: boolean;
   initialized: boolean;
   currentChannel: GroupChannel | null;
   currentMessageSearchQuery: MessageSearchQuery | null;
@@ -45,7 +45,7 @@ const initialState: MessageSearchState = {
   channelUrl: '',
   allMessages: [],
   loading: false,
-  isQueryInvalid: false,
+  isInvalid: false,
   initialized: false,
   currentChannel: null,
   currentMessageSearchQuery: null,
@@ -55,6 +55,11 @@ const initialState: MessageSearchState = {
   selectedMessageId: null,
   searchString: '',
   requestString: '',
+  /**
+   * messageSearchDispatcher is no longer used.
+   * Please use useMessageSearch() to get the store and update the state.
+   */
+  // messageSearchDispatcher: null,
 };
 
 export const MessageSearchContext = createContext<ReturnType<typeof createStore<MessageSearchState>> | null>(null);
@@ -162,9 +167,7 @@ const MessageSearchProvider: React.FC<MessageSearchProviderProps> = ({
   );
 };
 
-/**
- * Keep this function for backward compatibility.
- */
+// Keep this function for backward compatibility.
 const useMessageSearchContext = () => {
   const { state, actions } = useMessageSearch();
   return { ...state, ...actions };

src/modules/MessageSearch/context/__tests__/MessageSearch.migration.spec.tsx

@@ -59,11 +59,13 @@ describe('MessageSearch Migration Compatibility Tests', () => {
       'children',
       'requestString',
       'retryCount',
+      'setRetryCount',
       'selectedMessageId',
       'setSelectedMessageId',
       'scrollRef',
       'allMessages',
       'loading',
+      'isInvalid',
       'currentChannel',
       'currentMessageSearchQuery',
       'hasMoreResult',

…/__test__/MessageSearchProvider.spec.tsx …__tests__/MessageSearchProvider.spec.tsxsrc/modules/MessageSearch/context/__test__/MessageSearchProvider.spec.tsx renamed to src/modules/MessageSearch/context/__tests__/MessageSearchProvider.spec.tsx src/modules/MessageSearch/context/__test__/MessageSearchProvider.spec.tsx renamed to src/modules/MessageSearch/context/__tests__/MessageSearchProvider.spec.tsx

@@ -55,7 +55,7 @@ describe('MessageSearchProvider', () => {
   const initialState = {
     allMessages: [],
     loading: false,
-    isQueryInvalid: false,
+    isInvalid: false,
     initialized: false,
     currentChannel: null,
     currentMessageSearchQuery: null,
@@ -113,7 +113,7 @@ describe('MessageSearchProvider', () => {
         expect(newState.channelUrl).toBe(newUrl);
         expect(newState.allMessages).toEqual(initialState.allMessages);
         expect(newState.loading).toBe(initialState.loading);
-        expect(newState.isQueryInvalid).toBe(initialState.isQueryInvalid);
+        expect(newState.isInvalid).toBe(initialState.isInvalid);
       });
     });
   });
@@ -154,7 +154,7 @@ describe('MessageSearchProvider', () => {
         expect(updatedState.initialized).toBe(true);
         // Verify other states remain unchanged
         expect(updatedState.loading).toBe(initialState.loading);
-        expect(updatedState.isQueryInvalid).toBe(initialState.isQueryInvalid);
+        expect(updatedState.isInvalid).toBe(initialState.isInvalid);
         expect(updatedState.allMessages).toEqual(initialState.allMessages);
         expect(updatedState.hasMoreResult).toBe(initialState.hasMoreResult);
       });
@@ -175,7 +175,7 @@ describe('MessageSearchProvider', () => {
       await waitFor(() => {
         const updatedState = result.current.state;
         expect(updatedState.loading).toBe(false);
-        expect(updatedState.isQueryInvalid).toBe(false);
+        expect(updatedState.isInvalid).toBe(false);
         // Verify other states remain unchanged
         expect(updatedState.allMessages).toEqual(initialState.allMessages);
         expect(updatedState.currentChannel).toBe(initialState.currentChannel);
@@ -205,7 +205,7 @@ describe('MessageSearchProvider', () => {
         expect(updatedState.hasMoreResult).toBe(true);
         expect(updatedState.currentMessageSearchQuery).toEqual(mockQuery);
         // Verify other states remain unchanged
-        expect(updatedState.isQueryInvalid).toBe(initialState.isQueryInvalid);
+        expect(updatedState.isInvalid).toBe(initialState.isInvalid);
         expect(updatedState.initialized).toBe(initialState.initialized);
       });
     });
@@ -264,7 +264,7 @@ describe('MessageSearchProvider', () => {
       result.current.actions.setQueryInvalid();
       await waitFor(() => {
         const updatedState = result.current.state;
-        expect(updatedState.isQueryInvalid).toBe(true);
+        expect(updatedState.isInvalid).toBe(true);
         // Verify other states remain unchanged
         expect(updatedState.allMessages).toEqual(initialState.allMessages);
         expect(updatedState.loading).toBe(initialState.loading);
@@ -291,7 +291,7 @@ describe('MessageSearchProvider', () => {
         const updatedState = result.current.state;
         expect(updatedState.allMessages).toEqual([]);
         // Verify other states remain unchanged
-        expect(updatedState.isQueryInvalid).toBe(initialState.isQueryInvalid);
+        expect(updatedState.isInvalid).toBe(initialState.isInvalid);
         expect(updatedState.currentChannel).toBe(initialState.currentChannel);
       });
     });

…/__test__/useGetSearchedMessages.spec.ts …__tests__/useGetSearchedMessages.spec.tssrc/modules/MessageSearch/context/__test__/useGetSearchedMessages.spec.ts renamed to src/modules/MessageSearch/context/__tests__/useGetSearchedMessages.spec.ts src/modules/MessageSearch/context/__test__/useGetSearchedMessages.spec.ts renamed to src/modules/MessageSearch/context/__tests__/useGetSearchedMessages.spec.ts

@@ -37,7 +37,7 @@ describe('useMessageSearch', () => {
     expect(result.current.state).toEqual(expect.objectContaining({
       allMessages: [],
       loading: false,
-      isQueryInvalid: false,
+      isInvalid: false,
       initialized: false,
       currentChannel: null,
       currentMessageSearchQuery: null,
@@ -63,7 +63,7 @@ describe('useMessageSearch', () => {
       result.current.actions.startMessageSearch();
     });
 
-    expect(result.current.state.isQueryInvalid).toBe(false);
+    expect(result.current.state.isInvalid).toBe(false);
     expect(result.current.state.loading).toBe(false);
   });
 
@@ -89,7 +89,7 @@ describe('useMessageSearch', () => {
       result.current.actions.setQueryInvalid();
     });
 
-    expect(result.current.state.isQueryInvalid).toBe(true);
+    expect(result.current.state.isInvalid).toBe(true);
   });
 
   it('updates state when getNextSearchedMessages is called', () => {