Merge pull request #1301 from RedisInsight/fe/feature/RI-3679_fix-negative-pipeline

#RI-3679 - fix negative pipeline param

Author: rsergeenko

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

@@ -15,17 +15,20 @@ describe('getExecuteParams', () => {
     const btnParams3: CodeButtonParams = { pipeline: 'abc' }
     const btnParams4: CodeButtonParams = { results: 'single', mode: 'raw' }
     const btnParams5: CodeButtonParams = { results: 'single', mode: 'raw', pipeline: '4' }
+    const btnParams6: CodeButtonParams = { results: 'single', mode: 'raw', pipeline: '-4' }
 
     const expect1 = { activeRunQueryMode: RunQueryMode.Raw, resultsMode: ResultsMode.GroupMode, batchSize: 5 }
     const expect2 = { activeRunQueryMode: RunQueryMode.ASCII, resultsMode: ResultsMode.GroupMode, batchSize: 10 }
     const expect3 = { activeRunQueryMode: RunQueryMode.ASCII, resultsMode: ResultsMode.GroupMode, batchSize: 10 }
     const expect4 = { activeRunQueryMode: RunQueryMode.Raw, resultsMode: ResultsMode.Default, batchSize: 10 }
     const expect5 = { activeRunQueryMode: RunQueryMode.Raw, resultsMode: ResultsMode.Default, batchSize: 4 }
+    const expect6 = { activeRunQueryMode: RunQueryMode.Raw, resultsMode: ResultsMode.Default, batchSize: 10 }
 
     expect(getExecuteParams(btnParams1, paramsState)).toEqual(expect1)
     expect(getExecuteParams(btnParams2, paramsState)).toEqual(expect2)
     expect(getExecuteParams(btnParams3, paramsState)).toEqual(expect3)
     expect(getExecuteParams(btnParams4, paramsState)).toEqual(expect4)
     expect(getExecuteParams(btnParams5, paramsState)).toEqual(expect5)
+    expect(getExecuteParams(btnParams6, paramsState)).toEqual(expect6)
   })
 })

redisinsight/ui/src/utils/workbench.ts

@@ -20,7 +20,7 @@ const getExecuteParams = (params: CodeButtonParams = {}, state: ExecuteQueryPara
   } = state
   const { results, mode, pipeline } = params
 
-  const batchSize = (pipeline && isInteger(+pipeline)) ? +pipeline : batchSizeState
+  const batchSize = (pipeline && isInteger(+pipeline) && +pipeline >= 0) ? +pipeline : batchSizeState
   const resultsMode = (results && results in CodeButtonResults) ? CodeButtonResults[results] : resultsModeState
   const activeRunQueryMode = (mode && mode in CodeButtonRunQueryMode)
     ? CodeButtonRunQueryMode[mode]