#RI-4608 - remove GRAPH commands from command helper

Author: AmirAllayarovSofteq

redisinsight/ui/src/components/command-helper/CommandHelperWrapper.spec.tsx

@@ -193,4 +193,21 @@ describe('CliBodyWrapper', () => {
       unmount()
     })
   })
+
+  it('should not since when matched command is deprecated', () => {
+    const sinceId = 'cli-helper-since'
+    const cliHelperDefaultId = 'cli-helper-default'
+
+    cliSettingsSelector.mockImplementation(() => ({
+      matchedCommand: 'GRAPH.CONFIG SET',
+      isEnteringCommand: true,
+    }))
+
+    const { unmount, queryByTestId } = render(<CommandHelperWrapper />)
+
+    expect(queryByTestId(cliHelperDefaultId)).toBeInTheDocument()
+    expect(queryByTestId(sinceId)).not.toBeInTheDocument()
+
+    unmount()
+  })
 })

redisinsight/ui/src/components/command-helper/CommandHelperWrapper.tsx

@@ -11,7 +11,13 @@ import {
 import { sendEventTelemetry, TelemetryEvent } from 'uiSrc/telemetry'
 import { cliSettingsSelector } from 'uiSrc/slices/cli/cli-settings'
 import { appRedisCommandsSelector } from 'uiSrc/slices/app/redis-commands'
-import { generateArgs, generateArgsNames, getComplexityShortNotation } from 'uiSrc/utils'
+import {
+  generateArgs,
+  generateArgsNames,
+  getComplexityShortNotation,
+  removeDeprecatedModuleCommands,
+  checkDeprecatedModuleCommand,
+} from 'uiSrc/utils'
 import CommandHelper from './CommandHelper'
 import CommandHelperHeader from './CommandHelperHeader'
 
@@ -26,9 +32,12 @@ const CommandHelperWrapper = () => {
     searchingCommand,
     searchingCommandFilter
   } = useSelector(cliSettingsSelector)
-  const { spec: ALL_REDIS_COMMANDS, commandsArray: KEYS_OF_COMMANDS } = useSelector(appRedisCommandsSelector)
+  const { spec: ALL_REDIS_COMMANDS, commandsArray } = useSelector(appRedisCommandsSelector)
   const { instanceId = '' } = useParams<{ instanceId: string }>()
-  const lastMatchedCommand = (isEnteringCommand && matchedCommand) ? matchedCommand : searchedCommand
+  const lastMatchedCommand = (isEnteringCommand && matchedCommand && !checkDeprecatedModuleCommand(matchedCommand))
+    ? matchedCommand
+    : searchedCommand
+  const KEYS_OF_COMMANDS = removeDeprecatedModuleCommands(commandsArray)
   let searchedCommands: string[] = []
 
   useEffect(() => {

redisinsight/ui/src/constants/keys.ts

@@ -27,7 +27,6 @@ export const GROUP_TYPES_DISPLAY = Object.freeze({
   [KeyTypes.ReJSON]: 'JSON',
   [KeyTypes.JSON]: 'JSON',
   [KeyTypes.Stream]: 'Stream',
-  [ModulesKeyTypes.Graph]: 'Graph',
   [ModulesKeyTypes.TimeSeries]: 'TS',
   [CommandGroup.Bitmap]: 'Bitmap',
   [CommandGroup.Cluster]: 'Cluster',

redisinsight/ui/src/constants/workbenchResults.ts

@@ -24,14 +24,6 @@ export const MODULE_NOT_LOADED_CONTENT: { [key in RedisDefaultModules]?: any } =
     additionalText: ['These features enable multi-field queries, aggregation, exact phrase matching, numeric filtering, ', 'geo filtering and vector similarity semantic search on top of text queries.'],
     link: 'https://redis.io/docs/stack/search/'
   },
-  [RedisDefaultModules.Graph]: {
-    text: ['RedisGraph adds a Property Graph data structure to Redis. ', 'With this capability you can:'],
-    improvements: [
-      'Create graphs',
-      'Query property graphs using the Cypher query language with proprietary extensions'
-    ],
-    link: 'https://redis.io/docs/stack/graph/'
-  },
   [RedisDefaultModules.ReJSON]: {
     text: ['RedisJSON adds the capability to:'],
     improvements: [
@@ -58,7 +50,6 @@ export const MODULE_NOT_LOADED_CONTENT: { [key in RedisDefaultModules]?: any } =
 
 export const MODULE_TEXT_VIEW: { [key in RedisDefaultModules]?: string } = {
   [RedisDefaultModules.Bloom]: 'RedisBloom',
-  [RedisDefaultModules.Graph]: 'RedisGraph',
   [RedisDefaultModules.ReJSON]: 'RedisJSON',
   [RedisDefaultModules.Search]: 'RediSearch',
   [RedisDefaultModules.TimeSeries]: 'RedisTimeSeries',

redisinsight/ui/src/slices/interfaces/instances.ts

@@ -183,7 +183,6 @@ export const COMMAND_MODULES = {
   [RedisDefaultModules.Search]: REDISEARCH_MODULES,
   [RedisDefaultModules.ReJSON]: [RedisDefaultModules.ReJSON],
   [RedisDefaultModules.TimeSeries]: [RedisDefaultModules.TimeSeries],
-  [RedisDefaultModules.Graph]: [RedisDefaultModules.Graph],
   [RedisDefaultModules.Bloom]: [RedisDefaultModules.Bloom],
 }
 

redisinsight/ui/src/utils/cliHelper.tsx

@@ -162,9 +162,6 @@ const checkCommandModule = (command: string) => {
     case command.startsWith(ModuleCommandPrefix.TimeSeries): {
       return RedisDefaultModules.TimeSeries
     }
-    case command.startsWith(ModuleCommandPrefix.Graph): {
-      return RedisDefaultModules.Graph
-    }
     case command.startsWith(ModuleCommandPrefix.BF):
     case command.startsWith(ModuleCommandPrefix.CF):
     case command.startsWith(ModuleCommandPrefix.CMS):
@@ -221,6 +218,16 @@ const getCommandNameFromQuery = (
   }
 }
 
+const DEPRECATED_MODULES_PREFIXES = [
+  ModuleCommandPrefix.Graph
+]
+
+const checkDeprecatedModuleCommand = (command: string) =>
+  DEPRECATED_MODULES_PREFIXES.some((prefix) => command.startsWith(prefix))
+
+const removeDeprecatedModuleCommands = (commands: string[]) => commands
+  .filter((command) => !checkDeprecatedModuleCommand(command))
+
 export {
   cliParseTextResponse,
   cliParseTextResponseWithOffset,
@@ -239,4 +246,6 @@ export {
   getCommandNameFromQuery,
   wbSummaryCommand,
   replaceEmptyValue,
+  removeDeprecatedModuleCommands,
+  checkDeprecatedModuleCommand,
 }

redisinsight/ui/src/utils/tests/cliHelper.spec.ts

@@ -9,6 +9,8 @@ import {
   checkUnsupportedCommand,
   checkBlockingCommand,
   replaceEmptyValue,
+  removeDeprecatedModuleCommands,
+  checkDeprecatedModuleCommand,
 } from 'uiSrc/utils'
 import { MOCK_COMMANDS_SPEC } from 'uiSrc/constants'
 import { render, screen } from 'uiSrc/utils/test-utils'
@@ -89,7 +91,6 @@ const checkCommandModuleTests = [
   { input: 'FT.foo bar', expected: RedisDefaultModules.Search },
   { input: 'JSON.foo bar', expected: RedisDefaultModules.ReJSON },
   { input: 'TS.foo bar', expected: RedisDefaultModules.TimeSeries },
-  { input: 'GRAPH.foo bar', expected: RedisDefaultModules.Graph },
   { input: 'BF.foo bar', expected: RedisDefaultModules.Bloom },
   { input: 'CF.foo bar', expected: RedisDefaultModules.Bloom },
   { input: 'CMS.foo bar', expected: RedisDefaultModules.Bloom },
@@ -110,6 +111,19 @@ const checkBlockingCommandTests = [
   { input: [['foo', 'bar'], 'FT.foo bar'], expected: undefined },
 ]
 
+const checkDeprecatedModuleCommandTests = [
+  { input: 'FT.foo bar', expected: false },
+  { input: 'GRAPH foo bar', expected: false },
+  { input: 'GRAPH.foo bar', expected: true },
+  { input: 'FOO bar', expected: false },
+]
+
+const removeDeprecatedModuleCommandsTests = [
+  { input: ['FT.foo'], expected: ['FT.foo'] },
+  { input: ['GRAPH.foo', 'FT.foo'], expected: ['FT.foo'] },
+  { input: ['FOO', 'GRAPH.FOO', 'CF.FOO', 'GRAPH.BAR'], expected: ['FOO', 'CF.FOO'] },
+]
+
 describe('getCommandNameFromQuery', () => {
   test.each(getCommandNameFromQueryTests)('%j', ({ input, expected }) => {
     // @ts-ignore
@@ -185,3 +199,15 @@ describe('checkBlockingCommand', () => {
     expect(checkBlockingCommand(...input)).toEqual(expected)
   })
 })
+
+describe('checkDeprecatedModuleCommand', () => {
+  test.each(checkDeprecatedModuleCommandTests)('%j', ({ input, expected }) => {
+    expect(checkDeprecatedModuleCommand(input)).toEqual(expected)
+  })
+})
+
+describe('removeDeprecatedModuleCommands', () => {
+  test.each(removeDeprecatedModuleCommandsTests)('%j', ({ input, expected }) => {
+    expect(removeDeprecatedModuleCommands(input)).toEqual(expected)
+  })
+})