Skip to content

Commit 1649b30

Browse files
authored
testing: fix can toggle inline test coverage for non-text files outside the workspace (microsoft#236386)
Fixes microsoft#235346
1 parent e689b91 commit 1649b30

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

src/vs/workbench/contrib/testing/browser/codeCoverageDecorations.ts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,8 @@ import { KeybindingWeight } from '../../../../platform/keybinding/common/keybind
3939
import { ILogService } from '../../../../platform/log/common/log.js';
4040
import { bindContextKey, observableConfigValue } from '../../../../platform/observable/common/platformObservableUtils.js';
4141
import { IQuickInputService, QuickPickInput } from '../../../../platform/quickinput/common/quickInput.js';
42-
import * as coverUtils from './codeCoverageDisplayUtils.js';
43-
import { testingCoverageMissingBranch, testingCoverageReport, testingFilterIcon, testingRerunIcon } from './icons.js';
44-
import { ManagedTestCoverageBars } from './testCoverageBars.js';
42+
import { ActiveEditorContext } from '../../../common/contextkeys.js';
43+
import { TEXT_FILE_EDITOR_ID } from '../../files/common/files.js';
4544
import { getTestingConfiguration, TestingConfigKeys } from '../common/configuration.js';
4645
import { TestCommandId } from '../common/constants.js';
4746
import { FileCoverage } from '../common/testCoverage.js';
@@ -50,6 +49,9 @@ import { TestId } from '../common/testId.js';
5049
import { ITestService } from '../common/testService.js';
5150
import { CoverageDetails, DetailType, IDeclarationCoverage, IStatementCoverage } from '../common/testTypes.js';
5251
import { TestingContextKeys } from '../common/testingContextKeys.js';
52+
import * as coverUtils from './codeCoverageDisplayUtils.js';
53+
import { testingCoverageMissingBranch, testingCoverageReport, testingFilterIcon, testingRerunIcon } from './icons.js';
54+
import { ManagedTestCoverageBars } from './testCoverageBars.js';
5355

5456
const CLASS_HIT = 'coverage-deco-hit';
5557
const CLASS_MISS = 'coverage-deco-miss';
@@ -772,6 +774,7 @@ registerAction2(class FilterCoverageToTestInEditor extends Action2 {
772774
TestingContextKeys.isTestCoverageOpen,
773775
TestingContextKeys.coverageToolbarEnabled.notEqualsTo(true),
774776
TestingContextKeys.hasPerTestCoverage,
777+
ActiveEditorContext.isEqualTo(TEXT_FILE_EDITOR_ID),
775778
),
776779
group: 'navigation',
777780
},

0 commit comments

Comments
 (0)