Skip to content

Commit e18b5bd

Browse files
authored
fix invalid workflow (#4817)
* fix invalid workflow * fix failing ui tests
1 parent 4ffccae commit e18b5bd

File tree

4 files changed

+12
-3
lines changed

4 files changed

+12
-3
lines changed

.github/workflows/code-coverage.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,6 @@ jobs:
103103
- name: Post or Update Coverage Summary Comment
104104
if: ${{ steps.findPr.outputs.number != '' }}
105105
uses: actions/github-script@v7
106-
if: ${{ steps.findPr.outputs.number != '' }}
107106
with:
108107
script: |
109108
const fs = require('fs');

redisinsight/ui/src/pages/database-analysis/components/header/Header.spec.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,7 @@ describe('DatabaseAnalysisHeader', () => {
140140

141141
const analizeButtonId = screen.getByTestId('start-database-analysis-btn')
142142
expect(analizeButtonId).toBeInTheDocument()
143-
expect(analizeButtonId).toHaveTextContent('Analyze')
143+
expect(analizeButtonId.textContent).toContain('New Report')
144144
})
145145

146146
it.skip('should call onChangeSelectedAnalysis after change selector', async () => {

redisinsight/ui/src/pages/rdi/pipeline-management/components/source-pipeline-dialog/SourcePipelineModal.spec.tsx

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,16 @@ jest.mock('uiSrc/telemetry', () => ({
3838
sendEventTelemetry: jest.fn(),
3939
}))
4040

41+
jest.mock('uiSrc/slices/rdi/pipeline', () => ({
42+
...jest.requireActual('uiSrc/slices/rdi/pipeline'),
43+
rdiPipelineSelector: jest.fn(),
44+
}))
45+
46+
jest.mock('uiSrc/slices/app/context', () => ({
47+
...jest.requireActual('uiSrc/slices/app/context'),
48+
appContextPipelineManagement: jest.fn(),
49+
}))
50+
4151
jest.mock('uiSrc/components/base/display', () => {
4252
const actual = jest.requireActual('uiSrc/components/base/display')
4353

redisinsight/ui/src/pages/rdi/pipeline-management/components/source-pipeline-dialog/SourcePipelineModal.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import React, { useState } from 'react'
1+
import React, { useEffect, useState } from 'react'
22
import { keys } from '@elastic/eui'
33
import { useDispatch, useSelector } from 'react-redux'
44
import { useParams } from 'react-router-dom'

0 commit comments

Comments
 (0)