Skip to content

Commit 0ed63ef

Browse files
authored
Remove unnecessary dependency from tests (#3248)
1 parent a1eed29 commit 0ed63ef

File tree

1 file changed

+3
-9
lines changed

1 file changed

+3
-9
lines changed

extension/src/test/suite/plots/data/index.test.ts

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -35,15 +35,12 @@ suite('Plots Data Test Suite', () => {
3535
})
3636

3737
const buildPlotsData = (
38-
experimentIsRunning: boolean,
3938
missingRevisions: string[] = [],
4039
mutableRevisions: string[] = []
4140
) => {
42-
const { internalCommands, updatesPaused, mockPlotsDiff, dvcRunner } =
41+
const { internalCommands, updatesPaused, mockPlotsDiff } =
4342
buildDependencies(disposable)
4443

45-
stub(dvcRunner, 'isExperimentRunning').returns(experimentIsRunning)
46-
4744
const mockGetMissingRevisions = stub().returns(missingRevisions)
4845
const mockGetMutableRevisions = stub().returns(mutableRevisions)
4946

@@ -70,7 +67,7 @@ suite('Plots Data Test Suite', () => {
7067
// eslint-disable-next-line sonarjs/cognitive-complexity
7168
describe('PlotsData', () => {
7269
it('should call plots diff when there are no revisions to fetch and no experiment is running (workspace updates)', async () => {
73-
const { data, mockPlotsDiff } = buildPlotsData(false, [], [])
70+
const { data, mockPlotsDiff } = buildPlotsData([], [])
7471

7572
await data.update()
7673

@@ -80,7 +77,6 @@ suite('Plots Data Test Suite', () => {
8077

8178
it('should call plots diff when an experiment is running in the workspace (live updates)', async () => {
8279
const { data, mockPlotsDiff } = buildPlotsData(
83-
true,
8480
[],
8581
[EXPERIMENT_WORKSPACE_ID]
8682
)
@@ -91,7 +87,7 @@ suite('Plots Data Test Suite', () => {
9187
})
9288

9389
it('should call plots diff when an experiment is running in a temporary directory (live updates)', async () => {
94-
const { data, mockPlotsDiff } = buildPlotsData(true, [], ['a7739b5'])
90+
const { data, mockPlotsDiff } = buildPlotsData([], ['a7739b5'])
9591

9692
await data.update()
9793

@@ -101,7 +97,6 @@ suite('Plots Data Test Suite', () => {
10197

10298
it('should call plots diff when an experiment is running and there are missing revisions (checkpoints)', async () => {
10399
const { data, mockPlotsDiff } = buildPlotsData(
104-
true,
105100
['53c3851', '4fb124a', '42b8736', '1ba7bcd'],
106101
[]
107102
)
@@ -120,7 +115,6 @@ suite('Plots Data Test Suite', () => {
120115

121116
it('should call plots diff when an experiment is running and there are missing revisions and one of them is mutable', async () => {
122117
const { data, mockPlotsDiff } = buildPlotsData(
123-
true,
124118
['53c3851', '4fb124a', '42b8736', '1ba7bcd'],
125119
['1ba7bcd']
126120
)

0 commit comments

Comments
 (0)