Browse Source

[Resolves #1221] Unclear parameters for Consumers` Search. Inconsistency of Search line on Consumers page. (#1429)

* Changed Search component placeholder

* Updates test

Co-authored-by: Damir Abdulganiev <dabdulganiev@provectus.com>
Co-authored-by: Roman Zabaluev <rzabaluev@provectus.com>
Damir Abdulganiev 3 years ago
parent
commit
8639403448

+ 1 - 1
kafka-ui-react-app/src/components/ConsumerGroups/List/List.tsx

@@ -22,7 +22,7 @@ const List: React.FC = () => {
       <PageHeading text="Consumers" />
       <PageHeading text="Consumers" />
       <ControlPanelWrapper hasInput>
       <ControlPanelWrapper hasInput>
         <Search
         <Search
-          placeholder="Search"
+          placeholder="Search by Consumer Group ID"
           value={searchText}
           value={searchText}
           handleSearch={handleInputChange}
           handleSearch={handleInputChange}
         />
         />

+ 4 - 1
kafka-ui-react-app/src/components/ConsumerGroups/List/__test__/List.spec.tsx

@@ -30,7 +30,10 @@ describe('List', () => {
 
 
     describe('when searched', () => {
     describe('when searched', () => {
       it('renders only searched consumers', () => {
       it('renders only searched consumers', () => {
-        userEvent.type(screen.getByPlaceholderText('Search'), 'groupId1');
+        userEvent.type(
+          screen.getByPlaceholderText('Search by Consumer Group ID'),
+          'groupId1'
+        );
         expect(screen.getByText('groupId1')).toBeInTheDocument();
         expect(screen.getByText('groupId1')).toBeInTheDocument();
         expect(screen.getByText('groupId2')).toBeInTheDocument();
         expect(screen.getByText('groupId2')).toBeInTheDocument();
       });
       });