Skip to content

Commit 5110c8d

Browse files
authored
Merge pull request #4288 from RedisInsight/fe/bugfix/RI-6597
#RI-6597 - fix error
2 parents cd38993 + de528aa commit 5110c8d

File tree

2 files changed

+19
-1
lines changed

2 files changed

+19
-1
lines changed

redisinsight/ui/src/components/query/query-card/QueryCard.spec.tsx

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -158,6 +158,24 @@ describe('QueryCard', () => {
158158
expect(queryCliResultEl).toBeInTheDocument()
159159
})
160160

161+
it('should render properly result when it has pure number', () => {
162+
const { getByTestId } = render(
163+
<QueryCard
164+
{...instance(mockedProps)}
165+
resultsMode={ResultsMode.GroupMode}
166+
result={[{
167+
status: CommandExecutionStatus.Success,
168+
response: 1,
169+
}]}
170+
isOpen
171+
command="del key"
172+
/>
173+
)
174+
const queryCliResultEl = getByTestId('query-cli-result')
175+
176+
expect(queryCliResultEl.textContent).toBe('(integer) 1')
177+
})
178+
161179
it('should render QueryCardCliResult when result reached response size threshold even w/o flag', () => {
162180
const { queryByTestId } = render(
163181
<QueryCard

redisinsight/ui/src/components/query/query-card/QueryCard.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ const QueryCard = (props: Props) => {
155155
const isSizeLimitExceededResponse = (result: Maybe<CommandExecutionResult[]>) => {
156156
const resultObj = result?.[0]
157157
// response.includes - to be backward compatible with responses which don't include sizeLimitExceeded flag
158-
return resultObj?.sizeLimitExceeded === true || resultObj?.response?.includes('Results have been deleted')
158+
return resultObj?.sizeLimitExceeded === true || resultObj?.response?.includes?.('Results have been deleted')
159159
}
160160

161161
return (

0 commit comments

Comments
 (0)