Bugfix/ri 7625 handle no such index error

redisinsight/api/src/__mocks__/errors.ts

@@ -25,6 +25,12 @@ export const mockRedisUnknownIndexName: ReplyError = {
   message: 'Unknown Index name',
 };
 
+export const mockRedisUnknownIndexNameV8: ReplyError = {
+  name: 'ReplyError',
+  command: 'FT.INFO',
+  message: 'idx: no such index',
+};
+
 export const mockRedisWrongNumberOfArgumentsError: ReplyError = {
   name: 'ReplyError',
   command: 'GET',

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

@@ -13,6 +13,7 @@ import {
   mockDatabaseClientFactory,
   mockRedisNoPermError,
   mockRedisUnknownIndexName,
+  mockRedisUnknownIndexNameV8,
   mockStandaloneRedisClient,
 } from 'src/__mocks__';
 import { RedisearchService } from 'src/modules/browser/redisearch/redisearch.service';
@@ -155,6 +156,38 @@ describe('RedisearchService', () => {
         { replyEncoding: 'utf8' },
       );
     });
+    it('should create index for standalone v8', async () => {
+      when(standaloneClient.sendCommand)
+        .calledWith(expect.arrayContaining(['FT.INFO']))
+        .mockRejectedValue(mockRedisUnknownIndexNameV8);
+      when(standaloneClient.sendCommand)
+        .calledWith(expect.arrayContaining(['FT.CREATE']))
+        .mockResolvedValue('OK');
+
+      await service.createIndex(
+        mockBrowserClientMetadata,
+        mockCreateRedisearchIndexDto,
+      );
+
+      expect(standaloneClient.sendCommand).toHaveBeenCalledTimes(2);
+      expect(standaloneClient.sendCommand).toHaveBeenCalledWith(
+        [
+          'FT.CREATE',
+          mockCreateRedisearchIndexDto.index,
+          'ON',
+          mockCreateRedisearchIndexDto.type,
+          'PREFIX',
+          2,
+          ...mockCreateRedisearchIndexDto.prefixes,
+          'SCHEMA',
+          mockCreateRedisearchIndexDto.fields[0].name,
+          mockCreateRedisearchIndexDto.fields[0].type,
+          mockCreateRedisearchIndexDto.fields[1].name,
+          mockCreateRedisearchIndexDto.fields[1].type,
+        ],
+        { replyEncoding: 'utf8' },
+      );
+    });
     it('should create index for cluster', async () => {
       databaseClientFactory.getOrCreateClient = jest
         .fn()

redisinsight/api/src/modules/browser/redisearch/redisearch.service.ts

@@ -104,7 +104,13 @@
           );
         }
       } catch (error) {
-        if (!error.message?.toLowerCase()?.includes('unknown index name')) {
+        const noIndexMessages = ['unknown index name', 'no such index'];
+
+        if (
+          !noIndexMessages.some((keyword) =>
+            error.message?.toLowerCase().includes(keyword),
+          )
+        ) {
           throw error;
         }
       }

redisinsight/ui/src/components/base/layout/horizontal-rule/HorizontalRule.tsx

@@ -10,6 +10,8 @@ const HorizontalRule = ({
   className,
   size = 'full',
   margin = 'l',
+  color,
+  colorVariable,
   ...rest
 }: HorizontalRuleProps) => {
   const classes = classNames('RI-horizontal-rule', className)
@@ -19,9 +21,11 @@ const HorizontalRule = ({
       size={size}
       margin={margin}
       className={classes}
+      color={color}
+      colorVariable={colorVariable}
       {...rest}
     />
   )
 }
 
-export default HorizontalRule
+export default HorizontalRule

redisinsight/ui/src/components/base/layout/horizontal-rule/horizontal-rule.styles.ts

@@ -47,12 +47,16 @@ const horizontalRuleStyles = {
 export interface HorizontalRuleProps extends HTMLAttributes<HTMLHRElement> {
   size?: HorizontalRuleSize
   margin?: HorizontalRuleMargin
+  color?: string
+  colorVariable?: string
 }
 
 export const StyledHorizontalRule = styled.hr<
   Omit<HorizontalRuleProps, 'size' | 'margin'> & {
     size?: HorizontalRuleSize
     margin?: HorizontalRuleMargin
+    color?: string
+    colorVariable?: string
   }
 >`
   ${({ size = 'full' }) => horizontalRuleStyles.size[size]}
@@ -65,6 +69,9 @@ export const StyledHorizontalRule = styled.hr<
   flex-shrink: 0;
   flex-grow: 0;
 
-  background-color: var(--hrBackgroundColor);
+  background-color: ${({ color, colorVariable }) =>
+    color || colorVariable
+      ? (color ?? `var(--${colorVariable})`)
+      : 'var(--hrBackgroundColor)'};
   height: 1px;
 `

redisinsight/ui/src/pages/browser/components/create-redisearch-index/CreateRedisearchIndex.tsx

@@ -268,7 +268,7 @@ const CreateRedisearchIndex = ({ onClosePanel, onCreateIndex }: Props) => {
             />
           </FlexItem>
         </Row>
-        <HorizontalRule margin="s" />
+        <HorizontalRule margin="s" colorVariable="separatorColor" />
         <Col grow={false} gap="s">
           <Row align="center" gap="xs">
             <Text>Identifier</Text>
@@ -317,7 +317,7 @@ const CreateRedisearchIndex = ({ onClosePanel, onCreateIndex }: Props) => {
           </AddMultipleFields>
         </Col>
       </StyledContent>
-      <HorizontalRule margin="xs" />
+      <HorizontalRule margin="xs" colorVariable="separatorColor" />
       <StyledFooter justify="end" gap="m">
         <FlexItem>
           <SecondaryButton

redisinsight/ui/src/pages/browser/components/create-redisearch-index/CreateRedisearchIndexWrapper.tsx

@@ -75,7 +75,7 @@ const CreateRedisearchIndexWrapper = ({
       </FlexItem>
       <Spacer size="xl" />
     </StyledHeader>
-    <HorizontalRule margin="xs" />
+    <HorizontalRule margin="xs" colorVariable="separatorColor" />
     <CreateRedisearchIndex
       onCreateIndex={onCreateIndex}
       onClosePanel={onClosePanel}