Skip to content

Commit 7064677

Browse files
committed
Remove unnecessary diff output log for test errors
1 parent 0093af8 commit 7064677

File tree

3 files changed

+8
-3
lines changed

3 files changed

+8
-3
lines changed

extensions/ql-vscode/src/cli.ts

+1
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,7 @@ export interface TestCompleted {
123123
expected: string;
124124
diff: string[] | undefined;
125125
failureDescription?: string;
126+
failureStage?: string;
126127
}
127128

128129
/**

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

+3-1
Original file line numberDiff line numberDiff line change
@@ -294,7 +294,9 @@ export class QLTestAdapter extends DisposableObject implements TestAdapter {
294294
: 'failed';
295295
let message: string | undefined;
296296
if (event.failureDescription || event.diff?.length) {
297-
message = ['', `${state}: ${event.test}`, event.failureDescription || event.diff?.join('\n'), ''].join('\n');
297+
message = event.failureStage === 'RESULT'
298+
? ['', `${state}: ${event.test}`, event.failureDescription || event.diff?.join('\n'), ''].join('\n')
299+
: ['', `${event.failureStage?.toLowerCase()} error: ${event.test}`, event.failureDescription || `${event.messages[0].severity}: ${event.messages[0].message}`, ''].join('\n');
298300
void testLogger.log(message);
299301
}
300302
this._testStates.fire({

extensions/ql-vscode/src/vscode-tests/no-workspace/test-adapter.test.ts

+4-2
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ describe('test-adapter', () => {
9696
type: 'test',
9797
state: 'errored',
9898
test: gPath,
99-
message: `\nerrored: ${gPath}\npqr\nxyz\n`,
99+
message: `\ncompilation error: ${gPath}\nERROR: abc\n`,
100100
decorations: [
101101
{ line: 1, message: 'abc' }
102102
]
@@ -149,14 +149,16 @@ describe('test-adapter', () => {
149149
pass: false,
150150
diff: ['pqr', 'xyz'],
151151
// a compile error
152+
failureStage: 'COMPILATION',
152153
messages: [
153-
{ position: { line: 1 }, message: 'abc' }
154+
{ position: { line: 1 }, message: 'abc', severity: 'ERROR' }
154155
]
155156
});
156157
yield Promise.resolve({
157158
test: Uri.parse('file:/ab/c/e/f/h.ql').fsPath,
158159
pass: false,
159160
diff: ['jkh', 'tuv'],
161+
failureStage: 'RESULT',
160162
messages: []
161163
});
162164
})()

0 commit comments

Comments
 (0)