Skip to content

Commit 70883fb

Browse files
tmatKevinRansom
authored andcommitted
Remove unused ProjectDiagnosticAnalyzer (#8304)
* Remove unused ProjectDiagnosticAnalyzer * Update proj file
1 parent ba87a35 commit 70883fb

File tree

3 files changed

+0
-90
lines changed

3 files changed

+0
-90
lines changed

vsintegration/src/FSharp.Editor/Diagnostics/ProjectDiagnosticAnalyzer.fs

Lines changed: 0 additions & 58 deletions
This file was deleted.

vsintegration/src/FSharp.Editor/FSharp.Editor.fsproj

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,6 @@
6161
<Compile Include="Debugging\BreakpointResolutionService.fs" />
6262
<Compile Include="Debugging\LanguageDebugInfoService.fs" />
6363
<Compile Include="Diagnostics\DocumentDiagnosticAnalyzer.fs" />
64-
<Compile Include="Diagnostics\ProjectDiagnosticAnalyzer.fs" />
6564
<Compile Include="Diagnostics\SimplifyNameDiagnosticAnalyzer.fs" />
6665
<Compile Include="Diagnostics\UnusedDeclarationsAnalyzer.fs" />
6766
<Compile Include="Diagnostics\UnusedOpensDiagnosticAnalyzer.fs" />

vsintegration/tests/UnitTests/ProjectDiagnosticAnalyzerTests.fs

Lines changed: 0 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -32,34 +32,3 @@ type ProjectDiagnosticAnalyzerTests() =
3232

3333
let args = mkProjectCommandLineArgs (dllName, [fileName])
3434
checker.GetProjectOptionsFromCommandLineArgs (projectName, args)
35-
36-
#if PROJECT_ANALYSIS
37-
[<Test>]
38-
member public this.ProjectDiagnosticsDontReportJustProjectErrors_Bug1596() =
39-
// https://github.com/Microsoft/visualfsharp/issues/1596
40-
let fileContents = """
41-
let x = 3
42-
printf "%d" x
43-
"""
44-
let options = CreateProjectAndGetOptions(fileContents)
45-
let additionalOptions = {options with OtherOptions = Array.append options.OtherOptions [| "--times" |]}
46-
47-
let errors = FSharpProjectDiagnosticAnalyzer.GetDiagnostics(additionalOptions) |> Async.RunSynchronously
48-
Assert.AreEqual(1, errors.Length, "Exactly one warning should have been reported")
49-
50-
let warning = errors.[0]
51-
Assert.AreEqual(DiagnosticSeverity.Warning, warning.Severity, "Diagnostic severity should be a warning")
52-
Assert.AreEqual("The command-line option 'times' is for test purposes only", warning.GetMessage())
53-
54-
[<Test>]
55-
member public this.ProjectDiagnosticsShouldNotReportDocumentErrors_Bug1596() =
56-
// https://github.com/Microsoft/visualfsharp/issues/1596
57-
let fileContents = """
58-
let x = "string value that cannot be printed with %d"
59-
printf "%d" x
60-
"""
61-
let options = CreateProjectAndGetOptions(fileContents)
62-
63-
let errors = FSharpProjectDiagnosticAnalyzer.GetDiagnostics(options) |> Async.RunSynchronously
64-
Assert.AreEqual(0, errors.Length, "No semantic errors should have been reported")
65-
#endif

0 commit comments

Comments
 (0)