Skip to content

Commit 5a07329

Browse files
committed
test: temp disabling of gfortran lint test
Revert commit when [FEAT] Gfortran v11+enhanced linting support #523 is fullfiled
1 parent 375cc16 commit 5a07329

File tree

1 file changed

+18
-17
lines changed

1 file changed

+18
-17
lines changed

test/linter-provider.test.ts

Lines changed: 18 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -28,24 +28,25 @@ suite('Linter integration', () => {
2828
doc = await workspace.openTextDocument(fileUri);
2929
await window.showTextDocument(doc);
3030
});
31+
// FIXME: different versions of gfortran report the error at a different column number
32+
// need to implement a the compiler versioning see #523
33+
// test('GNU - API call to doLint produces correct diagnostics', async () => {
34+
// const diags = await new FortranLintingProvider()['doLint'](doc);
35+
// const ref: Diagnostic[] = [
36+
// new Diagnostic(
37+
// new Range(new Position(21 - 1, 18 - 1), new Position(21 - 1, 18 - 1)),
38+
// 'Syntax error in argument list at (1)',
39+
// DiagnosticSeverity.Error
40+
// ),
41+
// new Diagnostic(
42+
// new Range(new Position(7 - 1, 9 - 1), new Position(7 - 1, 9 - 1)),
43+
// "Type specified for intrinsic function 'size' at (1) is ignored [-Wsurprising]",
44+
// DiagnosticSeverity.Warning
45+
// ),
46+
// ];
3147

32-
test('GNU - API call to doLint produces correct diagnostics', async () => {
33-
const diags = await new FortranLintingProvider()['doLint'](doc);
34-
const ref: Diagnostic[] = [
35-
new Diagnostic(
36-
new Range(new Position(21 - 1, 18 - 1), new Position(21 - 1, 18 - 1)),
37-
'Syntax error in argument list at (1)',
38-
DiagnosticSeverity.Error
39-
),
40-
new Diagnostic(
41-
new Range(new Position(7 - 1, 9 - 1), new Position(7 - 1, 9 - 1)),
42-
"Type specified for intrinsic function 'size' at (1) is ignored [-Wsurprising]",
43-
DiagnosticSeverity.Warning
44-
),
45-
];
46-
47-
deepStrictEqual(diags, ref);
48-
});
48+
// deepStrictEqual(diags, ref);
49+
// });
4950

5051
test('Include path globs & internal variable resolution', async () => {
5152
const paths = linter['getGlobPathsFromSettings']('linter.includePaths');

0 commit comments

Comments
 (0)