Microsoft/vscode
1
0
Fork 0
mirror of https://github.com/microsoft/vscode.git synced 2025-12-24 04:09:28 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add test for not caching cancelled response

Browse Source
This commit is contained in:
Matt Bierner
2018-12-10 11:32:43 -08:00
parent 863aee78b6
commit 68e0182064
1 changed files with 19 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
extensions/typescript-language-features/src/test/cachedResponse.test.ts
View File

@@ -8,7 +8,7 @@ import 'mocha';
import * as vscode from 'vscode';
import * as Proto from '../protocol';
import { CachedResponse } from '../tsServer/cachedResponse';
import { ServerResponse } from '../typescriptService';
import { ServerResponse, CancelledResponse } from '../typescriptService';
suite('CachedResponse', () => {
test('should cache simple response for same document', async () => {
@@ -16,7 +16,7 @@ suite('CachedResponse', () => {
const response = new CachedResponse();
let seq = 0;
const makeRequest = async () => createSuccessResponse(`test-${seq++}`);
const makeRequest = async () => createResponse(`test-${seq++}`);
assertResult(await response.execute(doc, makeRequest), 'test-0');
assertResult(await response.execute(doc, makeRequest), 'test-0');
@@ -28,7 +28,7 @@ suite('CachedResponse', () => {
const response = new CachedResponse();
let seq = 0;
const makeRequest = async () => createSuccessResponse(`test-${seq++}`);
const makeRequest = async () => createResponse(`test-${seq++}`);
assertResult(await response.execute(doc1, makeRequest), 'test-0');
assertResult(await response.execute(doc1, makeRequest), 'test-0');
@@ -37,6 +37,20 @@ suite('CachedResponse', () => {
assertResult(await response.execute(doc1, makeRequest), 'test-2');
assertResult(await response.execute(doc1, makeRequest), 'test-2');
});
test('should not cache canceled response', async () => {
const doc = await vscode.workspace.openTextDocument({ language: 'javascript', content: '' });
const response = new CachedResponse();
let seq = 0;
const makeRequest = async () => createResponse(`test-${seq++}`);
const result1 = await response.execute(doc, async () => new CancelledResponse('cancleed'));
assert.strictEqual(result1.type, 'cancelled');
assertResult(await response.execute(doc, makeRequest), 'test-0');
assertResult(await response.execute(doc, makeRequest), 'test-0');
});
});
function assertResult(result: ServerResponse<Proto.Response>, command: string) {
@@ -47,7 +61,7 @@ function assertResult(result: ServerResponse<Proto.Response>, command: string) {
}
}
function createSuccessResponse(command: string): Proto.Response {
function createResponse(command: string): Proto.Response {
return {
type: 'response',
body: {},
@@ -57,3 +71,4 @@ function createSuccessResponse(command: string): Proto.Response {
seq: 1
};
}