RI-6188 There is no dropdown to specify how to add elements for adding a new list - tests updated

Author: Kristiyan Ivanov

redisinsight/api/src/modules/browser/list/list.service.spec.ts

@@ -93,6 +93,22 @@ describe('ListService', () => {
       ).resolves.not.toThrow();
       expect(service.createListWithExpiration).not.toHaveBeenCalled();
     });
+
+    it('create list with expiration and push at the head', async () => {
+      when(mockStandaloneRedisClient.sendCommand)
+        .calledWith([
+          BrowserToolListCommands.LPush,
+          mockPushElementDto.keyName,
+          ...mockPushElementDto.elements,
+        ])
+        .mockResolvedValue(1);
+
+      await expect(
+        service.createList(mockBrowserClientMetadata, mockPushElementDto),
+      ).resolves.not.toThrow();
+      expect(service.createListWithExpiration).not.toHaveBeenCalled();
+    });
+
     it('key with this name exist', async () => {
       when(mockStandaloneRedisClient.sendCommand)
         .calledWith([BrowserToolKeysCommands.Exists, mockPushElementDto.keyName])
@@ -475,7 +491,7 @@ describe('ListService', () => {
     it("shouldn't throw error", async () => {
       when(mockStandaloneRedisClient.sendPipeline)
         .calledWith([
-          [BrowserToolListCommands.LPush, dto.keyName, ...dto.elements],
+          [BrowserToolListCommands.RPush, dto.keyName, ...dto.elements],
           [BrowserToolKeysCommands.Expire, dto.keyName, dto.expire],
         ])
         .mockResolvedValue([
@@ -490,11 +506,11 @@ describe('ListService', () => {
     it('should throw error', async () => {
       const replyError: ReplyError = {
         ...mockRedisNoPermError,
-        command: 'LPUSH',
+        command: 'RPUSH',
       };
       when(mockStandaloneRedisClient.sendPipeline)
         .calledWith([
-          [BrowserToolListCommands.LPush, dto.keyName, ...dto.elements],
+          [BrowserToolListCommands.RPush, dto.keyName, ...dto.elements],
           [BrowserToolKeysCommands.Expire, dto.keyName, dto.expire],
         ])
         .mockResolvedValue([[replyError, []]]);

redisinsight/ui/src/pages/browser/components/add-key/AddKeyList/AddKeyList.spec.tsx

@@ -3,6 +3,7 @@ import { instance, mock } from 'ts-mockito'
 import { render, screen, fireEvent } from 'uiSrc/utils/test-utils'
 import AddKeyList, { Props } from './AddKeyList'
 import AddKeyFooter from '../AddKeyFooter/AddKeyFooter'
+import { HEAD_DESTINATION } from 'uiSrc/pages/browser/modules/key-details/components/list-details/add-list-elements/AddListElements'
 
 const mockedProps = mock<Props>()
 
@@ -33,6 +34,16 @@ describe('AddKeyList', () => {
     expect(valueInput).toHaveValue(value)
   })
 
+  it('should set destination properly', () => {
+    render(<AddKeyList {...instance(mockedProps)} />)
+    const destinationSelect = screen.getByTestId('destination-select')
+    fireEvent.change(
+      destinationSelect,
+      { target: { value: HEAD_DESTINATION } }
+    )
+    expect(destinationSelect).toHaveValue(HEAD_DESTINATION)
+  })
+
   it('should render disabled add key button with empty keyName', () => {
     const { container } = render(<AddKeyList {...instance(mockedProps)} />)
     expect(container.querySelector('.btn-add')).toBeDisabled()
@@ -52,7 +63,6 @@ describe('AddKeyList', () => {
 
   it('should not allow deleting the last element', () => {
     render(<AddKeyList {...instance(mockedProps)} keyName="name" />)
-    fireEvent.click(screen.getByTestId('add-item'))
     const deleteButtons = screen.getAllByTestId('remove-item')
     expect(deleteButtons[0]).toBeDisabled()
   })

redisinsight/ui/src/pages/browser/modules/key-details/components/list-details/add-list-elements/AddListElements.spec.tsx

@@ -41,7 +41,6 @@ describe('AddListElements', () => {
 
   it('should not allow deleting the last element', () => {
     render(<AddListElements {...instance(mockedProps)} />)
-    fireEvent.click(screen.getByTestId('add-item'))
     const deleteButtons = screen.getAllByTestId('remove-item')
     expect(deleteButtons[0]).toBeDisabled()
   })