diff --git a/extensions/ql-vscode/src/history-item-label-provider.ts b/extensions/ql-vscode/src/history-item-label-provider.ts index 73fc30dbe..6a7e48906 100644 --- a/extensions/ql-vscode/src/history-item-label-provider.ts +++ b/extensions/ql-vscode/src/history-item-label-provider.ts @@ -57,7 +57,7 @@ export class HistoryItemLabelProvider { t: item.startTime, q: item.getQueryName(), d: item.initialInfo.databaseInfo.name, - r: `${resultCount} results`, + r: `(${resultCount} results)`, s: statusString, f: item.getQueryFileName(), '%': '%', diff --git a/extensions/ql-vscode/src/vscode-tests/no-workspace/history-item-label-provider.test.ts b/extensions/ql-vscode/src/vscode-tests/no-workspace/history-item-label-provider.test.ts index 2184a57d6..67727edb2 100644 --- a/extensions/ql-vscode/src/vscode-tests/no-workspace/history-item-label-provider.test.ts +++ b/extensions/ql-vscode/src/vscode-tests/no-workspace/history-item-label-provider.test.ts @@ -27,10 +27,10 @@ describe('HistoryItemLabelProvider', () => { expect(labelProvider.getLabel(fqi)).to.eq('xxx'); fqi.userSpecifiedLabel = '%t %q %d %s %f %r %%'; - expect(labelProvider.getLabel(fqi)).to.eq(`${dateStr} query-name db-name in progress query-file.ql 456 results %`); + expect(labelProvider.getLabel(fqi)).to.eq(`${dateStr} query-name db-name in progress query-file.ql (456 results) %`); fqi.userSpecifiedLabel = '%t %q %d %s %f %r %%::%t %q %d %s %f %r %%'; - expect(labelProvider.getLabel(fqi)).to.eq(`${dateStr} query-name db-name in progress query-file.ql 456 results %::${dateStr} query-name db-name in progress query-file.ql 456 results %`); + expect(labelProvider.getLabel(fqi)).to.eq(`${dateStr} query-name db-name in progress query-file.ql (456 results) %::${dateStr} query-name db-name in progress query-file.ql (456 results) %`); }); it('should interpolate query when not user specified', () => { @@ -40,10 +40,10 @@ describe('HistoryItemLabelProvider', () => { config.format = '%t %q %d %s %f %r %%'; - expect(labelProvider.getLabel(fqi)).to.eq(`${dateStr} query-name db-name in progress query-file.ql 456 results %`); + expect(labelProvider.getLabel(fqi)).to.eq(`${dateStr} query-name db-name in progress query-file.ql (456 results) %`); config.format = '%t %q %d %s %f %r %%::%t %q %d %s %f %r %%'; - expect(labelProvider.getLabel(fqi)).to.eq(`${dateStr} query-name db-name in progress query-file.ql 456 results %::${dateStr} query-name db-name in progress query-file.ql 456 results %`); + expect(labelProvider.getLabel(fqi)).to.eq(`${dateStr} query-name db-name in progress query-file.ql (456 results) %::${dateStr} query-name db-name in progress query-file.ql (456 results) %`); }); it('should get query short label', () => {