Skip to content

Commit a86adbd

Browse files
authored
Merge pull request #279 from jcreedcmu/jcreed/hide-empty-test-dirs
Only create nonempty test suites
2 parents b577c12 + 4968ad8 commit a86adbd

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

extensions/ql-vscode/src/test-adapter.ts

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -168,15 +168,20 @@ export class QLTestAdapter extends DisposableObject implements TestAdapter {
168168
this._tests.fire(<TestLoadStartedEvent>{ type: 'started' });
169169

170170
const testDirectories = this.qlTestDiscovery.testDirectories;
171+
const children = testDirectories.map(
172+
testDirectory => QLTestAdapter.createTestSuiteInfo(testDirectory, testDirectory.name)
173+
);
171174
const testSuite: TestSuiteInfo = {
172175
type: 'suite',
173176
label: 'CodeQL',
174177
id: '.',
175-
children: testDirectories.map(
176-
testDirectory => QLTestAdapter.createTestSuiteInfo(testDirectory, testDirectory.name))
178+
children
177179
};
178180

179-
this._tests.fire(<TestLoadFinishedEvent>{ type: 'finished', suite: testSuite });
181+
this._tests.fire(<TestLoadFinishedEvent>{
182+
type: 'finished',
183+
suite: children.length > 0 ? testSuite : undefined
184+
});
180185
}
181186

182187
public async run(tests: string[]): Promise<void> {

0 commit comments

Comments
 (0)