Add Fix::applicability to JSON output (#4341)

This commit is contained in:
Micha Reiser 2023-05-10 16:34:53 +02:00 committed by GitHub
parent 04097d194c
commit ddbe5a1243
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 7 additions and 4 deletions

View file

@ -39,6 +39,7 @@ impl Serialize for ExpandedMessages<'_> {
let fix = message.fix.as_ref().map(|fix| { let fix = message.fix.as_ref().map(|fix| {
json!({ json!({
"applicability": fix.applicability(),
"message": message.kind.suggestion.as_deref(), "message": message.kind.suggestion.as_deref(),
"edits": &ExpandedEdits { edits: fix.edits(), source_code: &source_code }, "edits": &ExpandedEdits { edits: fix.edits(), source_code: &source_code },
}) })

View file

@ -189,14 +189,13 @@ def fibonacci(n):
let fib_source = SourceFileBuilder::new("fib.py", fib).finish(); let fib_source = SourceFileBuilder::new("fib.py", fib).finish();
#[allow(deprecated)]
let unused_variable = Diagnostic::new( let unused_variable = Diagnostic::new(
UnusedVariable { UnusedVariable {
name: "x".to_string(), name: "x".to_string(),
}, },
TextRange::new(TextSize::from(94), TextSize::from(95)), TextRange::new(TextSize::from(94), TextSize::from(95)),
) )
.with_fix(Fix::unspecified(Edit::deletion( .with_fix(Fix::suggested(Edit::deletion(
TextSize::from(94), TextSize::from(94),
TextSize::from(99), TextSize::from(99),
))); )));

View file

@ -7,6 +7,7 @@ expression: content
"code": "F401", "code": "F401",
"message": "`os` imported but unused", "message": "`os` imported but unused",
"fix": { "fix": {
"applicability": "Suggested",
"message": "Remove unused import: `os`", "message": "Remove unused import: `os`",
"edits": [ "edits": [
{ {
@ -37,6 +38,7 @@ expression: content
"code": "F841", "code": "F841",
"message": "Local variable `x` is assigned to but never used", "message": "Local variable `x` is assigned to but never used",
"fix": { "fix": {
"applicability": "Suggested",
"message": "Remove assignment to unused variable `x`", "message": "Remove assignment to unused variable `x`",
"edits": [ "edits": [
{ {

View file

@ -93,6 +93,7 @@ fn test_stdin_json() -> Result<()> {
"code": "F401", "code": "F401",
"message": "`os` imported but unused", "message": "`os` imported but unused",
"fix": {{ "fix": {{
"applicability": "Unspecified",
"message": "Remove unused import: `os`", "message": "Remove unused import: `os`",
"edits": [ "edits": [
{{ {{